Laravel 親子関係で紐づけた子モデルのデータを更新したい

Question

実現したいこと

• 親子関係で紐づいた子モデルのステータスを更新したい

前提

Laravel初心者です。現在Todoアプリを制作しています。
大目標(Todo)とそれを達成するための小目標(TodoList)があり、それぞれ親子関係になっています。
子モデルの小目標が達成されたときに完了ボタンを押すことでステータスが未完了(false)→完了(true)になるようにしたいです。

• Todo（親モデル）
• TodoList（子モデル）
• TodoList（子モデル）にstatusカラム(boolean型)が存在し、デフォルトはfalseになっています。

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

完了ボタンを押した際下記のエラーが発生してしまいます。

エラーメッセージ

Attempt to assign property "status" on null

該当のソースコード

Todo.php

<?php
namespace App\Models;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;

class Todo extends Model
{
    // リレーション     
    // ひとつの大目標(todo)はいくつかの小目標（todo_lists)を持つことができる
      public function todo_lists()
      {
        return $this->hasMany('App\Models\TodoList');
      }
    // リレーション     
}

TodoList.php

<?php 
namespace App\Models;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;

class TodoList extends Model
{
    use HasFactory;
    
    // リレーション 
    // 小目標（todo_list）は一つの大目標（todo)を持つ
      public function todo()
      {
        return $this->belongsTo('App\Models\Todo');
      }
    // リレーション 
}

web.php

<?php

use App\Http\Controllers\ProfileController;
use Illuminate\Support\Facades\Route;
use App\Http\Controllers\TodoController;
use App\Http\Controllers\TodoListController;

// 小目標（todo_list）、大目標（todo)にネスト
Route::resource('todos', TodoController::class);
Route::resource('todo.todo_lists',TodoListController::class)

TodoListController.php

<?php

namespace App\Http\Controllers;
use Illuminate\Http\Request;
use App\Models\Todo;
use App\Models\TodoList;


class TodoListController extends Controller
{

    public function update(Request $request, $id)
    {   
        //該当データ検索
        $todo_list = TodoList::find($id);
        //ステータスを未完了（デフォルト）→完了 false:未完了 true:完了
        $todo_list->status= true;
        //データの保存
        $todo_list->save();
        // 詳細画面のリダイレクト
        return redirect()->route('todos.show',$todo_list->todo_id);
      
    }

TodoController.php

<?php
namespace App\Http\Controllers;
use Illuminate\Http\Request;
use App\Models\Todo;
use App\Models\User;
use App\Models\TodoList;

class TodoController extends Controller
{
    public function show($id)
    {
      $todo= Todo::find($id);
      $todo->todo_lists();
      return view('todos.show',compact('todo'));
    }

}

todos/show.blade.php

@extends('layouts.app')
@section('content')

    <div class="container show_box">
          <h2 class="show_content_title">❗️やること❗️</h2>
          <table>
        //未完了タスクの表示
            @foreach($todo->todo_lists as $todo_list)
             @if($todo_list->status==false)
            <tr>
              <td>{{$todo_list->list_name}}</td>
             //ステータスを更新
              <td>
                <form action="/todo/{{$todo->id}}/todo_lists/{{$todo_list->id}}" method="post">
                @csrf
                @method('PUT')
                <input type="hidden" name="status" value="{{$todo_list->status}}">
                  <div class=green_button><button type="submit"><strong>完了</strong></button></div>
                </form>
              </td>
            </tr>
            @endif
          @endforeach
          </table>


          <h2 class="devide_content_title">⭐️終わったこと⭐️</h2>
          <table>
            //完了タスクの表示
            @foreach($todo->todo_lists as $todo_list)
            @if($todo_list->status==true)
            <tr>
              <td>{{$todo_list->list_name}}</td>
              <td>{{$todo_list->updated_at->format('Y-m-d')}}</td>
            </tr>
            @endif
            @endforeach
          </table>
        </div><!--done_box-->
    </div>
    
@endsection

試したこと

親モデルのコントローラ(TodoController)のupdateメソッドに記載などしてみましたが
うまくいきませんでした。
原因が分からない為、ご教授頂ければと思います。

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

Laravel９

Answer 1

まずRoute::resource('todo.todo_lists',TodoListController::class)によってどういうルートが定義されてるか確認。
php artisan route:list
多分こんなルート。
todo/{todo}/todo_lists/{todo_list}
この{todo}や{todo_list}はパラメータでURLとして組み立てられた時はtodo/1/todo_lists/2みたいになる。
ここまではルーティングの話。

ルーティングとコントローラーを対応させる時はパラメータとメソッドの引数名を揃える。

public function update(Request $request, $todo, $todo_list)
{
}

質問の原因は引数名ではなく$todo, $todo_listの2つ必要なのに$id一つしかないから。
この$idはTodoのidなのにTodoListで使ってるから正しく動かない。

引数名の一致が必須なのは次の段階のルートモデル結合を使う時。↑の場合は厳密には引数名は関係ないけどLaravelならルートモデル結合を使うのが普通なので最初から引数名を揃えたほうがいい。

public function update(Request $request, Todo $todo, TodoList $todo_list)
{
    $todo_list->status = true;
    $todo_list->save();
}

Comments

[hanasaki]

回答頂いた方法で上手くいきました！
分かりやすく解説して下さってありがとうございました。