SQLSTATE[HY000]: General error: 1364 Field '〇〇' doesn't have a default value の解決方法

Question

PHP Laravel

roomテーブルとmessageテーブルがあり、room hasMany messageの関係です。

以下のフォームでメッセージを送信すると下記のエラーが出てしまします。

●フォーム

<input type="hidden" name="send_by" value="company">
    <input type="hidden" name="room_id" value="{{$room->id}}">
    <input type="text" name="text">
    <input type="submit" value="送信">

---

●エラー分
SQLSTATE[HY000]: General error: 1364 Field 'room_id' doesn't have a default value (SQL: insert into messages (text, updated_at, created_at) values (テスト, 2021-12-14 07:41:31, 2021-12-14 07:41:31))

誤字や、変数の設定などはミスがないと思いますが、どこが原因なのか教えていただけますでしょうか？ value="{{$room->id}}"をvalue="1"のように置き換えても同じエラーが出ました。

以下メッセージテーブルの内容になります。

public function up()
    {
        Schema::create('messages', function (Blueprint $table) {
            $table->id();
            $table->integer('room_id');
            $table->string('send_by');
            $table->string('text');
            $table->timestamps();
        });
    }

Messageテーブルの構造になります。

Messageコントローラー

class MessageController extends Controller
{
    public function CompanyMessageStore(Request $request)
    {
        $user = Auth::user();

        $message = new Message();

        $message->text = $request->text();
        $message->save();

        return back();
    }
}

Messageモデル
<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;

class Message extends Model
{
use HasFactory;
public function room()
{
return $this->belongsTo('App\Models\Room');
}
}

Roomモデル

 public function messages()
    {
        return $this->hasMany('App\Models\Message');
    }

Comments

[skys215]

データを保存するコードとmodelファイルのコードをお願いします。

[Yoshiaki_Miyata]

失礼いたしました。
以下になります。

---------------------------------------
コントローラー
public function CompanyMessageStore(Request $request)
{
$user = Auth::user();

$message = new Message();

$message->text = $request->text();
$message->save();

return back();
}

-------------------------
Messageモデル
<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;

class Message extends Model
{
use HasFactory;
public function room()
{
return $this->belongsTo('App\Models\Room');
}
}
-------------------------
Roomモデル

public function messages()
{
return $this->hasMany('App\Models\Message');
}

Answer 1

主題以外にもミスがあるのでまとめて指摘します。

messages.room_id の外部キー制約

public function up()
{
    Schema::create('messages', function (Blueprint $table) {
        $table->id();
        $table->integer('room_id')->constrained(); // ←これを追加
        $table->string('send_by');
        $table->string('text');
        $table->timestamps();
    });
}

こうしておくと，存在する rooms.id 以外は messages.room_id にインサートできなくなるので，制約事項として妥当になります。

messages.text の NOT NULL 制約

最初 $table->string('text') の部分を $table->string('text')->nullable() にしていませんでしたか？画像を見る限り，デフォルト値が NULL で NULL 許容になっているので，php artisan migrate:fresh を実行するなどして作り直してください。

$request->text() メソッドは存在しない

$request->text の間違いです。但し，以下のような問題点がありますので，代わりに $request->input('text') を使用することを個人的には推奨しています。

• Laravel の request() や Request::__get() は地雷， Request::input() と Request::route() を使え - Qiita

class MessageController extends Controller
{
    // 蛇足: できればメソッド名は companyMessageStore のように小文字で始まるほうが望ましい
    public function companyMessageStore(Request $request)
    {
        $user = Auth::user();

        $message = new Message();
        $message->text = $request->input('text'); // ←ここを修正
        $message->save();

        return back();
    }
}

本題: room_id のセット

フォームから送信された room_id が使われていないので，コントローラで作成するモデルにセットするコードを書いてください。

class MessageController extends Controller
{
    public function companyMessageStore(Request $request)
    {
        $user = Auth::user();

        $message = new Message();
        $message->room_id = $request->input('room_id'); // ←これを追加
        $message->text = $request->input('text');
        $message->save();

        return back();
    }
}

但し， BelongsTo の外部キーに関してはリレーションを使ってセットするほうがお作法としてはきれいです。

<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;

class Message extends Model
{
    use HasFactory;

    public function room()
    {
        return $this->belongsTo(Room::class); // ←蛇足: https://www.php.net/manual/ja/language.oop5.basic.php#language.oop5.basic.class.class
    }
}

class MessageController extends Controller
{
    public function companyMessageStore(Request $request)
    {
        $user = Auth::user();

        $message = new Message();
        $message->room()->associate($request->input('room_id')); // ←変更
        $message->text = $request->input('text');
        $message->save();

        return back();
    }
}

個人的にはほぼこれで理想形と感じる形です。あとは $request のバリデーションも含められれば，十分実用的なコードになると思います。

Comments

[Yoshiaki_Miyata]

ご回答いただきありがとうございました。

質問以外でも適切でない箇所が複数ありましたので、そちらへのアドバイスもありがとうございます。

いただいた内容で試したところ、無事に正しく動きました。今後も同じ開発の際に活かしていきます。

Answer 2

room_idはNOT NULLの為、必須になっております。
メッセージを挿入時点ではroom_idが指定してないのでこのエラーが出ました。

解決策１：
room_idは必須であれば、挿入の時にroom_idを指定する。

public function CompanyMessageStore(Request $request){
...
$member_id = 123456789;//<--
$message->member_id = $member_id; //<--
$message->text = $request->text();

解決策２：
room_idは無くてもいいなら、マイグレーションを使ってroom_idをnullable(true)にする。

Comments

[Yoshiaki_Miyata]

回答ありがとうございます！
room_idは必須なので、解決策1で考えております。

重ねてご質問失礼いたします。

>room_idは必須であれば、挿入の時にroom_idを指定する。

とありますが、メッセージフォームに
<input type="hidden" name="room_id" value="{{$room->id}}">

と「hidden」を指定すれば、勝手に挿入されると考えていたのですが、そうではなくて、

input type="hidden"はあ、あくまでも、こちらから入力をしなくてもコントローラーに渡すという意味で、コントローラーに渡ったとは、こちらで$room_idを指定する必要があるということでしょうか？

お手数ですがご確認お願い致します。

[skys215]

はい、ただコントローラーに渡すだけです。
もしも指定必要なくてもいいならば、$message->text = $request->text();は書かなくてもいいじゃないでしょうか。
コントローラーは渡してきたデータはどう使うかはわかりませんので、指定する必要があります。
ただ、$request->all()と組み合わせば一個一個指定しなくても済む方法はあります。
https://readouble.com/laravel/8.x/ja/eloquent.html　の「複数代入」に書いてあります。

Answer 3

質問にmessagesのCREATE TABLE文を追記してください。

insert into messages (text, updated_at, created_at) values (テスト, 2021-12-14 07:41:31, 2021-12-14 07:41:31)
              ↓
insert into messages (text, updated_at, created_at) values ('テスト', '2021-12-14 07:41:31', '2021-12-14 07:41:31')

SQLの掟として文字列はシングルクォートで囲まなければなりません。
DATETIME型も同様。参考 MySQL 日付型と時刻型(DATE, TIME, DATETIME, TIMESTAMP, YEAR)