Laravel 5.6查询Bagian表数据代码报错，求排查解决

我来帮你梳理下代码里的问题，主要有两个关键错误导致无法正常返回Bagian数据：

错误点分析 #

• 错误1：获取User时用错了方法
  User::where('nopek', '=', $nopek)->get()返回的是一个Collection集合，不是单个User模型。集合对象没有bagian属性，所以直接访问$user->bagian肯定会触发报错。你需要用first()方法获取匹配条件的单个User模型，才能正常访问其字段。

• 错误2：Bagian查询的写法逻辑错误
  Bagian::all()->where('kode_bagian', '=', $user->bagian)->get()存在两个问题：

  1. all()已经执行查询并返回全表数据的集合，后续的where是集合的内存筛选方法，这种先查全表再筛选的方式效率极低；
  2. 集合对象没有get()方法，这会直接触发「方法不存在」的报错。

修复后的基础版本代码 #

public function AuditorBagian_Edit($nopek) { 
    // 使用first()获取单个User模型，而非get()返回集合
    $user = User::where('nopek', $nopek)->first();

    // 先判断用户是否存在，避免空模型引发后续报错
    if (!$user) {
        return response()->json(['message' => '用户不存在'], 404);
    }

    // 直接用查询构造器在数据库层面筛选数据，无需先查全表
    $bagian_user = Bagian::where('kode_bagian', $user->bagian)->get(); 

    return response()->json($bagian_user); 
}

更优雅的Laravel关联写法（推荐） #

如果User和Bagian存在关联关系（比如User属于某个Bagian），可以通过模型关联简化代码，同时避免N+1查询问题：

第一步：在User模型中定义关联 #

// app/User.php
public function bagian() {
    // 第二个参数是User表中的关联字段（bagian），第三个参数是Bagian表的对应字段（kode_bagian）
    return $this->belongsTo(Bagian::class, 'bagian', 'kode_bagian');
}

第二步：控制器中使用预加载获取关联数据 #

public function AuditorBagian_Edit($nopek) { 
    // 使用with('bagian')预加载关联数据，避免多次查询数据库
    $user = User::where('nopek', $nopek)->with('bagian')->first();

    if (!$user) {
        return response()->json(['message' => '用户不存在'], 404);
    }

    // 直接返回关联的Bagian数据
    return response()->json($user->bagian); 
}

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