【laravel】Undefined variable: のエラーが表示される

Question

前提・実現したいこと

PHP(larvel)でタスク管理システムを作っています。
completed_atカラムに現在の日時を保存し、indexページをリダイレクトするタスクの完了ボタンを実装中に以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

todos/indexページ(完了ボタンを表示しようとしているページ)を
開いたところ、下記のようなメッセージが出ました。

Undefined variable: todo (View: C:\xampp\htdocs\Todolist_0813\Todo202008\Todo202008\Todo_laravel\resources\views\todos\index.blade.php)

該当のソースコード

        {{--index.blade.php内--}}
        {{--完了ボタン--}}
        <td><form action="{{ route('todo.update',['id' => $todo->id]) }}" method="post"> //エラーが出ている行
          {{ csrf_field() }}
          <button type="submit" class="btn btn-success">完了</button>
        </form>
        </td>

        {{--web.php--}}
Route::group(['prefix' => 'todos', 'middleware' => 'auth'], function(){
    Route::get('index', 'TodosController@index') ->name('todo.index');
    Route::get('create', 'TodosController@create') ->name('todo.create');
    Route::post('/index', 'TodosController@store') ->name('todo.store');
    Route::get('show/{id}', 'TodosController@show') ->name('todo.show');
    Route::get('edit/{id}', 'TodosController@edit') ->name('todo.edit');
    Route::post('update/{id}', 'TodosController@update') ->name('todo.update');
    Route::post('delete/{id}', 'TodosController@destroy') ->name('todo.destroy');
});

       {{--controller.php--}}
  public static function Date(){
      $date = date("y-m-d");
      return $date;
    }
public function update($id) {
      // 完了日を入力
      $todo = Todo::find($id);
      $todo->completed_at = self::Date();
      $todo->save();
        return redirect('todos/index',compact('todo'));
      }

    public function edit($id) {
      $todos = Todo::find($id);
      $frequency = frequency::all(); 
        return view('todos.edit',compact('todos','frequency'));
      }

    public static function index(){
        $frequencys = frequency::where('frequency','=', self::Datetime())
          ->whereHas('todos',function($q){
          $q->where('completed_at','<>', self::Date())
          ->orderBy('difficultyLevel', 'desc');
        })
        ->get();
        $datas = [];
        foreach ($frequencys as $frequency) {
          $datas[] = [
            'id' => $frequency->todos->id,
            'body' => $frequency->todos->body,
            'difficultyLevel' => $frequency->todos->difficultyLevel,
            'frequency' => $frequency->frequency
          ];
        }

        return view('todos/index',compact('frequencys','datas'));
    }

補足情報（FW/ツールのバージョンなど）

PHP 7.4.9
laravel 6.18.35
MySQL 5.7.26

Comments

[phper.k]

controller の edit()を提示してください。

[yambejp]

「Todo202008」が被ってますが大丈夫ですか？

[youplus40]

>yambejg　さん
被ると不具合が生じる可能性があるのでしょうか。

[yambejp]

> 被ると不具合が生じる可能性

pathが間違えているとか、パラメータがうまくとれないとか
可能性だけでいえばありとあらゆる問題が想定されます。
とりあえず違和感があるので指摘しました。
（別に確認する気がないなら放置してください）

Answer 1

    public function edit($id) {
-      $todos = Todo::find($id);
+      $todo = Todo::find($id);
      $frequency = frequency::all(); 
-        return view('todos.edit',compact('todos','frequency'));
+        return view('todos.edit',compact('todo','frequency'));
      }

Comments

[youplus40]

ご回答ありがとうございます。
しかしながら、コードを直しましたが同じエラーが出ております。

Answer 2

自己解決したので、コードを記載します。
index.blade.phpのforeach文の変数を$todoに、PACTHメソッドを使用、update()でcompactを削除して解決しました。

        <!--index.blade.php -->
        <!--完了ボタン -->
        <td><form action="{{ route('todo.update',$todo["id"]) }}" method="post">
          {{ csrf_field() }}
          {{ method_field('patch')}}
          <button type="submit" class="btn btn-success">完了</button>
        </form>
        </td>

        <!--web.php -->
Route::group(['prefix' => 'todos', 'middleware' => 'auth'], function(){
    Route::get('index', 'TodosController@index') ->name('todo.index');
    Route::get('create', 'TodosController@create') ->name('todo.create');
    Route::post('/index', 'TodosController@store') ->name('todo.store');
    Route::get('show/{id}', 'TodosController@show') ->name('todo.show');
    Route::get('edit/{id}', 'TodosController@edit') ->name('todo.edit');
    Route::patch('/todos/{id}','TodosController@update') ->name('todo.update');
    Route::post('delete/{id}', 'TodosController@destroy') ->name('todo.destroy');
});

    public function update($id) {
      $todo = Todo::find($id);
      $todo->completed_at = self::Date();
      $todo->save();
      return redirect('todos/index');

Answer 3

Undefined variable: todo

エラーを見ると、todoなんて定義されてないよ！って書いてありますね。
データを更新した際にこのエラーが表示されたということは、更新処理を行っている関数内で、
todoでない変数をViewに渡していることになります。

youplus40さんが更新処理を実装した際に定義した関数はeditなので、
そこで返している値を調べればいいわけですね。

てなわけで、phper.kさんの回答で問題が解消されるのではないかと。

Comments

[youplus40]

回答ありがとうございます。
「更新処理を実装した際に定義した関数はeditなので、」とありますが、
どの部分から判断されたのでしょうか。
index.blade.phpでは'todo.update'をルートとして選択しており、自分ではその認識がありませんでした。
何卒、ご教授のほど、よろしくお願いいたします。

[Suumon]

お…これは恥ずかしい。完全に読み間違いです。

再度、読み返してみました。
お手数ですが、Controller側のindex.blade.phpを表示する処理を上記に追記していただいてもよろしいしょうか？（TodosControllerクラスのindex）

[youplus40]

index部分を追記しましたので、ご確認いただければと存じます。

[Suumon]

index.blade.php内でforeach($datas as $todo)としたら動きますかね？

それでだめなら、どうしましょうね…。
原因はindex()かindex.blade.phpにあるかと思うので、デバックなりしてちゃんと想定した値が渡っているかとか調べるしかないかと。

[youplus40]

index.blade.phpを修正したところ、エラーはなくなりました。ありがとうございます。
ただ、completed_atにデータが入らず、ただindex.blade.phpにリダイレクトしているだけになります。
update()に問題があると思うのですが、var_dompを設定してもリダイレクトが起こるだけの状況です。

[Suumon]

エラーが解決されたようでなによりです。

上記の質問ですが、本質問とは別の問題ですので、新たに質問しなおしたほうが良いかと思います。
…とはいえ、ぱっと見気になったことがあったのでそれだけ指摘しておきますと、
〇 staticeに定義したDate()ですが、これは本当に動いているのでしょうか？
〇date("y-m-d")の戻り値を使用する程度の処理をわざわざ関数として分ける必要はないのではないでしょうか？

つまり、
```
$todo->completed_at = date("y-m-d");
```

[youplus40]

ありがとうございます。自己解決したコードで、Suumon様にお聞きしたエラーも改善しております。
○こちらは動いております。
○2箇所Date()を使用する部分があり、関数として使用しておりましたが、
　おっしゃるとおり、変更したところ問題なく動きましたので、関数として分ける必要はなかったです。
　コメントありがとうございます。