laravel 5.6 не читает переменную из контроллера в laravel

Question

Я новичок в Laravel, так что извините, если этот вопрос кажется глупым. Я пытаюсь сделать базовое приложение CRUD, кажется, что создание и чтение работают нормально, но у меня проблемы с редактированием, обновлением и удалением. Когда я нажимаю кнопку редактирования, она возвращает неопределенную переменную log_books, но, насколько я понимаю, я объявила об этом. Пожалуйста, помогите.

имя контроллера: logbook.php

namespace App\Http\Controllers;

use Request;

use Illuminate\Support\Facades\Input;

use App\Http\Requests;

use App\log_book;
use DB;
use Illuminate\Support\Facades\Auth;

class logbook extends Controller
{
/**
 * Display a listing of the resource.
 *
 * @return \Illuminate\Http\Response
 */
public function index()
{
    //
    /**
    *
    *@if(Auth::check())
    */

    $log_books = log_book::all();
    return view('home')->with('log_books',$log_books);

    /**
    *
    *@endif
    */

}

/**
 * Show the form for creating a new resource.
 *
 * @return \Illuminate\Http\Response
 */
public function create()
{
    //

/**
 *
 *   @if(Auth::check())
 */
        return view('log_books.create');
/**
 *
 *   @endif
 */
}

/**
 * Store a newly created resource in storage.
 *
 * @param  \Illuminate\Http\Request  $request
 * @return \Illuminate\Http\Response
 */
public function store(Request $request)
{
    //
    log_book::create(Request::all());
    return redirect('home')->with('message','name has been added successfully');
}

/**
 * Display the specified resource.
 *
 * @param  int  $id
 * @return \Illuminate\Http\Response
 */
public function show($id)
{
    //
}

/**
 * Show the form for editing the specified resource.
 *
 * @param  int  $id
 * @return \Illuminate\Http\Response
 */
public function edit($id)
{
    //
    /**
     *
     *@if(Auth::check())
     */

    $log_books = log_book::findorFail($id);
    //$logs = DB::table('log_books')->where('id', $id)->first();
    return view('log_books.edit', compact($log_books))->with('id', $id);

    /**
     *
     *@endif
     */


}

/**
 * Update the specified resource in storage.
 *
 * @param  \Illuminate\Http\Request  $request
 * @param  int  $id
 * @return \Illuminate\Http\Response
 */
public function update(Request $request, $id)
{
    //
}

/**
 * Remove the specified resource from storage.
 *
 * @param  int  $id
 * @return \Illuminate\Http\Response
 */
public function destroy($id)
{
    //
    $log_books = log_books::findorFail($id);

    $destroy = $log_books->delete();

    if($destroy) {
        return redirect('home', compact($log_books))->with('message', 'Record has been deleted');
    }
}
}

Мой маршрут: web.php

Route::get('/', function () {
return view('welcome');
});

Route::get('create', function () {
return view('create');
});

Auth::routes();

Route::get('/home', 'HomeController@index')->name('home');
Route::get('/create', 'logbook@create')->name('create');
Route::post('store', 'logbook@store');
Route::get('/home', 'logbook@index');
Route::get('edit/{id}', 'logbook@edit');
Route::delete('destroy/{id}', 'logbook@destroy');

My View: home.blade.php

 @foreach($log_books as $row)
                                    <tr>
                                        <td>{{$row->name}}</td>
                                        <td>{{$row->present_date}}</td>
                                        <td>{{$row->time_in}}</td>
                                        <td>{{$row->time_out}}</td>
                                        <td><a href="{{ URL::to('edit',$row->id) }}"><input type="button" class="btn btn-success" value="Edit"></a>&nbsp;<a href="{{ URL::to('destroy',$row->id) }}"><input type="button" class="btn btn-danger" value="Delete"></a></td>
                                    </tr>
 @endforeach

редактировать вид: edit.blade.php

<form method="post" action="edit">
                    {{csrf_field()}}

                    <input type="hidden" name="_method" value="PUT">
                    <input required="yes" type="text" placeholder="Full name" class="form-control" name="name" value="{{$log_books->name}}"><br>
                    <input required="yes" onfocus="this.type= 'date'" onblur="this.type='text'" placeholder="Date" class="form-control" name="present_date" value="{{$log_books->present_date}}"><br>
                    <input required="yes" onfocus="this.type= 'time'" onblur="this.type='text'" placeholder="Time in" class="form-control" name="time_in" value="{{$log_books->time_in}}"><br>
                    <input onfocus="this.type= 'time'" onblur="this.type='text'" placeholder="Time out" class="form-control" name="time_out" value="{{$log_books->time_out}}"><br>

                    <input type="submit" class="btn btn-primary" value="Submit" name="submit">
</form>

Простите мой код, если он выглядит испорченным, представления создания и редактирования находятся в папке с именем log_books. Спасибо за вашу помощь.

Answer 1

Ваш метод edit передает недопустимое значение в команду compact:

return view('log_books.edit', compact($log_books))->with('id', $id);

Скорее всего, это должно быть имя переменной, которую вы хотите сжать, а не сама переменная:

return view('log_books.edit', compact('log_books'))->with('id', $id);

Чтобы полностью избежать этих типов ошибок, просто определите массив:

return view('log_books.edit', ['log_books' => $log_books])...

Answer 2

Laravel имеет привязку модели маршрута, вы можете использовать эту функцию в методах show и edit следующим образом:

В контроллере

public function edit(Log_Book $log_book)
{
    return view('log_books.edit', compact($log_book));
}

В web.php

Route::get('/edit/{log_book}', 'logbook@edit');

Также вам не нужно проверять аутентификацию в методах контроллера, используйте auth middleware.

В контроллере

public function __construct()
{
    return $this->middleware('auth');
}