質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.52%

削除や更新ができない

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 1,398

thunder_zak

score 18

現在、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)
    {
        //
    }
}


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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

0

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

 route.php

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

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

 MoviesController.php

use App\Movie; // useを追記

class MoviesController extends Controller
{
    public function edit(Request $request, $id) // $idにrouteで設定した{movies}の値が入ります。変数名を変えてもOKです
    {
        return view('movies_edit', [
            'movie' => Movie::find($id),
        ]);
    }
    public function destroy($id) {
        Movie::find($id)->delete();
        return redirect('/');
    }
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/05/05 18:23

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

    キャンセル

0

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

//Route::post('/movies_edit/{movies}', function(Movie $movies) {
Route::post('/movies_edit/{movies}', function($movies) {
//
}

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.52%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる