使用Laravel Collections对比Eloquent查询结果遇空数组问题
解决Laravel Collection diff返回空的问题
我来帮你分析下这个问题——你遇到的情况其实是Laravel Collection的diff方法默认比较逻辑导致的,具体原因和解决方案如下:
问题根源
Laravel Collection的diff方法对于Eloquent模型实例,默认是通过模型的主键(也就是这里的users.id)来判断两个元素是否相等。而你的两个查询都是针对同一个用户(where('users.id', '=', $idUser)),所以返回的所有模型实例的主键都是同一个值。这就导致diff会错误地认为$todo里的所有元素都存在于$done中,最终返回空集合。
解决方案
方法1:用diffUsing自定义比较规则
直接指定判断两个培训是否相同的逻辑,比如通过formation_id和typesession_id的组合来唯一标识一个培训会话:
$remainingCollection = $todo->diffUsing($done, function ($item1, $item2) { // 当两个培训的formation_id和typesession_id完全匹配时,视为同一个元素 if ($item1->formation_id === $item2->formation_id && $item1->typesession_id === $item2->typesession_id) { return 0; // 返回0表示两个元素相等 } // 这里返回排序用的比较结果,可根据实际字段调整 return $item1->formation_id <=> $item2->formation_id; }); $remainingCollection = $remainingCollection->all(); return $remainingCollection;
方法2:转换为关联数组后求差集
先把两个集合转换成以培训唯一标识为键的关联数组,再用diffKeys方法求差集:
// 将$todo转换为以"formation_id-typesession_id"为键的数组 $todoMap = $todo->mapWithKeys(function ($item) { $uniqueKey = "{$item->formation_id}-{$item->typesession_id}"; return [$uniqueKey => $item]; }); // 同理转换$done $doneMap = $done->mapWithKeys(function ($item) { $uniqueKey = "{$item->formation_id}-{$item->typesession_id}"; return [$uniqueKey => $item]; }); // 求差集并恢复为索引数组 $remainingCollection = $todoMap->diffKeys($doneMap)->values()->all(); return $remainingCollection;
方法3:直接在数据库层面查询未完成的培训(推荐)
其实可以不用查询两个集合再在内存中求差,直接写一个数据库查询,一次性获取用户需要完成但尚未完成的培训,这样性能更优:
$remainingCollection = User::select('formations.typesession_id', 'formation_metier_ua.formation_id', 'formations.codeformation', 'users.id', 'formations.nomformation', 'users.nom', 'users.prenom', 'formations.dureeformation') ->distinct() ->join('uas', 'users.ua_id', '=', 'uas.id') ->join('formation_metier_ua', 'uas.id', '=', 'formation_metier_ua.ua_id') ->join('formations', 'formation_metier_ua.formation_id', '=', 'formations.id') ->join('typesessions', 'typesessions.id', '=', 'formations.typesession_id') ->where('users.id', '=', $idUser) // 排除已经完成的培训ID ->whereNotIn('formations.id', function ($subQuery) use ($idUser) { $subQuery->select('parcoursformations.formation_id') ->from('parcoursformations') ->where('parcoursformations.user_id', '=', $idUser); }) ->get() ->all(); return $remainingCollection;
这个方法减少了一次数据库查询,直接在数据库层面完成过滤,避免了内存中处理集合的开销,是最推荐的解决方案。
内容的提问来源于stack exchange,提问作者davidvera
相关产品推荐
相关产品推荐

