More than 3 years have passed since last update.

Laravelで直接データベースをいじったら地獄を見た件

Last updated at 2022-03-21Posted at 2022-03-21

みなさん、こんにちは！

アプリを完成しましたが、データの削除の際に外部キーのエラーがあったので修正しようとしたところ、地獄を見たので愚痴らせてください。

サービスについてはこちら

laravel学んで2か月で自サービスを開発した話 Part1

何が起きたか

以前、凍結されたぶりを削除しようとしたら、

はい、外部キーの参照エラーです。
まあ、これは予想通りだったので、じゃけん直していこうね

注意！ここからは私の試行錯誤が含まれます。真似すると地獄を見ますのでやめましょう！

マイグレーションファイルを直接編集する

とりあえず、マイグレーションファイルを編集してphp artisan migrateすればええやろ！

create_processed_products_table.php

     Schema::create('processed_products', function (Blueprint $table) {
            $table->id();
            $table->foreignId('admin_id')->constrained('admins');
            $table->foreignId('product_id')->constrained();
            $table->foreignId('product_id')
            ->onUpdate('cascade')
            ->onDelete('cascade')
            ->constrained('products');
            $table->boolean('result');
            $table->timestamps();
        });

cascadeを付けて変更すればええやん！それで、php artisan migrate　っと

nothing migration

あ、あれっ？変更が反映されないぞ？

まあ、もしかしたら、変更がされているかもしれないし、削除ボタンと

知ってた

データベースを直接編集する

マイグレーションファイルが編集できないんだったら、データベースを直接編集すればええやん！

ということで、データーベースの外部キーをRESTRICTからCASCADEに変更してと・・・

これなら大丈夫でしょ！

３３－４

さすがに知り合いに聞いてみた。

エンジニアさん「このエラーに関してどんなアプローチをしたんですか？」

マイグレーションファイルとデータベースの値を直接変更しちゃった

エンジニアさん「今から緊急会議です。ZOOMつないでください。」

ええええええええええええ！！

結局何がいけなかったのか

エンジニアさんと実際に話してみてわかったことが

マイグレーションファイルは編集せずに、データベースに変更を加えるときは新しいマイグレーションファイルを作る
データベースを直接いじらない
データベースとマイグレーションの状況を合わせる
ということらしいので、さっそく助言もいただきながら修正してみた。

マイグレーションファイルの作成

すでにあるマイグレーションファイルをいじるのはご法度なので、マイグレーションファイルを作成します

processed_comments_table_drop_foreign.php

<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

class ProcessedCommentsTableDropForeign extends Migration
{

    public function up()
    {
        Schema::table('processed_comments', function (Blueprint $table) {
            $table->dropForeign('processed_comments_comment_id_foreign');
        });
    }

    public function down()
    {
        Schema::table('processed_comments', function (Blueprint $table) {
            $table->foreign('comment_id')->references('id')->on('comments');
        });

    }
}

このマイグレーションファイルを更新して既にある外部キーをドロップします

しかし、upメゾットだけだと、更新時にエラーが起きた時、完全性が損なわれるので、downメゾットにはupメゾットの逆のことを書いていきます
この場合だと、外部キーをつける工程をdownメゾットに書きます

この逆のメゾットを考えるのが難しかった・・・

また、勝手にデータベースをいじってcascadeをつけてしまった分を補完するマイグレーションファイルを作ります

change_foreignkey_restrict_to_cascade.php

<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

class ChangeForeignkeyRestrictToCascade extends Migration
{

    public function up()
    {

        Schema::table('processed_products', function (Blueprint $table) {
            $table->dropForeign('processed_products_product_id_foreign');
        });
        Schema::table('processed_comments', function (Blueprint $table) {
            $table->foreign('comment_id')->references('id')->on('comments')
            ->onUpdate('cascade')
            ->onDelete('cascade');
        });
        Schema::table('processed_products', function (Blueprint $table) {
            $table->foreign('product_id')->references('id')->on('products')
            ->onUpdate('cascade')
            ->onDelete('cascade');
        });
         
    }

    public function down()
    {
        Schema::table('processed_products', function (Blueprint $table) {
            $table->dropForeign('processed_products_product_id_foreign');
            $table->foreign('product_id')->references('id')->on('products');
        });
        Schema::table('processed_comments', function (Blueprint $table) {
            $table->dropForeign('processed_comments_comment_id_foreign');
        });
    }
}