manera más limpia de omitir un foreach si el conjunto está vacío

Question

No es un problema importante, pero me preguntaba si existe una forma más clara de hacerlo. Sería bueno evitar anidar mi código con una declaración if innecesaria. Si $items está vacío, php arroja un error.

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

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

Probablemente podría simplemente usar el '@' supresor de errores, pero sería un poco hacky.

Answer 1

Hay un millón de formas de hacerlo.

La primera sería seguir adelante y ejecutar la matriz a través de foreach de todos modos, suponiendo que usted tiene una matriz.

En otros casos se trata de lo que pueda necesitar:

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

Nota: a todas las personas que se quejan de encasillado, tenga en cuenta que el PO pidió forma más limpia de saltarse un foreach si el array es vacío (el énfasis es mío). Un valor de verdadero, falso, números o cadenas es no se considera vacío.

Answer 2

No recomendaría suprimir la salida de advertencia. Sin embargo, recomendaría usar is_array en lugar de !empty. Si $items pasa a ser un escalar distinto de cero, entonces el foreach seguirá produciendo un error si usa !empty.

Answer 3

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

Answer 4

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

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

Answer 5

Creo que el mejor enfoque aquí es planificar su código para que $ items sea siempre una matriz. La solución más fácil es inicializarla en la parte superior de su código con $ items = array(). De esta manera, representará una matriz vacía incluso si no le asigna ningún valor.

Todas las demás soluciones son bastante sucias para mí.

Answer 6

que tengo la siguiente función en mi "biblioteca estándar"

/// Convert argument to an array. 
function a($a = null) { 
    if(is_null($a)) 
     return array(); 
    if(is_array($a)) 
     return $a; 
    if(is_object($a)) 
     return (array) $a; 
    return $_ = func_get_args(); 
} 

Básicamente, esto no hace nada con matrices/objetos y convertirlos en otros tipos de matrices. Esto es extremadamente útil para usar con las declaraciones foreach y funciones de matriz

foreach(a($whatever) as $item).... 

    $foo = array_map(a($array_or_string).... 

    etc 

Answer 7

La mejor forma de hacerlo es inicializar todas las variables sangrienta antes de su uso.
No solo resolverá este tonto "problema" sino que también le ahorrará una tonelada de dolores de cabeza reales.

Por lo tanto, la introducción de $ artículos como $items = array(); es lo que realmente quería.

Answer 8

La lógica ternaria lo lleva a una línea sin errores. Esto resuelve el problema de las variables indebidamente emitidas y las variables indefinidas.

foreach (is_array($Items) || is_object($Items) ? $Items : array() as $Item) { 

Escribir un comentario es un poco difícil, pero es la manera más segura de manejarlo.

Answer 9

La mejor práctica es definir la variable como una matriz en la parte superior de su código.

foreach ((array) $ myArr como $ oneItem) {..}

también funcionará pero se duplicará esto cada vez (array) de conversión que necesita para recorrer la matriz.

dado que es importante no duplicar ni siquiera una palabra de tu código, será mejor que lo definas como una matriz vacía en la parte superior.

Answer 10

Puede comprobar si $ artículos es en realidad una matriz y si contiene ningún artículo en:

if(is_array($items) && count($items) > 0) 
{ 
    foreach($items as $item) { } 
} 

Answer 11

Si la variable que necesita podría ser boolean false - por ejemplo. cuando no hay registros se devuelven desde la base de datos o array - cuando los registros se devuelven, puede hacer lo siguiente:

foreach (($result ? $result : array()) as $item) 
    echo $item; 

enfoque con el molde ((Array)$result) produce una serie de recuento 1 cuando la variable es boolean false, que ISN' t lo que probablemente quieras