エラー「syntax error, unexpected variable "$items"」の解消

Question

実現したいこと

Laravelでトップページを表示しようとすると下記のエラーが表示されますのでエラーを解消したいです。
よろしくお願いします。

MainController.phpのindexメソッド「 $items = DB::select('select * from todoList order by id');」で
エラー：syntax error, unexpected variable "$items"

前提

Laravelでトップページを表示しようとするとエラーが表示されます。
（その前に更新処理を作成していて、MainController.phpのchangeメソッドを作成するとエラーが発生しました。）

動作の流れを下記に記載します。
①URL：「http://127.0.0.1/」にアクセス
②web.php　Route::get('/', 'App\Http\Controllers\MainController@index');
③MainControllerのchangeメソッドの「 $items = DB::select('select * from todoList order by id');」で
エラー
→正解はindexメソッドがよばれなければいけないのに、なぜかchangeメソッドがよばれている？
※

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

syntax error, unexpected variable "$items"

該当のソースコード

web.php

<?php

use Illuminate\Support\Facades\Route;


//初期表示一覧画面
Route::get('/', 'App\Http\Controllers\MainController@index');

//新規登録入力画面
Route::get('/addInput', 'App\Http\Controllers\MainController@addInput');
//新規登録確認画面
Route::post('/addConfirm', 'App\Http\Controllers\MainController@addConfirm');
//新規登録実行
Route::post('/add', 'App\Http\Controllers\MainController@add');

//更新入力画面
Route::post('/changeInput', 'App\Http\Controllers\MainController@changeInput');
//更新確認画面
Route::post('/changeConfirm', 'App\Http\Controllers\MainController@changeConfirm');
//更新実行
Route::post('/change', 'App\Http\Controllers\MainController@change');

MainController.php

<?php

namespace App\Http\Controllers;

use Illuminate\Support\Facades\DB;
use Illuminate\Http\Request;

class MainController extends Controller
{
    public function index() {
        $items = DB::select('select * from todoList order by id');

        return view('todo', ['items' => $items]);
    }

    public function addInput() {
        return view('addInput');
    }

    public function addConfirm(Request $request) {
        $item = [
            'id' => $request->id,
            'task'  => $request->task,
            'priority'  => $request->priority,
            'deadline'  => $request->deadline,
            'manager'  => $request->manager,
            'remarks'  => $request->remarks
        ];
        return view('addConfirm', ['item' => $item]);
    }

    public function add(Request $request) {
        $param = [
            'task'  => $request->task,
            'priority'  => $request->priority,
            'deadline'  => $request->deadline,
            'manager'  => $request->manager,
            'remarks'  => $request->remarks
        ];
        // dd($param);
        DB::insert('insert into todolist (task, priority, deadline, manager, remarks) 
        values (:task, :priority, :deadline, :manager, :remarks)', $param);
        
        $items = DB::select('select * from todoList order by id');
        return view('todo', ['items' => $items]);
    }

    public function changeInput(Request $request) {
        // dd($request);
        $id = $request->id;
        $param = ['id' => $id];
        // dd($param);
        $items = DB::select('select * from todoList where id = :id', $param);
        $item = $items[0];

        // dd($item);
        return view('changeInput', ['item' => $item]);
    }

    public function changeConfirm(Request $request) {
        $item = [
            'id' => $request->id,
            'task'  => $request->task,
            'priority'  => $request->priority,
            'deadline'  => $request->deadline,
            'manager'  => $request->manager,
            'remarks'  => $request->remarks
        ];
        // dd($item);
        return view('changeConfirm', ['item' => $item]);
    }

    public function change(Request $request) {
        $param = [
            'task'  => $request->task,
            'priority'  => $request->priority,
            'deadline'  => $request->deadline,
            'manager'  => $request->manager,
            'remarks'  => $request->remarks,
            'id' => $request->id
        ];
        dd($param);
        DB::update('update todoList set task = :task, priority = :priority
        , deadline = :deadline, manager = :manager, remarks = :remarks where id = :id', $param)
        
        $items = DB::select('select * from todoList order by id');
        return view('todo', ['items' => $items]);
    }
}

todo.blade.php

<!DOCTYPE html>
<html>
  <head>
    <meta charset="UTF-8">
    <link rel="stylesheet" href="main.css">
    <title>初期表示一覧画面</title>
    <script>
      //勤務表編集選択画面でラジオボタン選択中は変更削除ボタンを有効にする
      //ラジオボタンを選択していないときは無効にする
      function check(){
        var flag = true; // 選択されているか否かを判定するフラグ
      
      //ラジオボタンの数だけ判定を繰り返す（ボタンを表すインプットタグがあるので１引く）
        for(var i=0; i < document.form.ids.length-1;i++){
            // i番目のラジオボタンがチェックされているかを判定
            if(document.form.ids[i].checked){ 
                flag = true;
                document.getElementById("henko").disabled = false;
                document.getElementById("sakujyo").disabled = false;
            } 
  }
}
</script>
  </head>
  <body>
    <h1>todoリスト</h1>
    <form name="form">
      <table border="1">
      @csrf
        <tr>
          <th>選択</th>
          <th>No.</th>
          <th>タスク</th>
          <th>優先度</th>
          <th>期日</th>
          <th>担当者</th>
          <th>備考</th>
        </tr>
        @foreach ($items as $item)
        <tr>
          <td><input type="radio" name="id" id="ids" value="{{$item->id}}" onClick="return check()"></td>
          <td><input type="text" value="{{$item->id}}" readonly></td>
          <td><input type="text" value="{{$item->task}}" readonly></td>
          <td><input type="text" value="{{$item->priority}}" readonly></td>
          <td><input type="text" value="{{$item->deadline}}" readonly></td>
          <td><input type="text" value="{{$item->manager}}" readonly></td>
          <td><input type="text" value="{{$item->remarks}}" readonly></td>
        </tr>
        @endforeach
      </table>
    <button type="submit" formaction="addInput" formmethod="get">新規</button><button type="submit" formaction="changeInput" formmethod="post" name="henko" id="henko" disabled>更新</button><button type="submit" formaction="deleteConfirm" formmethod="post" name="sakujyo" id="sakujyo" disabled>削除</button>
    </form>
  </body>
</html>

試したこと

MainController.phpのchangeメソッドをすべて削除するとトップページが表示されます。

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

ここにより詳細な情報を記載してください。

Answer 1

        DB::update('update todoList set task = :task, priority = :priority
        , deadline = :deadline, manager = :manager, remarks = :remarks where id = :id', $param)
        
        $items = DB::select('select * from todoList order by id');

直前の行のセミコロンが抜けているため、次の$itemsが文法的に矛盾してしまっています。

Comments

[hfuob]

ありがとうございます。解決しました。