Argument non valide pour la fonction foreach()

Il m'arrive souvent de gérer des données qui peuvent être un tableau ou une variable nulle et d'en nourrir certaines foreachavec ces données.

$values = get_values();

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

Lorsque vous alimentez un foreach avec des données qui ne sont pas un tableau, vous obtenez un avertissement:

Avertissement: Argument non valide fourni pour foreach () dans [...]

En supposant qu'il n'est pas possible de refactoriser la get_values()fonction pour toujours renvoyer un tableau (compatibilité descendante, code source non disponible, quelle que soit la raison), je me demande quelle est la manière la plus propre et la plus efficace d'éviter ces avertissements:

• Casting $valuesdans un tableau
• Initialisation $valuesau tableau
• Envelopper le foreachavec unif
• Autre (veuillez suggérer)

Personnellement, je trouve que c'est le plus propre - je ne sais pas si c'est le plus efficace, l'esprit!

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

La raison de ma préférence est qu'il n'alloue pas de tableau vide quand vous n'avez rien pour commencer de toute façon.

Celui-ci, ça va? beaucoup plus propre et tout en une seule ligne.

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

J'utilise généralement une construction similaire à celle-ci:

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

Notez que cette version particulière n'est pas testée, elle est tapée directement dans SO depuis la mémoire.

Edit: ajout de la vérification Traversable

Veuillez ne pas dépendre de la diffusion comme solution , même si d'autres suggèrent cela comme une option valide pour éviter une erreur, cela pourrait en provoquer une autre.

Attention: Si vous vous attendez à ce qu'une forme spécifique de tableau soit renvoyée, cela peut vous échouer. Plus de contrôles sont nécessaires pour cela.

Par exemple, la conversion d'un booléen en un tableau (array)booln'entraînera PAS un tableau vide, mais un tableau avec un élément contenant la valeur booléenne sous la forme d'un int: [0=>0]ou [0=>1].

J'ai écrit un test rapide pour présenter ce problème . (Voici un test de sauvegarde au cas où la première URL de test échouerait.)

Inclus sont des tests pour: null, false, true, un class, un arrayet undefined.

Testez toujours votre entrée avant de l'utiliser dans foreach. Suggestions:

1. Vérification rapide du type :$array = is_array($var) or is_object($var) ? $var : [] ;
2. Tapez les tableaux d'indices dans les méthodes avant d'utiliser un foreach et de spécifier les types de retour
3. Enveloppant foreach dedans si
4. Utilisation de try{}catch(){}blocs
5. Concevoir le bon code / tester avant la sortie de la production
6. Pour tester un tableau par rapport à la forme appropriée, vous pouvez utiliser array_key_existsune clé spécifique ou tester la profondeur d'un tableau (quand il en est un!) .
7. Extrayez toujours vos méthodes d'assistance dans l'espace de noms global de manière à réduire le code en double

Essaye ça:

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

;)

$values = get_values();

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

Le problème est toujours nul et Casting est en fait la solution de nettoyage.

Tout d'abord, chaque variable doit être initialisée. Toujours.
Le casting n'est pas une option.
if get_values ​​(); peut renvoyer une variable de type différent, cette valeur doit être vérifiée, bien sûr.

Extension plus concise du code de @ Kris

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

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

en particulier pour utiliser le code du modèle à l'intérieur

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

Si vous utilisez php7 et que vous souhaitez gérer uniquement les erreurs non définies, c'est le IMHO le plus propre

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

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

Cela ne vérifie pas s'il s'agit d'un tableau, mais ignore la boucle si la variable est nulle ou un tableau vide.

Je ne sais pas si c'est le cas, mais ce problème semble se produire plusieurs fois lors de la migration de sites wordpress ou de la migration de sites dynamiques en général. Si tel est le cas, assurez-vous que l'hébergement vers lequel vous migrez utilise la même version PHP que votre ancien site utilise.

Si vous ne migrez pas votre site et que c'est juste un problème qui s'est posé, essayez de mettre à jour vers PHP 5. Cela résout certains de ces problèmes. Cela peut sembler une solution idiote mais a fait l'affaire pour moi.

Un cas exceptionnel pour cet avis se produit si vous définissez le tableau sur null dans la boucle foreach

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

Que diriez-vous de cette solution:

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

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

Avertissement argument non valide fourni pour l' foreach()affichage des tweets. aller à /wp-content/plugins/display-tweets-php. Insérez ensuite ce code sur la ligne numéro 591, il fonctionnera parfaitement.

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

Il semble également y avoir une relation avec l'environnement:

J'ai eu cette erreur "argument non valide fourni poureach ()" uniquement dans l'environnement de développement, mais pas dans prod (je travaille sur le serveur, pas sur localhost).

Malgré l'erreur, un var_dump a indiqué que le tableau était bien là (dans les deux cas, app et dev).

Le if (is_array($array))tour du a foreach ($array as $subarray)résolu le problème.

Désolé, je ne peux pas expliquer la cause, mais comme il m'a fallu un certain temps pour trouver une solution, j'ai pensé à mieux partager cela comme une observation.

Utilisez la fonction is_array, lorsque vous passerez un tableau à une boucle foreach.

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

Qu'en est-il de la définition d'un tableau vide comme repli s'il get_value()est vide?
Je ne peux pas penser à un moyen le plus court.

$values = get_values() ?: [];

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

Je vais utiliser une combinaison de empty, issetet is_arraycomme

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

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

je ferais la même chose qu'Andy mais j'utiliserais la fonction 'vide'.

ainsi:

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
  } 
}