laravel MySQLからデータを取得できない

前提・実現したいこと

laravel MySQLを使った入力フォームの作成をしています
データの登録フォームと登録したデータのリストまでは完成済みで、現在はそのリストのデータを編集するフォームを作る段階です
dbテーブル構造はオートインクリメントのidと他名前などの情報となっていて、idでどのデータを編集するのかを特定しselect句where id = $idでそのデータを取得したいです

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

以下のコードなのですがデータが表示されません
select句が違うのか、bladeの文がおかしいのかわかりません....

エラーメッセージ

該当のソースコード

blade
1<HTML>
2    <BODY>
3        <TABLE>
4            <tr>
5                <th> 氏名 </th><th> 電話番号 </th><th> メールアドレス </th><th> </th><th> </th>
6            </tr>
7
8            @foreach($items as $item)
9                <tr>
10                    <td>{{$item->userName}}</td>
11                    <td></td>
12                    <td>{{$item->email}}</td>
13
14                    <form action="edit" method="post">
15                        {{ csrf_field() }}
16
17                        <td><input type="hidden" name="id" value="{{$item->userId}}"></td>
18
19                        <td> <input type=submit value="編集"> </td>
20                        <td> <input type="submit" value="削除"> </td>
21                        
22                    </form>
23
24                </tr>
25            @endforeach
26
27        </TABLE>
28        <a href="regist"> <input type="button" value="新規"> </a>
29    </BODY>
30</HTML>

controller
1public function edit(Request $request)
2    {
3        $id = $request->id;
4        $items = DB::select('select * from address where userId= ".$id." ');
5
6        return view('edit',$items);
7
8    }
9

blade
1<HTML>
2    <BODY>
3        <div>
4            @isset($items)
5            <form action="regist" method="post">
6                {{ csrf_field() }}
7                <p>
8                    <label>
9                        氏名: <input type="text" name="name" value="{{ $items->userName }}" required>
10                    </label>
11                </p>
12                <p>
13                    <label>
14                        電話番号: <input type="tel" name="tel" value="{{ $items->tel }}">
15                    </label>
16                </p>
17                <p>
18                    <label>
19                        メールアドレス: <input type="email" name="email" value="{{ $items->email }}">
20                    </label>
21                </p>
22                <p>
23                <input type="submit"　value="更新"> <input type="button" value="戻る">
24                </p>
25            </form>
26            @endisset
27        </div>
28        <div>
29            <a href="list"> <input type="button" value="アドレス一覧"> </a>
30        </div>
31    </BODY>
32</HTML>

試したこと

ここに問題に対して試したことを記載してください。

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

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

行動規範の内容に同意します

回答2件

ベストアンサー

diff
1-        $items = DB::select('select * from address where userId= ".$id." ');
2+        $items = DB::select('select * from address where userId= '.$id);

こりあえず動かすだけなら、ここ修正すればいいけど、SQLインジェクションの脆弱性あるから、NG。
いったい何を参考にコード書いているのでしょうか？

Laravel の作法にも従っていないし、レビューしたら、ほぼ全て書き換えが必要です。

投稿2021/03/04 04:37

phper.k

総合スコア3923

nanako_azami

2021/03/04 11:18

無事この記述でいけました。ありがとうございます。そもそもformで送る必要ないと気づきました

行動規範の内容に同意します

いえ、PHPの文字列への理解です。

シングルクォーテーション内では変数展開されませんから、
'select * from address where userId= ".$id." '
だと変数$idにどんな値が入っていても、実行されるSQLはselect * from address where userId= ".$id." です。
ログ確認してみてください。

'select * from address where userId= '.$id.' 'のようにすれば変数展開はされますが、その前にこのSQLで本当に想定の結果が得られるかはDBに直接実行して確認してみてください。
これに限らず、PHPから実行する前に直実行。

そうすれば「SQLが悪いか何が悪いか」迷わずに済みます。
単体で実行できるものは個別確認しておくことで問題切り分けが明確になります。
これに限らず。

ただ。このままだとSQLインジェクションが発生しますし、Laravelの組み方としてどうか？という疑問はあります。
ドキュメント確認して、本当にselect()メソッド使わなきゃいけない程複雑なSQLなのかどうかは一考してください。

投稿2021/03/04 04:41

編集2021/03/04 04:43