laravelでデータベースに渡した値を表示させたい。

#環境
mac osx catalina

Docker version 19.03

laravel 6

mysql 8.0
#実現したいこと
タスク追加フォームで入力した値を表示させたい

#問題点
データベースにコメントが入らずにIDのみ登録される。

#やったこと
データベースクライアントで確認した。←コメントのみ空の状態
データベースクライアントでコメントを追加した←追加成功

以上のことからデータベースへの受け渡し方法に問題があると仮定した←今ここ

色々調べましたがエラーも出ていないことから私自身の理解度に問題があるのかもしれません。
もし良ければ下記コードをご確認頂きご教示頂けると嬉しいです。

Todo.controller.php

<?php

namespace App\Http\Controllers;

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

class TodoController extends Controller
{
    public function index(Request $request)
    {
        $items = DB::select('select * from Todos');
        return view('Todos.index', ['items' => $items]);
    }


    public function post(Request $request)
    {
        $items = DB::select('select * from Todos');
        return view('Todos.index', ['items' => $items]);
    }


    public function create(Request $request)
    {
        $param = [
            'id' => $request->id,
            'comment' => $request->comment,
        ];
        DB::insert('insert into Todos (id, comment) values
                (id, comment)', $param);
        return redirect('/Todo');
    }
}

view/layouts/index.blade.php

<html>

<head>
  <title>@yield('title')</title>
  <style>
    body {
      font-size: 16pt;
      color: #999;
      margin: 5px;
    }

    h1 {
      font-size: 50pt;
      text-align: right;
      color: #f6f6f6;
      margin: -20px 0px -30px 0px;
      letter-spacing: -4pt;
    }

    ul {
      font-size: 12pt;
    }

    hr {
      margin: 25px 100px;
      border-top: 1px dashed #ddd;
    }

    .menutitle {
      font-size: 25pt;
      font-weight: bold;
      margin: 0px;
    }

    .content {
      margin: 10px;
    }

    .footer {
      font-size: 20pt;
      margin: 10px;
    }

    th {
      background-color: #999;
      color: fff;
      padding: 5px 10px;
    }

    td {
      border: solid 1px #aaa;
      color: #999;
      padding: 5px 10px;
    }

    task {
      font-size: 50pt;
      width: 70px;
      height: 70px;
    }
  </style>
</head>

<body>
  <h1>@yield('title')</h1>
  @section('menubar')
  <h2 class="menutitle"> Todoリスト</h2>
  タスクを追加
  <br>
  <form action="/Todo" method="post">
    {{ csrf_field() }}

    <input type="text" name="task" size="70">

