Podano niepoprawny argument dla foreach ()

Często zdarza mi się obsługiwać dane, które mogą być tablicą lub zmienną zerową, i karmić niektóre foreachz tych danych.

$values = get_values();

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

Gdy karmisz foreach danymi, które nie są tablicą, pojawia się ostrzeżenie:

Ostrzeżenie: podano nieprawidłowy argument dla foreach () w [...]

Zakładając, że nie można refaktoryzować get_values()funkcji, aby zawsze zwracała tablicę (kompatybilność wsteczna, niedostępny kod źródłowy, bez względu na inny powód), zastanawiam się, który jest najczystszym i najbardziej wydajnym sposobem uniknięcia tych ostrzeżeń:

• Przesyłanie $valuesdo tablicy
• Inicjowanie $valuesdo tablicy
• Zawijanie foreachza pomocąif
• Inne (proszę zasugerować)

Osobiście uważam, że jest to najczystsze - nie jestem pewien, czy jest najbardziej wydajne, pamiętaj!

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

Powodem dla mnie jest to, że nie przydziela pustej tablicy, kiedy i tak nie masz nic na początek.

Co powiesz na ten? dużo czystsze i wszystko w jednej linii.

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

Zwykle używam konstrukcji podobnej do tej:

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

Zauważ, że ta konkretna wersja nie jest testowana, jest wpisana bezpośrednio do SO z pamięci.

Edycja: dodano czek przejezdny

Nie polegaj na przesyłaniu jako rozwiązania , nawet jeśli inni sugerują to jako prawidłową opcję, aby zapobiec błędowi, może to spowodować inną.

Pamiętaj: jeśli oczekujesz zwrotu określonej formy tablicy, może to Cię nie powieść. Wymagane są w tym celu dodatkowe kontrole.

Np. Rzutowanie wartości logicznej na tablicę NIE(array)bool spowoduje, że tablica będzie pusta, ale tablica z jednym elementem zawierającym wartość logiczną jako int: lub .[0=>0][0=>1]

Napisałem szybki test, aby przedstawić ten problem . (Oto test zapasowy na wypadek, gdyby pierwszy testowy adres URL zawiódł).

Zawarte są testy dla: null, false, true, A class, arrayi undefined.

Zawsze sprawdzaj dane wejściowe przed użyciem ich w foreach. Propozycje:

1. Szybkie sprawdzanie typu :$array = is_array($var) or is_object($var) ? $var : [] ;
2. Wpisz tablice podpowiedzi w metodach przed użyciem foreach i określeniem typów zwracanych
3. Zawijanie foreach wewnątrz if
4. Używanie try{}catch(){}bloków
5. Projektowanie właściwego kodu / testowania przed wydaniami produkcyjnymi
6. Aby przetestować tablicę pod kątem prawidłowej formy, możesz użyć array_key_existsokreślonego klucza lub przetestować głębokość tablicy (gdy jest to jeden!) .
7. Zawsze wyodrębniaj metody pomocnicze do globalnej przestrzeni nazw w taki sposób, aby zredukować zduplikowany kod

Spróbuj tego:

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

;)

$values = get_values();

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

Problem jest zawsze zerowy, a Casting jest w rzeczywistości rozwiązaniem czyszczącym.

Przede wszystkim należy zainicjować każdą zmienną. Zawsze.
Przesyłanie nie jest opcją.
if get_values ​​(); może zwrócić zmienną innego typu, tę wartość należy oczywiście sprawdzić.

Bardziej zwięzłe rozszerzenie kodu @ Kris

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

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

szczególnie do używania wewnątrz kodu szablonu

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

Jeśli używasz php7 i chcesz obsłużyć tylko niezdefiniowane błędy, jest to najczystszy IMHO

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

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

To nie sprawdza, czy jest to tablica, ale pomija pętlę, jeśli zmienna jest pusta lub pusta.

Nie jestem pewien, czy tak jest, ale ten problem wydaje się występować wiele razy podczas migracji witryn Wordpress lub ogólnie migracji witryn dynamicznych. W takim przypadku upewnij się, że hosting, do którego przeprowadzasz migrację, używa tej samej wersji PHP, której używa stara witryna.

Jeśli nie przeprowadzasz migracji swojej witryny, a to jest tylko problem, który się pojawił, spróbuj zaktualizować do wersji PHP 5. To rozwiązuje niektóre z tych problemów. Może to wydawać się głupim rozwiązaniem, ale załatwiło sprawę.

Wyjątkowy przypadek tego zawiadomienia występuje, jeśli ustawisz tablicę na zero w pętli foreach

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

A co z tym rozwiązaniem:

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

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

Podano niepoprawny argument ostrzeżenia dla foreach()wyświetlanych tweetów. idź do /wp-content/plugins/display-tweets-php. Następnie wstaw ten kod do linii numer 591, będzie działał idealnie.

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

Wydaje się również, że istnieje związek ze środowiskiem:

Miałem ten błąd „dostarczono nieprawidłowy argument foreach ()” tylko w środowisku deweloperskim, ale nie w prod (pracuję na serwerze, a nie localhost).

Pomimo błędu zmienna_dump wskazała, że ​​tablica jest w porządku (w obu przypadkach app i dev).

if (is_array($array))Wokół foreach ($array as $subarray)rozwiązało problem.

Przepraszam, że nie potrafię wyjaśnić przyczyny, ale ponieważ znalezienie rozwiązania zajęło mi trochę czasu, pomyślałem o lepszym udostępnieniu tego jako spostrzeżenia.

Użyj funkcji is_array, gdy przekażesz tablicę do pętli foreach.

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

Co powiesz na zdefiniowanie pustej tablicy jako zastępczej, jeśli get_value()jest pusta?
Nie mogę wymyślić najkrótszej drogi.

$values = get_values() ?: [];

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

Użyję kombinacji empty, isseta is_arrayjako

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

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

zrobiłbym to samo co Andy, ale używałbym funkcji „pustej”.

tak:

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