Laravel使用DB Seeder批量更新时foreach报Undefined array key 1错误求助

报错根因 #

你代码中默认$application->answers的遍历索引和$application->vacancy->questions()返回的ID数组索引是一一对应的，但实际运行时两者长度不匹配：当answers的数量多于questions数组的长度时，遍历到大于questions数组最大下标的$key就会抛出Undefined array key错误。
另外这种无显式排序的索引绑定逻辑，即使长度匹配也可能出现answer和question对应关系错位的问题。

修复代码 #

基础修复（避免报错） #

先做长度校验，仅对两者重叠的索引执行更新：

Application::chunk(500, function($applications) {
    foreach($applications as $application){
        $questions = $application->vacancy->questions()->pluck('id')->toArray();
        $answers = $application->answers ?? [];
        $matchCount = min(count($questions), count($answers));
        if($matchCount > 0){       
            for ($i = 0; $i < $matchCount; $i++) {
                $answers[$i]->update([
                    'vacancy_question_id' => $questions[$i]
                ]);
            }
        }
    }
});

严谨修复（保证对应关系正确） #

如果业务逻辑要求answer和question按创建顺序一一对应，需要对两个关联查询统一加排序规则，避免顺序错位：

Application::chunk(500, function($applications) {
    foreach($applications as $application){
        // 显式按创建时间升序取问题ID，保证顺序固定
        $questions = $application->vacancy->questions()
            ->orderBy('created_at', 'asc')
            ->pluck('id')
            ->toArray();
        // 回答也按相同规则排序
        $answers = $application->answers()
            ->orderBy('created_at', 'asc')
            ->get() ?? [];
        $matchCount = min(count($questions), count($answers));
        if($matchCount > 0){       
            for ($i = 0; $i < $matchCount; $i++) {
                $answers[$i]->update([
                    'vacancy_question_id' => $questions[$i]
                ]);
            }
        }
    }
});

批量更新优化建议 #

当前代码每个answer单独执行一次update查询，批量处理数据时性能较低，可以整理更新数据后用批量更新逻辑减少数据库请求：

// 示例：单application下的answer批量更新逻辑
$updateData = [];
for ($i = 0; $i < $matchCount; $i++) {
    $updateData[] = [
        'id' => $answers[$i]->id,
        'vacancy_question_id' => $questions[$i]
    ];
}
// 用upsert批量更新，唯一键为id
Answer::upsert($updateData, ['id'], ['vacancy_question_id']);

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