由于引用变量传递,为 foreach 提供的参数无效

Getting Invalid argument supplied for foreach as a result of pass by reference variable

我正在升级使用引用传递的代码库 主要功能

    function splitSqlFile(&$ret, $sql)
    {
       $sql               = trim($sql);
       $sql_len           = strlen($sql);
       $char              = '';
       $string_start      = '';
       $in_string         = false;

      for ($i = 0; $i < $sql_len; ++$i) {
           $char = $sql[$i];
        if ($in_string) {
            for (;;) {
                $i = strpos($sql, $string_start, $i);
                if (!$i) {
                    $ret[] = $sql;
                    return true;
                }else if ($string_start == '`' || $sql[$i-1] != '\'){
                    ......
                }else {
                    ......
                } // end if...elseif...else
            } // end for
        }
        else if ($char == ';') {
            $ret[]    = substr($sql, 0, $i);
            $sql      = ltrim(substr($sql, min($i + 1, $sql_len)));
            $sql_len  = strlen($sql);
            if ($sql_len) {
                $i = -1;
            } else {
                // The submited statement(s) end(s) here
                return true;
            }
        }else if (($char == '"') || ($char == '\'') || ($char == '`')) {
            $in_string    = true;
            $string_start = $char;
        } // end else if (is start of string)

        // for start of a comment (and remove this comment if found)...
        else if ($char == '#' || ($char == ' ' && $i > 1 && $sql[$i-2] . $sql[$i-1] == '--')) {
            ......
            if (!$end_of_comment) {
            // no eol found after '#', add the parsed part to the returned
            // array and exit
                $ret[]   = trim(substr($sql, 0, $i-1));
                return true;
            } else {
                .....
            } // end if...else
        } // end else if (is comment)
    } // end for

    // add any rest to the returned array
    if (!empty($sql) && trim($sql) != '') {
        $ret[] = $sql;
    }
    return true;
}

调用函数

    $sqlUtility->splitSqlFile($pieces, $sql_query);
    foreach ($pieces as $piece) 
    {
      .......
    }

如果上面的变量splitSqlFile(&$ret, $sql)前面有“&”,程序运行成功,但是如果去掉,现在splitSqlFile($ ret, $sql), 它会开始返回 'invalid argument supplied for foreach' error.and 当我尝试使用 "is_array" 函数检查它是否是一个数组时,结果总是 "NULL".

为什么会出现错误:

通过从 $ret 中删除 &,您将不再在函数调用中引用该变量。在这种情况下,$pieces。因此,当您在调用函数后对 $pieces 执行 foreach 时,它将出错,因为 $pieces 在该点基本上是一个空变量。

function splitSqlFile(&$ret,$sql) {
    $ret[] = 'stuff';
}
splitSqlFile($pieces,$sql);
// $pieces will be an array as 0 => 'stuff'
foreach ($pieces as $piece) { } // will not error

对比:

function splitSqlFile($ret,$sql) {
    $ret[] = 'stuff';
}
splitSqlFile($pieces,$sql);
// $pieces will be a null variable, since it was never assigned anything
foreach ($pieces as $piece) { } // will error

替代无参考:

因此,如果您想删除 & 并且不再通过引用传递,则必须对函数进行其他更改以取回该值。根据代码库的不同,这可能意味着在使用该函数的任何地方都需要做大量工作!

示例:

function splitSqlFile($sql) {
    $ret = [];
    $ret[] = 'stuff';
    return array('result'=>true,'ret'=>$ret);
}
// $result will contain multiple things to utilize

// if you will only need that variable once (does not accumulate)
$result = splitSqlFile($sql);
foreach ($result['pieces'] as $piece) { }

// if that variable is added by multiple calls, and displayed later... merge
$pieces = [];
$result = splitSqlFile($sql_1);
$pieces = array_merge($pieces,$result['pieces']);
$result = splitSqlFile($sql_2);
$pieces = array_merge($pieces,$result['pieces']);
foreach ($pieces as $piece) { }

第二个示例(随手传入数组...会让人感到困惑):

function splitSqlFile($pieces_in,$sql) {
    $pieces_in[] = 'stuff';
    return array('result'=>true,'pieces_out'=>$pieces_in);
}
$pieces = [];
$result = splitSqlFile($pieces,$sql_1);
$pieces = $result['pieces_out'];
$result = splitSqlFile($pieces,$sql_2);
$pieces = $result['pieces_out'];
foreach ($pieces as $piece) { }

如您所见,它不仅改变了必须处理的 return 值,而且还改变了它的调用方式。再一次,如果这个函数在代码中的一千个地方被使用……严重的头痛!

结论:

老实说,我会保留原样。这样做是为了让调试数据的积累更容易、更直接。否则,您需要进行大量代码更改才能删除引用。

不过这只是我对此事的看法。