Laravel Eager Loading失效排查：关联数据重复查询

嘿，我一眼就看出问题所在了——你在视图里调用的$classroom->availableHours()（带括号）是在重新触发查询构造器，完全没用到你预加载好的关联数据！

为什么会这样？ #

当你在模型里定义availableHours()关联方法时，带括号调用它会返回一个查询构造器实例（比如HasMany），每次调用都会执行新的SQL查询，不管你有没有预加载。而预加载的数据其实是存在$classroom->availableHours（不带括号）这个集合属性里的，这才是你已经加载好的、不需要再查库的数据。

你之前的视图代码里，每次循环都用where([['day_id','=',$day->id],['hour_of_day',$i]])->first()，相当于每次循环都发起一次新的SELECT请求，这就是Telescope显示大量重复查询、请求耗时近2秒的原因。

怎么解决？ #

我们需要先把预加载的availableHours集合整理成一个便于快速查找的映射结构，比如按day_id-hour_of_day作为键，这样循环时直接通过键取值，不用再查数据库。

第一步：在控制器里构建映射

在你预加载关联之后，添加一段代码把集合转成键值对：

$classroom->load('availableHours.allowedDepartments.exclusiveSubjects.subject', 'availableHours.allowedDepartments.excludedSubjects.subject', 'availableHours.allowedDepartments.department');

// 把可用时段按 day_id-hour_of_day 分组，方便快速查找
$availableHoursMap = $classroom->availableHours->keyBy(function($hour) {
    return "{$hour->day_id}-{$hour->hour_of_day}";
});

// 把映射传给视图
return view('classrooms.show', compact('classroom', 'days', 'maxHour', 'availableHoursMap'));

第二步：修改视图代码，用映射替代查询

把视图里的循环逻辑改成直接从映射里取数据，彻底告别重复查询：

<!-- 遍历一天中的所有时段 -->
@for($i=1;$i<=$maxHour;$i++)
<tr>
    <td><a href="/classrooms/{{$classroom->number}}/0/{{$i}}/edit">{{$i}}</a></td>
    <!-- 遍历所有日期 -->
    @foreach($days as $day)
        @php 
            // 生成当前日期+时段的键
            $lookupKey = "{$day->id}-{$i}";
        @endphp
        @if(isset($availableHoursMap[$lookupKey]))
            @php 
                // 直接从映射里取预加载好的时段数据
                $hour = $availableHoursMap[$lookupKey];
            @endphp
            <td>
                <a href="/classrooms/{{$classroom->number}}/{{$day->id}}/{{$i}}/edit">
                    <!-- 显示时段容量 -->
                    {{ $hour->capacity }}<br/>
                    @foreach($hour->allowedDepartments as $allowedDepartment)
                        <!-- 显示允许的院系及专属/排除科目 -->
                        {{ $allowedDepartment->department->name }}
                        @foreach ($allowedDepartment->exclusiveSubjects as $subjectDepartment)
                            {{ $subjectDepartment->subject->code }}
                        @endforeach<br/>
                    @endforeach
                </a>
            </td>
        @else
            <!-- 没有可用时段的情况，可自定义显示样式 -->
            <td class="bg-red-500 text-white">不可用</td>
        @endif
    @endforeach
</tr>
@endfor

额外优化建议 #

• 尽量把数据处理逻辑放在控制器或服务层，视图只负责渲染，这样代码更易维护。
• 如果需要频繁做这种键值映射，可以考虑在模型里添加一个访问器方法，比如getAvailableHoursMapAttribute()，这样每次预加载后都能直接用$classroom->available_hours_map。

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