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

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

ただいまの
回答率

88.81%

Laravelで画像とテキストをフォームから更新できない。

受付中

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 201

toyop

score 18

こんにちは。Laravel初心者の大学生です。

現在Laravelで投稿の編集フォームを作っています。
編集フォームのテキストエリアにはDBに登録されているテキストが表示されるようにし、画像は新しい画像が選択された場合のみ更新できるようにしたいです。
自分でコードを組んでみたのですが、画像を選択していないと

SQLSTATE[HY000]: General error: 1364 Field 'image' doesn't have a default value (SQL: insert into tests (user_idnameupdated_atcreated_at) values (1, あああ, 2020-07-19 06:59:31, 2020-07-19 06:59:31))

というエラーが表示され、また、画像を選択した場合は編集されるのではなく、DBに新しくデータが追加されてしまいます。

以下がコードになるのですが、どこで間違えてしまっているのかご教示願いたいです。

web.php

Route::get('/tests/edit/{id}', 'TestController@edit'); //編集
    Route::patch('/tests', 'TestController@update'); //更新

TestController.php

use Illuminate\Support\Facades\Storage;

public function edit(Request $request)
    { 
        $test = Test::findOrFail($request->id);
        return view('test.edit', ['test' => $test]);
    }

public function update(Request $request, Test $test)
    {
        $test->user_id = $request->user()->id;
        $test->name = $request->name;

        if($request->hasFile('image')) {
            Storage::delete('public/image/' . $test->image); //元の画像を削除
            $path = $request->file('image')->store('public/image'); //画像のパス
            $test->image = basename($path);
            $test->save();
        }

        $test->save();

        return redirect('/tests')->with('flash_message', '編集が完了しました');
    }

edit.blade.php

@extends('layouts.app')

@section('content')
<div>
    <form action="/tests" method="post" enctype="multipart/form-data">
        {{ csrf_field() }}
        {{ method_field('patch') }}
            料理名:<input type="text" name="name" value='{{ $test->name }}'><br>
            画像: <input type="file" name="image"><br>
            <input type='submit' value='更新'>
    </form>
</div>
@endsection

よろしくお願いいたします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • SanQ

    2020/07/19 16:48

    DBの対象テーブルの「image」カラムの設定はどうなっていますか?nullableですか?

    キャンセル

  • toyop

    2020/07/19 17:07

    コメントありがとうございます。
    投稿する時点で画像は必須にしてあるので、nullableにはなっていないです。

    キャンセル

回答 1

0

このコードはここに問題があると思います。
このコードはDBに既に存在するレコードを上書きするのではなく、新たにレコードを作成するコードになっていること。

すでに存在するDBのレコードの値を上書きする際は、Modelクラスにあるsave()を使用するのではなく、update()を使用します。
update()についてはこちら
Laravel Database: Query Builder
日本語版
データベース:利用開始 7.x Laravel - ReaDouble

あと、SQLSTATE[HY000]: General error: 1364 Field 'image' doesn't have a default valueの言わんとすることは、現在存在しているテーブルのレコードのimageカラムは値が入ってなくてはいけないのに、何も入れてくれていないという規約違反をしているからエラーだよ、ということです。
今回の例で言うと、save()を使って新たにレコードを作成するのにも関わらずimageが空だからエラーを出しています。このエラーは上記の問題を解決させ、もし写真の変更がなければそのレコードのimageは何もしない、というコードを書けば表示されなくなると思います。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/07/19 18:49

    ご回答ありがとうございます。
    ご指摘いただいたtest->save()の2か所をtest->update()に変えました。
    エラーは出なくなったのですが、データベースを確認してもなぜかデータが更新されません...。
    これの原因もわかりますでしょうか。よろしくお願いいたします。

    キャンセル

  • 2020/07/19 19:01 編集

    update()で紹介した記事は読まれましたでしょうか?
    おそらくその記事ではtest->update()のような使い方はしていないと思います

    キャンセル

  • 2020/07/19 19:39

    ちなみに''DB::table''のDBは''use Illuminate\Support\Facades\DB;''を入れることで使用できます。

    キャンセル

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

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

関連した質問

同じタグがついた質問を見る