le moyen le plus propre de sauter un foreach si le tableau est vide

Question

Ce n’est pas un problème majeur, mais je me demandais s’il existait une méthode plus propre. Il serait bon d'éviter d'imbriquer mon code avec une instruction if inutile. Si $items est vide, php génère une erreur.

 $items = array('a','b','c');

if(!empty($items)) { // <-Remove this if statement
  foreach($items as $item) {
    print $item;
  }
}
 

Je pourrais probablement simplement utiliser le suppresseur d'erreur '@', mais ce serait un peu hacky.

Answer 1

Il y a un million de façons de le faire.

La première serait d’exécuter le tableau dans foreach de toute façon, en supposant que vous en ayez un.

Dans d'autres cas, c'est ce dont vous pourriez avoir besoin:

   foreach((array)$items as $item) {
    print $item;
  }
 

Remarque: Si vous convertissez une valeur dans un tableau, vous obtenez un tableau, quelle que soit sa valeur. Par exemple, null, true, false donnent tous un tableau vide.

Answer 2

Le meilleur moyen consiste à initialiser chaque variable sanglante avant utilisation.
Cela ne résoudra pas seulement ce "problème" idiot, mais vous épargnera également une tonne de vrais maux de tête.

Donc, introduire les éléments $ en tant que $items = array(); est ce que vous vouliez vraiment .

Answer 3

$items = array('a','b','c');

if(is_array($items))) {
  foreach($items as $item) {
    print $item;
  }
}

Answer 4

Je ne recommanderais pas supprimer la sortie d’avertissement. J’ai, cependant, conseille d’utiliser is_array au lieu de ! vide. Si $items se trouve être un scalaire non nul, alors la boucle foreach sera toujours erreur si vous utilisez ! vide.

Answer 5

Je pense que la meilleure approche ici est de planifier votre code afin que $items est toujours un tableau. La solution la plus simple consiste à initialiser en haut de votre code avec $items=array(). De cette façon qu'il représentera un tableau vide même si vous n’y attribuer aucune valeur.

Toutes les autres solutions sont assez sales hacks pour moi.