我经常遇到这样的情况:处理可以是数组或空变量的数据,并将这些数据提供给一些foreach。

$values = get_values();

foreach ($values as $value){
  ...
}

当你给foreach提供不是数组的数据时,你会得到一个警告:

警告:在[…]中为foreach()提供的参数无效

假设不可能重构get_values()函数以总是返回一个数组(向后兼容,没有可用的源代码,无论其他原因),我想知道哪种是避免这些警告的最干净和最有效的方法:

将$值转换为数组 初始化数组的$values 用if包foreach 其他(请建议)


当前回答

我通常使用类似这样的结构:

/**
 * Determine if a variable is iterable. i.e. can be used to loop over.
 *
 * @return bool
 */
function is_iterable($var)
{
    return $var !== null 
        && (is_array($var) 
            || $var instanceof Traversable 
            || $var instanceof Iterator 
            || $var instanceof IteratorAggregate
            );
}

$values = get_values();

if (is_iterable($values))
{
    foreach ($values as $value)
    {
        // do stuff...
    }
}

请注意,这个特定的版本没有经过测试,它直接从内存输入到SO中。

编辑:增加可遍历检查

其他回答

我通常使用类似这样的结构:

/**
 * Determine if a variable is iterable. i.e. can be used to loop over.
 *
 * @return bool
 */
function is_iterable($var)
{
    return $var !== null 
        && (is_array($var) 
            || $var instanceof Traversable 
            || $var instanceof Iterator 
            || $var instanceof IteratorAggregate
            );
}

$values = get_values();

if (is_iterable($values))
{
    foreach ($values as $value)
    {
        // do stuff...
    }
}

请注意,这个特定的版本没有经过测试,它直接从内存输入到SO中。

编辑:增加可遍历检查

从PHP >= 7.1.0开始使用is_iterable

https://www.php.net/manual/en/function.is-iterable.php

if (is_iterable($value)) {
    foreach ($value as $v) {
        ...
    }
}

这似乎也与环境有关:

我有“无效参数提供foreach()”错误仅在开发环境中,但不在prod(我在服务器上工作,而不是localhost)。

尽管有错误,但var_dump指示数组在那里(在app和dev两种情况下)。

围绕foreach ($array as $subarray)的if (is_array($array))解决了这个问题。

对不起,我不能解释原因,但我花了一段时间来找出解决方案,我想更好地分享这作为一个观察。

首先,每个变量都必须初始化。总是这样。 强制转换不是一个选项。 如果get_values ();可以返回不同类型的变量,当然这个值必须检查。

如果你正在使用php7,你只想处理未定义的错误,这是最干净的IMHO

$array = [1,2,3,4];
foreach ( $array ?? [] as $item ) {
  echo $item;
}