    <input type="submit" value="追加">
  </form>
  <hr size="1">
  <div class="content">
    @yield('content')
  </div>
  <div class="footer">
    @yield('footer')
  </div>

</body>

</html>

view/Todos/index.blade.php

@extends('layouts.index')
@section('title','Todoリスト')
@section('menubar')
@parent
Todoリスト
@endsection

@section('content')

<table>
  <tr>
    <th>ID</th>
    <th>コメント</th>
    <th>状態</th>
  </tr>
  @foreach ($items as $item)
  <tr>
    <td>{{$item->id}}</td>
    <td>{{$item->comment}}</td>
    <td>作業中</td>
    <td><button type="button" name="delete" value="value">削除</button></td>
  </tr>
  @endforeach
</table>
@endsection

web.php

<?php

/*
|--------------------------------------------------------------------------
| Web Routes
|--------------------------------------------------------------------------
|
| Here is where you can register web routes for your application. These
| routes are loaded by the RouteServiceProvider within a group which
| contains the "web" middleware group. Now create something great!
|
*/

Route::get('/', function () {
    return view('welcome');
});

Route::get('/hello', 'LineBotController@index');
Route::get('/Todo', 'TodoController@index');
Route::post('/Todo', 'TodoController@create');

退会済みユーザー

2020/01/02 06:34

Todo.php を提示してください。

twinparadox

2020/01/02 06:51

そのようなファイルは作成した記憶がありません。。。

退会済みユーザー

2020/01/02 06:54 編集

app/Todo.php は存在していない？もしかして、 https://www.amazon.co.jp/PHP%E3%83%95%E3%83%AC%E3%83%BC%E3%83%A0%E3%83%AF%E3%83%BC%E3%82%AF-Laravel%E5%85%A5%E9%96%80-%E6%8E%8C%E7%94%B0%E6%B4%A5%E8%80%B6%E4%B9%83/dp/4798052582/ref=tmm_hrd_swatch_0?_encoding=UTF8&qid=&sr= この本で学習していますか？

twinparadox

2020/01/02 06:58

はい。おっしゃる通りです。

退会済みユーザー

2020/01/02 06:59

どちらの質問にたいする回答ですか？両方？

twinparadox

2020/01/02 07:04

言葉足らずですみません。両方です。存在しているのはapp/user.phpのみになります。

退会済みユーザー

2020/01/02 07:20

件の書籍に関しては、なんというか、非常に変な作法で書かれていて、普通のLaravelの使い方で説明されていません。質問する際には、その点を明記される方が回答者に混乱を招くことがないと思います。話は変わって、ではどう回答すべきか、迷うところではあります。その変な使い方を受け入れた上で回答すべきか、通常のLaravelの使い方で回答すべきかという問題です。変な使い方は普通しないわけなので、ここの回答者もかなり戸惑うでしょうね。

twinparadox

2020/01/02 07:48

もしよろしければ、通常のlaravelの使い方での回答を頂きたいです。

m.ts10806

2020/01/02 09:04

Laravel6対応の書籍でないなら余計に参照する意味はないですね。というか、ある一定のPHPスキルを持っていれば日本語ドキュメントのほうで事足ります。 https://readouble.com/laravel/

mikkame

2020/01/02 10:10

Laravel6とLaravel5系はセマンティックバージョンの対応によって調整されただけでメジャーバージョンほど離れている訳ではありません

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

回答2件

Laravelのコマンドを上手に使うことで、Laravel的な命名規則や作法がわかりやすいので活用してください。
基本的に、そのコマンドで作成された関数名やクラス名が標準的な使い方だと思っていいです。

Laravel をセットアップした何よりもまず行うのが、Model の作成です。

php
1php artisan make:model Todo -a

-a オプションをつけることで、以下のファイルが一度に作成されます。

database/migrations/YYYY_MM_DD_HHMMSS_create_todos_table.php
database/factories/TodoFactory.php
app/controllers/TodoController.php
app/Todo.php

次に YYYY_MM_DD_HHMMSS_create_todos_table.php を編集して、適宜カラムを追加する。
routes/web.php にルーティングを記述する。

標準的なCRUDであれば、以下のように記述。

php
1Route::resource('todos', 'TodoController');

この記述で、標準的なルーティングが全て設定される。

Method	URI	Name	Action	Middleware
GET,HEAD	todos	todos.index	App\Http\Controllers\TodoController@index	web
POST	todos	todos.store	App\Http\Controllers\TodoController@store	web
GET,HEAD	todos/create	todos.create	App\Http\Controllers\TodoController@create	web
GET,HEAD	todos/{todo}	todos.show	App\Http\Controllers\TodoController@show	web
PUT,PATCH	todos/{todo}	todos.update	App\Http\Controllers\TodoController@update	web
DELETE	todos/{todo}	todos.destroy	App\Http\Controllers\TodoController@destroy	web
GET,HEAD	todos/{todo}/edit	todos.edit	App\Http\Controllers\TodoController@edit	web

Model に $fillable プロパティを設定

php
1<?php
2
3namespace App\Models;
4
5use Illuminate\Database\Eloquent\Model;
6use Illuminate\Database\Eloquent\SoftDeletes;
7
8class Todo extends Model
9{
10    use SoftDeletes;
11
12    protected $fillable = [
13        'comment'
14    ];
15}

Controller に処理を記述

php
1<?php
2
3namespace App\Http\Controllers;
4
5use App\Http\Requests\TodoFormRequest;
6use App\Models\Todo;
7use Illuminate\Http\RedirectResponse;
8use Illuminate\View\View;
9
10class TodoController extends Controller
11{
12    /**
13     * Display a listing of the resource.
14     *
15     * @return View
16     */
17    public function index()
18    {
19        $todos = Todo::query()
20            ->paginate();
21        return view('todo.index', compact('todos'));
22    }
23
24    /**
25     * Show the form for creating a new resource.
26     *
27     * @return View
28     */
29    public function create()
30    {
31        return view('todos.create');
32    }
33
34    /**
35     * Store a newly created resource in storage.
36     *
37     * @param TodoFormRequest $request
38     * @return RedirectResponse
39     */
40    public function store(TodoFormRequest $request)
41    {
42        $todo = new Todo();
43        $todo->fill(
44            $request->all()
45        )->save();
46        return redirect()->route('todos.show', compact('todo'));
47    }
48
49    /**
50     * Display the specified resource.
51     *
52     * @param Todo $todo
53     * @return View
54     */
55    public function show(Todo $todo)
56    {
57        return view('todos.show', compact('todo'));
58    }
59
60    /**
61     * Show the form for editing the specified resource.
62     *
63     * @param Todo $todo
64     * @return View
65     */
66    public function edit(Todo $todo)
67    {
68        return \view('todos.edit', compact('todo'));
69    }
70
71    /**
72     * Update the specified resource in storage.
73     *
74     * @param TodoFormRequest $request
75     * @param Todo $todo
76     * @return RedirectResponse
77     */
78    public function update(TodoFormRequest $request, Todo $todo)
79    {
80        $todo->fill(
81            $request->all()
82        )->save();
83        return redirect()->route('todos.show', compact('todo'));
84    }
85
86    /**
87     * Remove the specified resource from storage.
88     *
89     * @param Todo $todo
90     * @return RedirectResponse
91     * @throws \Exception
92     */
93    public function destroy(Todo $todo)
94    {
95        $todo->delete();
96        return redirect()->route('todos.index');
97    }
98}

この例では、バリデーションを、TodoFormRequest.php に記述するよう設定したので、

php
1php artisan make:request TodoFormRequest

で、app/Http/Requests/TodoFormRequest.php が作成される。

php
1<?php
2
3namespace App\Http\Requests;
4
5use Illuminate\Foundation\Http\FormRequest;
6
7class TodoFormRequest extends FormRequest
8{
9    /**
10     * Determine if the user is authorized to make this request.
11     *
12     * @return bool
13     */
14    public function authorize()
15    {
16        return true;
17    }
18
19    /**
20     * Get the validation rules that apply to the request.
21     *
22     * @return array
23     */
24    public function rules()
25    {
26        return [
27            'comment' => ['required', 'string', 'max:255']
28        ];
29    }
30}

すべてを解説するわけにもいかないので、この辺りで。

投稿2020/01/02 09:51

退会済みユーザー

総合スコア0

twinparadox

2020/01/02 12:28

丁寧にありがとうございます！すごいです。こんな少しの記述で色んな処理が行えるなんて。。。どれも貴重な情報をありがとうございます！

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

ベストアンサー

<input type="text" name="task" size="70">

inputのnameがtask

投稿2020/01/02 07:34

mikkame

総合スコア5036

twinparadox

2020/01/02 07:49

回答ありがとうございます！考えてみます。

twinparadox

2020/01/02 08:14

動きました。ありがとうございます！変更点① <input type="text" name="comment" size="70"> 変更点② public function create(Request $request) { $param = [ $request->id, $request->comment, ]; DB::insert('insert into Todos (id, comment) values (?, ?)', $param); return redirect('/Todo'); } htmlの基礎、プレースホルダーの使い方、デバッグについて理解していませんでした。