Laravel查询parent_wbs_id不进入内层循环报数组转字符串错误

问题根因 #

• 内层循环不执行的核心原因：你使用单引号包裹SQL语句，PHP单引号字符串不会解析内部变量，实际执行的SQL是查询wbs_id = "$task1->wbs_id"的记录，匹配不到数据所以$parent返回空数组，自然不会进入内层遍历。同时直接拼接变量到SQL还存在SQL注入风险。
• 数组转字符串报错的原因：\DB::select返回的是对象数组集合，直接使用echo等字符串输出函数打印数组类型变量就会触发该错误。

修复代码方案 #

基础修复（参数绑定解决查询为空问题） #

foreach ($task as $task1) {
    // 使用参数绑定传递变量，避免注入和变量解析问题
    $parent = \DB::select('select parent_wbs_id from projwbs where wbs_id = ?', [$task1->wbs_id]);        
    foreach ($parent as $parent1){
        \DB::table('tasks')->insert([
            'id' => $task1->task_id, 
            'text' => $task1->task_name, 
            'start_date' => $task1->act_start_date, 
            'duration' => round($days), 
            'parent' => $parent1->parent_wbs_id,
            'progress' => 1.0
        ]);
    }
}

简化写法（无需嵌套循环） #

如果wbs_id是projwbs表的唯一键，可以直接取单个字段值，省略内层循环：

foreach ($task as $task1) {
    // 直接查询单个字段值，无需遍历
    $parentWbsId = \DB::table('projwbs')
        ->where('wbs_id', $task1->wbs_id)
        ->value('parent_wbs_id');
    
    // 判断取值成功再插入
    if ($parentWbsId) {
        \DB::table('tasks')->insert([
            'id' => $task1->task_id, 
            'text' => $task1->task_name, 
            'start_date' => $task1->act_start_date, 
            'duration' => round($days), 
            'parent' => $parentWbsId,
            'progress' => 1.0
        ]);
    }
}

调试建议 #

如果需要打印查询结果排查问题，使用Laravel提供的dd()函数或者原生var_dump()/print_r()输出变量，不要直接用echo：

// 调试打印$parent的结构
dd($parent);

性能优化建议 #

当前代码属于典型的N+1查询问题，循环内每次都查询数据库，数据量大时性能很差，可以优化为批量查询+批量插入：

// 先取出所有需要的wbs_id
$wbsIds = collect($task)->pluck('wbs_id');
// 一次性查询所有对应的parent_wbs_id，用wbs_id作为键
$parentMap = \DB::table('projwbs')
    ->whereIn('wbs_id', $wbsIds)
    ->pluck('parent_wbs_id', 'wbs_id');

$insertData = [];
foreach ($task as $task1) {
    // 直接从预查询的映射表取对应值
    if (isset($parentMap[$task1->wbs_id])) {
        $insertData[] = [
            'id' => $task1->task_id, 
            'text' => $task1->task_name, 
            'start_date' => $task1->act_start_date, 
            'duration' => round($days), 
            'parent' => $parentMap[$task1->wbs_id],
            'progress' => 1.0
        ];
    }
}
// 批量插入，减少数据库IO
if (!empty($insertData)) {
    \DB::table('tasks')->insert($insertData);
}

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