Ungültiges Argument für foreach ()

Es passiert mir oft, Daten zu verarbeiten, die entweder ein Array oder eine Nullvariable sein können, und einige foreachmit diesen Daten zu versorgen.

$values = get_values();

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

Wenn Sie einen foreach mit Daten füttern, die kein Array sind, erhalten Sie eine Warnung:

Warnung: Ungültiges Argument für foreach () in [...]

Angenommen, es ist nicht möglich, die get_values()Funktion so umzugestalten , dass immer ein Array zurückgegeben wird (Abwärtskompatibilität, nicht verfügbarer Quellcode, aus welchem ​​anderen Grund auch immer), frage ich mich, welcher Weg am saubersten und effizientesten ist, um diese Warnungen zu vermeiden:

• Casting $valueszum Array
• Initialisierung $valuesauf Array
• Wickeln Sie die foreachmit einemif
• Andere (bitte vorschlagen)

Persönlich finde ich das am saubersten - ich bin mir nicht sicher, ob es am effizientesten ist!

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

Der Grund für meine Präferenz ist, dass kein leeres Array zugewiesen wird, wenn Sie sowieso nichts zu Beginn haben.

Wie wäre es mit diesem? viel sauberer und alles in einer Zeile.

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

Normalerweise verwende ich ein ähnliches Konstrukt:

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

Beachten Sie, dass diese bestimmte Version nicht getestet wird, sondern direkt aus dem Speicher in SO eingegeben wird.

Bearbeiten: Traversable Check hinzugefügt

Bitte verlassen Sie sich nicht auf das Casting als Lösung , auch wenn andere dies als gültige Option vorschlagen, um einen Fehler zu vermeiden, kann dies zu einem anderen führen.

Beachten Sie: Wenn Sie erwarten, dass eine bestimmte Form von Array zurückgegeben wird, schlägt dies möglicherweise fehl. Dafür sind weitere Prüfungen erforderlich.

Das Umwandeln eines Booleschen Werts in ein Array (array)boolführt beispielsweise NICHT zu einem leeren Array, sondern zu einem Array mit einem Element, das den Booleschen Wert als int: [0=>0]oder enthält [0=>1].

Ich habe einen kurzen Test geschrieben, um dieses Problem darzustellen . (Hier ist ein Backup-Test für den Fall, dass die erste Test-URL fehlschlägt.)

Enthalten sind Tests für: null, false, true, ein class, ein arrayund undefined.

Testen Sie Ihre Eingabe immer, bevor Sie sie in foreach verwenden. Vorschläge:

1. Schnelle Typprüfung :$array = is_array($var) or is_object($var) ? $var : [] ;
2. Geben Sie Hinweis-Arrays in Methoden ein, bevor Sie foreach verwenden und Rückgabetypen angeben
3. Wrap foreach innerhalb von if
4. Mit try{}catch(){}Blöcken
5. Entwerfen des richtigen Codes / Testens vor Produktionsfreigaben
6. Um ein Array gegen die richtige Form zu testen, können Sie es array_key_existsfür einen bestimmten Schlüssel verwenden oder die Tiefe eines Arrays testen (wenn es einer ist!) .
7. Extrahieren Sie Ihre Hilfsmethoden immer in den globalen Namespace, um doppelten Code zu reduzieren

Versuche dies:

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

;)

$values = get_values();

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

Das Problem ist immer null und Casting ist in der Tat die Reinigungslösung.

Zunächst muss jede Variable initialisiert werden. Immer.
Casting ist keine Option.
if get_values ​​(); kann eine andere Typvariable zurückgeben, dieser Wert muss natürlich überprüft werden.

Prägnantere Erweiterung des Codes von @ Kris

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

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

speziell für die Verwendung von Inside Template Code

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

Wenn Sie PHP7 verwenden und nur undefinierte Fehler behandeln möchten, ist dies meiner Meinung nach das sauberste

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

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

Dies prüft nicht, ob es sich um ein Array handelt, sondern überspringt die Schleife, wenn die Variable null oder ein leeres Array ist.

Ich bin nicht sicher, ob dies der Fall ist, aber dieses Problem scheint bei der Migration von WordPress-Sites oder der Migration dynamischer Sites im Allgemeinen mehrmals aufzutreten. Wenn dies der Fall ist, stellen Sie sicher, dass das Hosting, auf das Sie migrieren, dieselbe PHP-Version verwendet, die Ihre alte Site verwendet.

Wenn Sie Ihre Site nicht migrieren und dies nur ein aufgetretenes Problem ist, versuchen Sie, ein Update auf PHP 5 durchzuführen. Dadurch werden einige dieser Probleme behoben. Könnte wie eine dumme Lösung erscheinen, hat aber den Trick für mich getan.

Ein Ausnahmefall für diesen Hinweis tritt auf, wenn Sie das Array in jeder for-Schleife auf null setzen

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

Wie wäre es mit dieser Lösung:

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

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

Warnung ungültiges Argument für foreach()Anzeige-Tweets angegeben. geh zu /wp-content/plugins/display-tweets-php. Fügen Sie dann diesen Code in Zeile 591 ein. Er läuft einwandfrei.

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

Es scheint auch eine Beziehung zur Umwelt zu geben:

Ich hatte diesen Fehler "ungültiges Argument für foreach () angegeben" nur in der Entwicklungsumgebung, aber nicht in prod (ich arbeite auf dem Server, nicht auf localhost).

Trotz des Fehlers zeigte ein var_dump an, dass das Array gut vorhanden war (in beiden Fällen app und dev).

Das if (is_array($array))rund um foreach ($array as $subarray)das Problem gelöst.

Es tut mir leid, dass ich die Ursache nicht erklären kann, aber da ich eine Weile gebraucht habe, um eine Lösung zu finden, dachte ich daran, dies besser als Beobachtung zu teilen.

Verwenden Sie die Funktion is_array, wenn Sie das Array an jede Schleife übergeben.

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

Was ist mit der Definition eines leeren Arrays als Fallback, wenn get_value()es leer ist?
Ich kann mir keinen kürzesten Weg vorstellen.

$values = get_values() ?: [];

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

Ich werde eine Kombination verwenden empty, issetund is_arrayals

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

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

Ich würde das gleiche tun wie Andy, aber ich würde die 'leere' Funktion verwenden.

wie so:

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