优化用于插入、更新或删除关系的代码

Question

我正在编写基于提供的输入和现有数据插入、更新或删除表行的逻辑。

• 如果没有输入，并且该行存在，则应删除该行。
• 如果有输入，但该行不存在，则应该创建它。
• 最后，如果有输入且行存在，则应更新行。

该项目是attributes 表的测试代码，该表具有name、value 和media_id（它引用已被赋予属性的媒体片段）。 name 和media_id 一起是唯一的（一个媒体不能有两个同名的属性）。使用插入、更新和删除对项目很重要，而不是如果为空则将它们留空。

我的问题是是否有更简单的方法来执行此逻辑。我可以只将它用于一个输入，但是当有两个（name 和 description）并且我以后可能会添加更多时，它似乎需要使用大量代码。有没有更有效的方法？

---

我的数据库迁移：

Schema::create('attributes', function (Blueprint $table) {
    $table->increments('id')->unsigned();
    $table->integer('media_id')->unsigned();
    $table->string('name');
    $table->text('value')->nullable();
    $table->timestamps();
    $table->unique(['media_id', 'name']);
    $table->foreign('media_id')->references('id')->on('media')->onDelete('cascade');
});

---

我的看法：

<form action="{{ Request::path() }}" method="post">
    <input type="hidden" name="_token" value="{{ csrf_token() }}">
    <label style="display:block" for="name">Name</label>
    <input type="text" name="name" id="name" value="{{ $attributes->get('name') ? $attributes->get('name')->value : '' }}">
    <label style="display:block" for="description">Description</label>
    <textarea name="description" id="description">{{ $attributes->get('description') ? $attributes->get('description')->value : '' }}</textarea>
    <button type="submit">Save</button>
</form>

---

我的临时控制器：

Route::post('/', function(Request $request) {
    $media = Media::find(1);

    $attributes = $media->attributes->keyBy('name');

    if($request->input('name')) {
        if($attributes->get('name')) {
            $attributes->get('name')->value = $request->input('name');
            if(!$attributes->get('name')->save()) {
                return "error updating name";
            }
        } else {
            $attribute = new Attribute;
            $attribute->name = 'name';
            $attribute->value = $request->input('name');
            if(!$media->attributes()->save($attribute)) {
                return "error inserting name";
            }
        }
    } else {
        if($attributes->get('name')) {
            if(!$attributes->get('name')->delete()) {
                return "error deleting name";
            }
        }
    }
    if($request->input('description')) {
        if($attributes->get('description')) {
            $attributes->get('description')->value = $request->input('description');
            if(!$attributes->get('description')->save()) {
                return "error updating description";
            }
        } else {
            $attribute = new Attribute;
            $attribute->name = 'description';
            $attribute->value = $request->input('description');
            if(!$media->attributes()->save($attribute)) {
                return "error inserting description";
            }
        }
    } else {
        if($attributes->get('description')) {
            if(!$attributes->get('description')->delete()) {
                return "error deleting description";
            }
        }
    }
});
return redirect('/');

（还有一个用于 GET 的控制器，它显示表单和数据库输出。我没有打扰它，因为我觉得没有必要。）

---

我死的简单模型：

class Attribute extends Model
{
    public function media()
    {
        return $this->belongsTo('App\Media');
    }
}

Answer 1

好的，这就是我得到的：

路线文件：

use Illuminate\Http\Request;
use App\Models\Media;
use App\Models\Attribute;

Route::get('/', function(Request $request) {
    $media = Media::all();
    $media->attributes;
    return response()->json($media);
});
Route::get('/media/{id}', function($id, Request $request) {
    $media = Media::find($id);
    $media->attributes;
    return response()->json($media);
});
Route::post('/media/{id}', function($id, Request $request) {
    if($request->has('name')) {
        $attr = Attribute::firstOrNew(['name' => 'name', "media_id" => $id]);
        $attr->value = $request->input('name');
        $attr->save();
    } else {
        Attribute::where('name', 'name')->where("media_id",$id)->first()->delete();
    }
    if($request->has('description')) {
        $attr = Attribute::firstOrNew(['name' => 'description', "media_id" => $id]);
        $attr->value = $request->input('description');
        $attr->save();
    } else {
        Attribute::where('name', 'description')->where("media_id",$id)->first()->delete();
    }
});
return redirect('/');

媒体模型：

namespace App\Models;

use Illuminate\Database\Eloquent\Model;

class Media extends Model
{
    protected $table = 'media';

    protected $appends = [];

    public function attributes() {
        return $this->hasMany(Attribute::class);
    }
}

属性模型

namespace App\Models;

use Illuminate\Database\Eloquent\Model;

class Attribute extends Model
{
    protected $table = 'attributes';

    protected $fillable = ['name', 'media_id'];

    public function media() {
        return $this->belongsTo(Media::class);
    }
}

基本上，通过将 Attributes 设为模型，我能够在属性表上执行 firstOrNew()，并在 firstOrNew() 上查找 ['name' => 'name', "media_id" => $id] 使其要么找到匹配的行，要么创建一个新行.然后，您可以根据这些更新值列。

更短，更简化，它可以让你的属性成为一个模型！

（我认为，几乎所有东西都应该是模型：P）

编辑：我将其修改为我最近的代码。将其从 belongsTo("Media") 更改为 belongsTo(Media::class)，并从路由文件中删除了 dd()。