Laravel Controller中For循环仅执行一次就中断问题排查

架构老林

2026-9-24

问题根源

你初始化$examQuestions时只创建了查询构造器实例，没有提前将对应考试的所有题目查询为集合，每次循环调用find()都会单独执行一次SQL查询，不仅效率极低，还可能出现查询约束异常。
缺少空值判断逻辑：如果某道查询到的题目没有设置isCorrect=true的正确答案，firstWhere()会返回null，此时访问$correctAnswer->content会触发致命错误，直接终止脚本执行，表现就是循环只跑了一次就中断。
for循环依赖数组索引连续，一旦$takenQuestions的索引出现断裂就会出现取值异常。

修复方案

直接替换为以下代码即可：

public function calculateResults(){
    $totalCorrect = 0;
    $takenQuestions = request()->input('taken_questions');
    $givenAnswers = request()->input('answer');
    $exam_id = request()->input('exam_id');
    // 提前一次性查询该考试的所有题目，按题目ID作为集合键，方便后续快速查找
    $examQuestions = examQuestion::where('exam_id',$exam_id)->get()->keyBy('id');

    // 改用foreach遍历，不用关心数组索引问题
    foreach($takenQuestions as $index => $questionId){
        // 从预查的集合中获取题目
        $givenQuestion = $examQuestions->get($questionId);
        if(!$givenQuestion) continue;
        
        $correctAnswer = $givenQuestion->answers->firstWhere('isCorrect',true);
        // 增加正确答案空判断，避免报错
        if(!$correctAnswer) continue;

        if($correctAnswer->content == $givenAnswers[$index]){
            $totalCorrect++;
        }
    }

    dd($totalCorrect);
}