エラー Trying to get property 'id' of non-object を解決したい

Question

##現状・お願い
現在、DBデータの削除機能を作成しています。
削除ボタンを押すと削除されるといった機能なのですが、ボタンを押すとエラーが出てしまいます。

ただしブラウザから戻るボタンを押してみると、DBのデータは削除されています。
このエラーメッセージを出さないようにするにはどのようなコードが必要か教えていただけますと幸いです。

##エラーコード
Trying to get property 'id' of non-object
(View: /home/ec2-user/environment/invoice_and_todo/resources/views/todo/editItem.blade.php)

###TodoController（一部抜粋）

    // 案件情報の追加・更新・削除
    public function addNewItem(Request $request){
        $item = new Item();
        $item->user_id = Auth::user()->id;
        $item->client_id = $request->client_id;
        $item->item_name = $request->item_name;
        $item->delivery_date = $request->delivery_date;
        $item->unit_price = $request->unit_price;
        $item->states = $request->states;
        $item->memo = $request->memo;
        $item->save();
        
        return redirect('/addItem');
    }
    public function editItem($id){
        $list = Item::find($id);
        return view('todo.editItem', compact('list'));
    }
    public function updateItem(Request $request, $id){
        $item = Item::where('id', '=', $request->id)->first();
        $item->user_id = Auth::user()->id;
        $item->client_id = $request->client_id;
        $item->item_name = $request->item_name;
        $item->delivery_date = $request->delivery_date;
        $item->unit_price = $request->unit_price;
        $item->states = $request->states;
        $item->memo = $request->memo;
        $item->save();
        
        return redirect('/user');
    }
    public function deleteItem(Request $request)
    {
        $item = Item::where('id', '=', $request->id)->first();
        $item->delete();

        return redirect()->back();
    }
}

###web.php(一部抜粋)

// 案件情報の追加・更新・削除
Route::post('addNewItem','TodoController@addNewItem');
Route::get('/editItem/{id}', 'TodoController@editItem');
Route::post('/updateItem/{id}','TodoController@updateItem');
Route::delete('/deleteItem/{id}','TodoController@deleteItem');

###editItem.blade.php

@extends('layouts.layout')
@section('title','案件管理画面')

@section('body')
    <!--サイドメニューここから-->
    <div class="row">
      <div class="col s0 l2">
        <ul id="slide-out" class="sidenav sidenav-fixed ">
          <li><a href="{{url('/user')}}">クライアント一覧<i class="material-icons left">person</i></a></li>
          <li><a href="#">案件カレンダー<i class="material-icons left">date_range</i></a></li>
          <li><a href="{{url('/addClient')}}">クライアント追加<i class="material-icons left">add</i></a></li>
          <li><a href="{{url('addItem')}}">案件追加<i class="material-icons left">add</i></a></li>
        </ul>
      </div>
    <!--サイドメニューここまで-->
      <div class="col s12 offset-l1 l8">
      <nav class="header">
          <a href="#" data-target="slide-out" class="sidenav-trigger btn-floating"><i class="medium z-depth-1 material-icons">add</i></a>
      <h2 class="center-align">案件管理画面</h2>
      </nav>
      <form action="/updateItem/{{$list->id}}" method="POST">
        {{ csrf_field() }}
          <div class="input-field col s12">
            <input id="item_name" type="text" class="validate" name="item_name" value="{{$list->item_name}}">
            <label for="item_name">案件名</label>
          </div>
          <div class="input-field col s12">
            <input id="delivery_date" type="text" class="datepicker" name="delivery_date" value="{{$list->delivery_date}}">
            <label for="delivery_date">納期</label>
          </div>
          <div class="input-field col s12">
            <input id="unit_price" type="text" class="validate" name="unit_price" value="{{$list->unit_price}}">
            <label for="unit_price">単価</label>
          </div>
          <div class="input-field col s12">
            <input id="memo" type="text" class="validate" name="memo" value="{{$list->memo}}">
            <label for="memo">備考欄</label>
          </div>
          <input type="submit" value="案件編集" class="waves-effect waves-light btn blue accent-1">
          <input type="hidden" name="client_id" value="{{$list->client_id}}" >
          <input type="hidden" name="states" value="{{$list->states}}" >
      </form>
      <form method="POST" action="/deleteItem/{{$list->id}}" method="post">
          {{ method_field('delete') }}
          {{csrf_field()}}
          <input type="submit" value="案件情報削除" class="waves-effect waves-light btn red lighten-1">
      </form>
      </div>
    </div>
    
@endsection

Answer 1

エラー自体はそこまで珍しいものではないのでそのまま調べてもらえれば過去質問だったり解説の記事が見つかります。意味はそこで調べてもらうとして（エラーメッセージの読み方と対処, 検索や質問の原則）

redirect()->back()は直前のURLに戻るもの（HTTPレスポンス5.devより）
なので、削除処理が正常に行われた結果、そもそも$list自体が取得できなくなったことによって、
存在したときは返ってきたはずのModelが返ってこない＝そもそもModel内容がないためそれぞれのプロパティも参照できない

となっているように思います。

見たところ「単一データの編集・削除」ページのようですから、
削除を行った場合は明示的に一覧ページなどにリダイレクトされたほうが良いでしょう。
deleteItem()にとっての「直前のURL」は/editItem/{id}なので。

---

蛇足：

ブラウザから戻るボタンを押してみると

これは動的ページを作っているのであればあまり良い確認の仕方と言えません。
データベースに直接アクセスして確認してください。

Comments

[HirakuMorishima]

ありがとうございます。

[m.ts10806]

どのように理解されたか教えてください。
https://teratail.com/help/question-tips#questionTips4-2