PHP中array_merge未合并数组，$mentor_fails返回空值问题排查

我帮你找到了两个核心问题，导致$mentor_fails无法正确记录失败的导师信息：

1. array_merge的使用错误 #

array_merge函数不会修改原数组，它只会返回一个合并后的新数组。你当前的代码只是调用了array_merge，但没有把返回值赋值给$mentor_fails，所以原数组始终是空的。

而且这里其实不需要用array_merge，直接往数组里追加元素更简单高效：

// 错误写法：
array_merge($mentor_fails, array('mentor_email'=>$mentor, 'project_id'=>$project));

// 正确写法：
$mentor_fails[] = array('mentor_email'=>$mentor, 'project_id'=>$project);

2. 异常捕获范围的问题 #

当前代码只在insert返回false时尝试记录失败信息，但如果数据库操作抛出了未被提前捕获的异常（比如SQL语法错误、权限问题），这部分记录逻辑根本不会执行，直接进入catch块，导致失败信息丢失。

应该把记录失败信息的逻辑放到catch块里，确保所有插入失败的情况（不管是返回false还是抛出异常）都能被记录。

修复后的完整代码 #

function assign_mentors_and_projects($mentors,$projects){ 
    global $mydb; 
    $mentor_fails = array(); 

    foreach($mentors as $mentor){ 
        foreach($projects as $project){ 
            try { 
                $update_mentors_and_projects_rlshp = $mydb->insert( 
                    'abcdefg', 
                    array('project_id'=>$project, 'mentor_id'=>$mentor), 
                    array('%s','%s')
                ); 

                if($update_mentors_and_projects_rlshp == false) { 
                    throw new Exception("exist already:(mentor-" . $mentor . " project_id-" . $project . ")\r\n"); 
                } 
            } catch (Exception $e) { 
                // 在这里记录失败信息，确保所有异常都能被捕获记录
                $mentor_fails[] = array(
                    'mentor_email' => $mentor, 
                    'project_id' => $project,
                    'error_message' => $e->getMessage() // 可选：记录错误详情，方便排查
                );
                echo $e->getMessage(); 
            } 
        } 
    } 
    return $mentor_fails; 
}

额外优化建议 #

• 尽量避免使用global变量传递数据库连接，建议通过函数参数传入，这样代码更健壮、可维护。
• 如果$mydb->insert在失败时已经会自动抛出异常，那可以去掉if($update_mentors_and_projects_rlshp == false)的判断，直接依赖异常捕获来处理所有失败情况。

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