Laravel中save()处理关联已存在场景的优化方案咨询

嘿，这个场景我太熟悉了！其实Laravel的Eloquent已经给我们准备了原生方法，完全不用手动去查中间表，而且代码会更简洁优雅。先给你梳理两个最优方案：

方案一：用syncWithoutDetaching()自动处理重复关联 #

这个方法简直是为你的需求量身定做的——它会尝试添加关联，但如果关联已经存在，就会自动忽略，不会抛出错误，还能告诉你是否真的添加了新关联。

首先先修正下你模型里的关联小问题（看起来是笔误？）：你的Task模型关联的应该是自身，所以belongsToMany的参数应该是Task::class而不是ChecklistCategoryTask::class，修正后的关联如下：

class Task extends Model
{
    public function childs()
    {
        return $this->belongsToMany(Task::class, 'task_has_predecessors', 'parent_id', 'child_id');
    }

    public function parents()
    {
        return $this->belongsToMany(Task::class, 'task_has_predecessors', 'child_id', 'parent_id');
    }
}

然后优化你的业务代码：

// 验证逻辑...
$parent = Task::find(request('parent'));
$child = Task::find(request('child'));

// 尝试添加关联，已存在则忽略
$result = $parent->childs()->syncWithoutDetaching([$child->id]);

// 根据返回结果判断提示信息
if (!empty($result['attached'])) {
    flash('Link saved.');
} else {
    flash()->warning('This link already exist.');
}

syncWithoutDetaching()会返回一个数组，其中attached键包含刚添加的关联ID列表，如果为空就说明关联已经存在啦。

方案二：用关联的exists()方法提前检查 #

如果你更倾向于先判断再执行的逻辑，也可以用Eloquent关联自带的exists()方法来检查关联是否存在，完全不用直接操作中间表：

// 验证逻辑...
$parent = Task::find(request('parent'));
$child = Task::find(request('child'));

if (!$parent->childs()->where('child_id', $child->id)->exists()) {
    $parent->childs()->save($child);
    flash('Link saved.');
} else {
    flash()->warning('This link already exist.');
}

这个方法通过关联查询来判断，完全符合Eloquent的ORM风格，比手动查DB::table更优雅。

额外建议：给中间表加复合唯一索引 #

不管用哪种方案，都建议给task_has_predecessors表的parent_id和child_id添加复合唯一索引，这样从数据库层面保证不会出现重复关联，即使代码层面有疏漏也能避免数据异常。

内容的提问来源于stack exchange，提问作者Atnaize