Laravel关联关系下测试页面层级信息展示优化问题

这个问题的核心是视图里的关联逻辑没有覆盖所有层级的关联路径——班级级测试的Department和School是通过Class间接关联的，不是Test直接关联的，所以直接调用$test->department会拿到空集合，进而调用first()后取name就会报错。

最优雅的解决方式是把层级名称的拼接逻辑从视图移到Test模型中，封装成一个访问器或者方法，这样视图只需要调用一个属性就能得到正确的展示文本，同时也符合MVC的职责分离原则。

步骤1：在Test模型中添加访问器 #

在你的Test模型里，新增一个访问器来处理不同场景的名称拼接：

/**
 * 获取测试的层级名称（学校 | 部门 | 班级）
 */
public function getHierarchyNameAttribute()
{
    $nameParts = [];

    // 场景1：班级级测试（优先处理，因为班级关联包含完整层级）
    if ($this->class->isNotEmpty()) {
        $class = $this->class->first();
        // 用optional()避免关联为空时的报错
        $nameParts[] = optional($class->department->school)->name;
        $nameParts[] = optional($class->department)->name;
        $nameParts[] = $class->name;
    }
    // 场景2：部门级测试
    elseif ($this->department->isNotEmpty()) {
        $department = $this->department->first();
        $nameParts[] = optional($department->school)->name;
        $nameParts[] = $department->name;
    }
    // 场景3：校级测试
    elseif ($this->school->isNotEmpty()) {
        $nameParts[] = $this->school->first()->name;
    }

    // 过滤掉空值（比如关联为空的情况），然后用 | 连接
    return implode(' | ', array_filter($nameParts));
}

步骤2：在视图中调用访问器 #

现在你的Blade视图可以简化成一行代码，完全不用处理复杂的判断逻辑：

{{ $test->hierarchy_name }}

额外优化：预加载关联避免N+1查询 #

为了提升性能，在查询Test数据的时候，记得预加载所有需要的关联，避免重复查询数据库：

// 在控制器中查询Test时预加载关联
$tests = Test::with([
    'school',
    'department',
    'class.department.school' // 预加载班级关联的部门和学校
])->get();

为什么这样更优雅？ #

• 职责分离：数据逻辑放在模型层，视图只负责展示，后续层级规则变化时，只需要修改模型里的方法，不用改动视图。
• 代码简洁：视图里一行代码搞定，可读性更强。
• 错误防护：用optional()处理可能为空的关联，避免出现Trying to get property 'name' of non-object这类报错。
• 可扩展性：如果以后新增层级（比如年级），只需要在模型的访问器里新增对应的逻辑即可。

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