Q&A
非同期通信をしたいと言うわけではないですよね？
#やりたいこと
下記イメージのように作業中ボタンを個別で完了ボタンに変更したい記述方法が分からないのでどなたかご教示下さい。
#やったこと
1.データベースに新しくdoneカラムを追加
2.マイグレーションファイルでboolean型のdoneカラムにデフォルト値（false)を設定
3.ボタン押下でステータスを０→１もしくは１→０にする ※←ここから分かりません
- done カラムの値を取り出す
- 0 → 1 or 1 → 0 に変換する
- 変換した値を done に設定する
- テーブルを更新する
これをやれば良いと思うのですが。。。
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('/Todo', 'TodoController@index'); Route::post('/Todo', 'TodoController@create'); Route::group(['middleware' => ['web']], function () { Route::post('/Todo', 'TodoController@create'); }); Route::post('/Todo/{table}', 'TodoController@delete'); Route::post('/done/{id}/', 'TodoController@done');
TodoController.php
<?php namespace App\Http\Controllers; use Illuminate\Http\Request; use App\Http\Requests\CreateTaskRequest; use Illuminate\Support\Facades\DB; use App\Todo; class TodoController extends Controller { public function index(Request $request) { $items = Todo::all(); return view('Todos.index', ['items' => $items]); } public function create(CreateTaskRequest $request) { $todo = new Todo; $form = $request->all(); unset($form['_token']); $todo->timestamps = false; $todo->fill($form)->save(); return redirect('/Todo'); } public function delete(Request $request) { Todo::find($request->table)->delete(); $items = Todo::all(); $items = DB::table('Todos')->orderBy('todo_id', 'asc')->get(); return redirect('/Todo'); } public function done($id) { $items = Todo::all(); $done = Todo::find($id); $done->save(); return redirect('/Todo'); } }
views/Todos/index.blade.php
@extends('layouts.index') @section('title','Todoリスト') @section('menubar') @parent Todoリスト3 @endsection @section('content') <table> <tr> <th>ID</th> <th>comment</th> <th>状態</th> </tr> @foreach ($items as $item) <tr> <td>{{$loop->iteration}}</td> <td>{{$item->comment}}</td> <td> <form action="{{ url('done/' . $item->id) }}" method="POST"> @csrf <button type="submit"> 作業中 </button> </form> </td> <td> <form action="{{ url('Todo/' . $item->id) }}" method="POST"> @csrf <button type="submit"> 削除 </button> </form> </td> </tr> @endforeach </table> @endsection
views/Todos/done.blade.php
@extends('layouts.index') @section('title','Todoリスト') @section('menubar') @parent Todoリスト @endsection @section('content') <table> <tr> <th>ID</th> <th>comment</th> <th>状態</th> </tr> @foreach ($items as $item) <tr> <td>{{$loop->iteration}}</td> <td>{{$item->comment}}</td> <td> <form action="/Todo" method="GET"> @csrf <button type="submit"> {{$item->done}} </button> </form> </td> <td> <form action="{{ url('Todo/' . $item->id) }}" method="POST"> @csrf <button type="submit"> 削除 </button> </form> </td> </tr> @endforeach </table> @endsection
create_todo_table.php
<?php use Illuminate\Database\Migrations\Migration; use Illuminate\Database\Schema\Blueprint; use Illuminate\Support\Facades\Schema; class CreateTodoTable extends Migration { /** * Run the migrations. * * @return void */ public function up() { Schema::create('Todos', function (Blueprint $table) { $table->bigIncrements('id'); $table->char('comment'); $table->integer('todo_id'); $table->boolean('done')->default(false); $table->timestamps(); }); } /** * Reverse the migrations. * * @return void */ public function down() { Schema::dropIfExists('Todos'); } }
todo_id はなんのために存在しているのですか？
$itemsを上書きしていますが、どちらを使っていきたいのでしょうか？
非同期通信をしたいというわけではありません。
todo _idは見た目上のidとして使用してます。
外部キーっていうんですか？
コントローラーのdone()の$itemsは特に意味がないかもしれません
それ以外の$itemsはコメントとtodo_idを更新してviewに渡してます。
今回の質問に関しては todo_id の存在は無関係ということですね。
はい。そうなります。
回答1件
0
ベストアンサー
php
1 public function done($id) 2 { 3 $items = Todo::all(); 4 $done = Todo::find($id); 5 $done->save(); 6 return redirect('/Todo'); 7 }
- なぜ使いもしない
$items = Todo::all();があるのか不明 $done = Todo::find($id);変数名おかしい。Todo::find($id) が返すのは Todo オブジェクトなんだから、$todo の方が妥当
↓
php
1 2 public function done($id) 3 { 4 $todo = Todo::find($id); 5 // $todo->done = !$todo->done; // 自分ならこう書くけど 6 $todo->done = ($todo->done) ? 0 : 1; // こっちの方がわかりやすいかも 7 $todo->save(); 8 return redirect('/Todo'); 9 }
投稿2020/01/22 13:11
退会済みユーザー
総合スコア0
返信ありがとうございます。
こちらだと何かが足りていないようです。
タスクの作業中ボタンを押下時にリダイレクトされていますが完了ボタンに切り替わりません。
view側の処理を足してあげると解答して頂いた記述でいけました。
ありがとうございます！
あなたの回答
tips
プレビュー
