Invalid argument supplied for foreach()

A menudo se me ocurre manejar datos que pueden ser una matriz o una variable nula y alimentar algunos foreachcon estos datos.

$values = get_values();

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

Cuando alimenta a un foreach con datos que no son una matriz, recibe una advertencia:

Advertencia: argumento no válido proporcionado para foreach () en [...]

Suponiendo que no es posible refactorizar la get_values()función para que siempre devuelva una matriz (compatibilidad con versiones anteriores, código fuente no disponible, cualquier otra razón), me pregunto cuál es la forma más limpia y eficiente de evitar estas advertencias:

• Fundición $valuesa matriz
• Inicializando $valuesa la matriz
• Envolviendo el foreachcon unif
• Otro (por favor sugiera)

Personalmente, creo que es la más limpia, ¡no estoy seguro de si es la más eficiente!

if (is_array($values) || is_object($values))
{
    foreach ($values as $value)
    {
        ...
    }
}

La razón de mi preferencia es que no asigna una matriz vacía cuando no tienes nada para empezar de todos modos.

¿Que tal este? Lote más limpio y todo en una sola línea.

foreach ((array) $items as $item) {
 // ...
 }

Usualmente uso una construcción similar a esta:

/**
 * 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...
    }
}

Tenga en cuenta que esta versión en particular no se prueba, se escribe directamente en SO desde la memoria.

Editar: agregada verificación Trazable

No dependa de la conversión como una solución , aunque otros sugieran esto como una opción válida para evitar un error, podría causar otra.

Tenga en cuenta: si espera que se devuelva una forma específica de matriz, esto podría fallarle. Se requieren más controles para eso.

Por ejemplo, lanzar un booleano a una matriz (array)bool, NO dará como resultado una matriz vacía, sino una matriz con un elemento que contenga el valor booleano como int: [0=>0]o [0=>1].

Escribí una prueba rápida para presentar este problema . (Aquí hay una prueba de respaldo en caso de que falle la primera URL de prueba).

Se incluyen las pruebas de: null, false, true, un class, una arrayy undefined.

Siempre pruebe su entrada antes de usarla en foreach. Sugerencias:

1. Comprobación rápida de tipos :$array = is_array($var) or is_object($var) ? $var : [] ;
2. Escriba matrices de sugerencias en métodos antes de usar un foreach y especificar tipos de retorno
3. Envolviendo foreach dentro si
4. Usando try{}catch(){}bloques
5. Diseño de código / prueba adecuados antes de los lanzamientos de producción
6. Para probar una matriz con la forma adecuada, puede usar array_key_existsuna clave específica o probar la profundidad de una matriz (¡cuando es una!) .
7. Siempre extraiga sus métodos auxiliares en el espacio de nombres global para reducir el código duplicado

Prueba esto:

//Force array
$dataArr = is_array($dataArr) ? $dataArr : array($dataArr);
foreach ($dataArr as $val) {
  echo $val;
}

;)

$values = get_values();

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

El problema siempre es nulo y, de hecho, la fundición es la solución de limpieza.

En primer lugar, cada variable debe ser inicializada. Siempre.
El casting no es una opción.
si get_values ​​(); puede devolver una variable de tipo diferente, este valor debe ser verificado, por supuesto.

Extensión más concisa del código de @ Kris

function secure_iterable($var)
{
    return is_iterable($var) ? $var : array();
}

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

especialmente para usar código interno de plantilla

<?php foreach (secure_iterable($values) as $value): ?>
    ...
<?php endforeach; ?>

Si está usando php7 y desea manejar solo errores indefinidos, esta es la IMHO más limpia

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

foreach ($arr ? $arr : [] as $elem) {
    // Does something 
}

Esto no comprueba si se trata de una matriz, pero omite el ciclo si la variable es nula o una matriz vacía.

No estoy seguro de si este es el caso, pero este problema parece ocurrir varias veces al migrar sitios de WordPress o migrar sitios dinámicos en general. Si este es el caso, asegúrese de que el alojamiento al que está migrando use la misma versión de PHP que usa su sitio anterior.

Si no está migrando su sitio y esto es solo un problema que ha surgido, intente actualizar a PHP 5. Esto soluciona algunos de estos problemas. Puede parecer una solución tonta, pero me funcionó.

Se produce un caso excepcional para este aviso si configura la matriz como nula dentro del bucle foreach

if (is_array($values))
{
    foreach ($values as $value)
    {
        $values = null;//WARNING!!!
    }
}

¿Qué tal esta solución?

$type = gettype($your_iteratable);
$types = array(
    'array',
    'object'
);

if (in_array($type, $types)) {
    // foreach code comes here
}

Argumento de advertencia no válido proporcionado para foreach()mostrar tweets. ir a /wp-content/plugins/display-tweets-php. Luego inserte este código en la línea número 591, se ejecutará perfectamente.

if (is_array($tweets)) {
    foreach ($tweets as $tweet) 
    {
        ...
    }
}

También parece haber una relación con el medio ambiente:

Tuve ese error de "argumento no válido proporcionado foreach ()" solo en el entorno de desarrollo, pero no en prod (estoy trabajando en el servidor, no localhost).

A pesar del error, var_dump indicó que la matriz estaba bien allí (en ambos casos, app y dev).

El if (is_array($array))alrededor foreach ($array as $subarray)resolvió el problema.

Lo siento, no puedo explicar la causa, pero como me llevó un tiempo encontrar una solución, pensé en compartir esto mejor como una observación.

Use la función is_array, cuando pasará la matriz a foreach loop.

if (is_array($your_variable)) {
  foreach ($your_variable as $item) {
   //your code
}
}

¿Qué pasa con la definición de una matriz vacía como alternativa si get_value()está vacía?
No puedo pensar en un camino más corto.

$values = get_values() ?: [];

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

Usaré una combinación de empty, issety is_arraycomo

$array = ['dog', 'cat', 'lion'];

if (!empty($array) && isset($array) && is_array($array) {
    //loop
    foreach ($array as $values) {
        echo $values; 
    }
}

Haría lo mismo que Andy, pero usaría la función 'vacía'.

al igual que:

if(empty($yourArray))
{echo"<p>There's nothing in the array.....</p>";}
else
{
foreach ($yourArray as $current_array_item)
  {
    //do something with the current array item here
  } 
}