标记为byref的函数参数（如&$errors）不一定是错误的，但它们对我来说是一种代码气味，因为我个人只需要在异常情况下这样做。在您的情况下，create_resource_banner($resource_id, &$errors)只使用$errors检查是否在其他地方有错误，这很奇怪，并且破坏了封装。* 调用 * 函数的代码应该只在没有以前的错误的情况下这样做。
所以，与之相反的是：

$bannerResult = create_resource_banner($resource_id, $errors);
if ($bannerResult !== null && $bannerResult['error'] && isset($bannerResult['message'])) {
    $errors[] = $bannerResult['message'];
}

字符串
我会这样写：

if(empty($errors)) {
    $bannerResult = create_resource_banner($resource_id);
    if ($bannerResult !== null && $bannerResult['error'] && isset($bannerResult['message'])) {
        $errors[] = $bannerResult['message'];
    }
}

型

我通过创建单独的函数来处理错误和执行必要的任务来解决这个问题。我调整了代码以匹配其余部分并正确工作。我不得不移动一些东西，但现在它工作了。