現在、Laravel5.1にて本を参考にウェブアプリを作成しています。
サンプルコードの通りに、一部変数名などだけ変更していますが、削除ボタンや、更新ボタンで処理が実行されずに困っております。
ボタン押下時にIDを渡しているのですが、そのIDが渡されていないような気がするのですが、どこをどう修正していいものかわかりません。

<?php

/*
|--------------------------------------------------------------------------
| アプリケーションのルート
|--------------------------------------------------------------------------
|
| ここでアプリケーションのルートを全て登録することが可能です。
| 簡単です。ただ、Laravelへ対応するURIと、そのURIがリクエスト
| されたときに呼び出されるコントローラを指定してください。
|
*/

use App\Movie;
use Illuminate\Http\Request;

/*
*映画ダッシュボードを表示
*TOPページ
*/
Route::get('/', function () {
    $movies = Movie::orderBy('created_at', 'asc')->get();
    return view('movies', [
        'movies' => $movies
    ]);
});

/*
*新しい映画を追加
*/
Route::post('/movies', function(Request $request){
 
    //バリデーション
    //バリデーターの取得
    $validator = Validator::make($request->all(), [
        //3文字以上255文字未満
        'item_title' => 'required|min:3|max:255',
        'item_number' => 'required | min:1 | max:3',
        'item_amount' => 'required | max:6',
        'published'   => 'required'
    ]);
    //バリデーション： エラー 
    if ($validator->fails()) {
        return redirect('/')
            ->withInput()
            ->withErrors($validator); 
    }


    //Eloquentモデルー映画の登録処理
    $movies = new Movie;
    $movies->item_title  = $request->item_title;
    $movies->item_number = $request->item_number;
    $movies->item_amount = $request->item_amount;
    $movies->published   = $request->published;
    $movies->save();
    return redirect('/');
});

//更新画面
Route::post('/movies_edit/{movies}', function(Movie $movies) {
    //{movies}id 値を取得 => Movie $movies id 値の1レコード取得
    return view('movies_edit', ['movie' => $movies]);
});

//更新処理
Route::post('/movies/update', 'MoviesController@update');


/*
*映画を削除する
*/

Route::delete('/movie/{movie}', function (Movie $movie) {
    $movie->delete();
    return redirect('/');
});

?>

<!-- resources/views/movies.blade.php -->

@extends('layouts.app')
@section('content')

    <!-- Bootstrapの定型コード -->
    <div class="panel-body">
        <!-- バリデーションエラーの表示に使用 -->
        @include('errors.errors')

        <!-- 映画登録フォーム -->
        <form action="{{ url('movies') }}" method="POST" class="form-horizontal">
            
            {{ csrf_field() }}

            <!-- 入力フォーム -->
            <div class="form-group">

                <!-- 映画タイトル -->
                <div class="col-sm-6">
                    <label for="movie" class="col-sm-3 control-label">映画タイトル</label>
                    <input type="text" name="item_title" id="movie-title" class="form-control">
                </div>

                <!-- 金額 -->
                <div class="col-sm-6">
                    <label for="amount" class="col-sm-3 control-label">金額</label>
                    <input type="text" name="item_amount" id="movie-amount" class="form-control">
                </div>

                <!-- 映画番号 -->
                <div class="col-sm-6">
                    <label for="number" class="col-sm-3 control-label">映画番号</label>
                    <input type="text" name="item_number" id="movie-number" class="form-control">
                </div>

                <!-- 発表日 -->
                  <div class="col-sm-6">
                    <label for="published" class="col-sm-3 control-label">発表日</label>
                    <input type="date" name="published" id="movie-published" class="form-control">
                </div>  

            </div>

            <!-- 映画登録ボタン -->
            <div class="form-group">
                <div class="col-sm-offset-3 col-sm-6">
                    <button type="submit" class="btn btn-default">
                        <i class="glyphicon glyphicon-plus" aria-hidden="true"></i> 登録
                    </button>
                </div>
            </div>
        </form>
    </div>

    <!-- 現在の登録されている映画を表示 -->
    @if (count($movies) > 0)
        <div class="panel panel-default">
            <div class="panel-heading">
                現在の登録映画
            </div>
            <div class="panel-body">
                <table class="table table-striped task-table">
                    <!-- テーブルヘッダー -->
                    <thead>
                        <th>映画一覧</th>
                        <th>&nbsp;</th>
                    </thead>
                    <!-- テーブル本体 -->
                    <tbody>
                        @foreach ($movies as $movie)
                            <tr>
                                <!-- 映画タイトル -->
                                <td class="table-text">
                                    <div>{{ $movie->item_title }}</div>
                                </td>
                                <!-- 映画金額 -->
                                <td class="table-text">
                                    <div>{{ $movie->item_amount }}</div>
                                </td>
                                <!-- 映画番号 -->
                                <td class="table-text">
                                    <div>{{ $movie->item_number }}</div>
                                </td>
                                <!-- 発表日 -->
                                <td class="table-text">
                                    <div>{{ $movie->published }}</div>
                                </td>

                                <!-- 映画: 更新ボタン -->
                                <td>
                                    <form action="{{ url('movies_edit/'.$movie->id) }}" method="POST">
                                        {{ csrf_field() }}
                                        <button type="submit" class="btn btn-primary">
                                            <i class="glyphicon glyphicon-trash"></i> 更新
                                        </button>
                                    </form>
                                </td>

                                <!-- 映画:削除ボタン -->
                                <td>
                                    <form action="{{ url('movie/'.$movie->id) }}" method="POST">
                                            {{ csrf_field() }}
                                            {{ method_field('DELETE') }}

                                        <button type="submit" class="btn btn-danger">
                                            <i class="glyphicon glyphicon-trash"></i>削除
                                        </button>
                                    </form>
                                </td>
                            </tr>
                        @endforeach
                    </tbody>
                </table>
            </div>
        </div>
    @endif
 </div>
