好吧,我会这样做的 .
首先给出变量名称,以帮助您了解其中的内容
第二个总是在 if else 中使用 {} ,即使 if 或 else 中只有一行,它也可以更容易地看到实际开始和结束的位置 .
然后总是使用缩进,这样您就可以直观地看到代码块的开始和结束位置 . 请记住,您可能需要在编写代码后的几周内回复一段代码,因此您可以轻松阅读,因为您可能需要了解您的工作 .
此外,当您输出有关匹配或不匹配的信息时,还需要包含您需要查看的数据,以便判断匹配是否正确 . 然后,如果它没有做你想要的,你可以很容易地看到什么是错的 .
$temp_answers = array();
$questions = array();
$temp_answers = Tempanswer::where('subject_id', $subject_id)
->where('student_id', $student_id)
->lists('temp_answer');
$questions = Question::where('subject_slug', $subject->slug)
->lists('letteranswer');
foreach ($temp_answers as $temp_answer) {
foreach ($questions as $question ) {
if($question == $temp_answer){
echo "Question = $question TempAnswer = $temp_answer > MATCH" . PHP_EOL;
} else {
echo "Question = $question TempAnswer = $temp_answer > MISMATCH" . PHP_EOL;
}
}
}