@endsection

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;

use App\Http\Requests;
use App\Http\Controllers\Controller;
use App\Book;
use Validator;

class MoviesController extends Controller
{
    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index()
    {
        //
    }

    /**
     * Show the form for creating a new resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function create()
    {
        //
    }

    /**
     * Store a newly created resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return \Illuminate\Http\Response
     */
    public function store(Request $request)
    {
        //
    }

    /**
     * Display the specified resource.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function show($id)
    {
        //
    }

    /**
     * Show the form for editing the specified resource.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function edit($id)
    {
        //
    }

    /**
     * Update the specified resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    //public function update(Request $request, $id)
    public function update(Request $request)
    {
        //バリデーション
        $validator = Validator::make($request->all(), [
            'id' => 'required',
            'item_title' => 'required|min:3|max:255',
            'item_number' => 'required|min:1|max:3',
            'item_amount' => 'required|max:6',
            'published' => 'required',
        ]);
        //バリデーション:エラー
            if ($validator->fails()) {
                return redirect('/')
                    ->withInput()
                    ->withErrors($validator);
        }
        //データ更新
        $movies = Movie::find($request->id);
        $movies->item_title   = $request->item_title;
        $movies->item_number = $request->item_number;
        $movies->item_amount = $request->item_amount;
        $movies->published   = $request->published;
        $movies->save();
    }

    /**
     * Remove the specified resource from storage.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function destroy($id)
    {
        //
    }
}

以上です、よろしくお願いいたします。

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

回答2件

ベストアンサー

答えはcontrollerに書かれています。
あとrouteに直接処理を書くのはあまりイケてないので、きちんとcontroller側に書くクセを付けたほうがよいでしょう。

route.php

url上の{movie}に当たる部分の値が呼び出すメソッドの引数に入ります。
また、{movie}の値を1文字以上の数値に制限します

php
1Route::post('/movies_edit/{movies}', 'MoviesController@edit')->where('movie', '\d+');
2Route::delete('/movie/{movie}', 'MoviesController@destroy')->where('movie', '\d+');

MoviesController.php

php
1use App\Movie; // useを追記
2
3class MoviesController extends Controller
4{
5	public function edit(Request $request, $id) // $idにrouteで設定した{movies}の値が入ります。変数名を変えてもOKです
6	{
7		return view('movies_edit', [
8			'movie' => Movie::find($id),
9		]);
10	}
11	public function destroy($id) {
12		Movie::find($id)->delete();
13		return redirect('/');
14	}
15}

投稿2017/05/05 03:24

編集2017/05/05 03:29

shiroyuki

総合スコア169

thunder_zak

2017/05/05 09:23

ありがとうございます。shiroyukiさんの方法で解決致しました。その後、すべてControllerに記述しました。

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

試してませんが、このあたりがおかしいのでは？

php
1//Route::post('/movies_edit/{movies}', function(Movie $movies) {
2Route::post('/movies_edit/{movies}', function($movies) {
3//
4}
5
6//Route::delete('/movie/{movie}', function (Movie $movie) {
7Route::delete('/movie/{movie}', function ($movie) {
8    //$movie->delete();
9    Movie::destroy($movie);
10    return redirect('/');
11});
12