Указан неверный аргумент для foreach ()

Мне часто приходится обрабатывать данные, которые могут быть массивом или нулевой переменной, и снабжать их foreachэтими данными.

$values = get_values();

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

Когда вы передаете foreach данными, которые не являются массивом, вы получаете предупреждение:

Предупреждение: неверный аргумент указан для foreach () в [...]

Предполагая, что невозможно реорганизовать get_values()функцию так, чтобы она всегда возвращала массив (обратная совместимость, недоступный исходный код по любой другой причине), мне интересно, какой самый чистый и эффективный способ избежать этих предупреждений:

• Кастинг $valuesв массив
• Инициализация $valuesв массив
• Заворачивать foreachсif
• Другое (пожалуйста, предложите)

Лично я считаю, что это самый чистый - не уверен, что это самый эффективный, ум!

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

Причиной моего предпочтения является то, что он не выделяет пустой массив, когда у вас все равно ничего не начинается.

Как насчет этого? много чище и все в одну строчку.

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

Я обычно использую конструкцию, подобную этой:

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

Обратите внимание, что эта конкретная версия не тестировалась, она набирается непосредственно в SO из памяти.

Редактировать: добавлен обходной чек

Пожалуйста, не полагайтесь на приведение в качестве решения , даже если другие предлагают этот вариант как допустимый, чтобы предотвратить ошибку, она может вызвать другую.

Помните: если вы ожидаете, что будет возвращена конкретная форма массива, это может вас не устроить. Для этого требуются дополнительные проверки.

Например , литье булева в массив (array)bool, будет НЕ привести к пустому массиву, а массив с одним элементом , содержащего логическое значение , как INT: [0=>0]или [0=>1].

Я написал быстрый тест, чтобы представить эту проблему . (Вот резервный тест на случай, если первый тестовый URL не пройден.)

Включены тесты для: null, false, true, A class, arrayи undefined.

Всегда проверяйте ввод перед использованием в foreach. Предложения:

1. Быстрая проверка типов :$array = is_array($var) or is_object($var) ? $var : [] ;
2. Напечатайте массивы хинтинга в методах перед использованием foreach и указанием возвращаемых типов
3. Обтекание foreach внутри, если
4. Использование try{}catch(){}блоков
5. Разработка правильного кода / тестирование перед выпуском продукции
6. Чтобы проверить массив на предмет правильной формы, вы можете использовать array_key_existsопределенный ключ или проверить глубину массива (если он один!) .
7. Всегда извлекайте ваши вспомогательные методы в глобальное пространство имен, чтобы уменьшить количество дублирующегося кода.

Попробуй это:

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

;)

$values = get_values();

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

Проблема всегда нулевая, и на самом деле Casting является чистящим решением.

Прежде всего, каждая переменная должна быть инициализирована. Всегда.
Кастинг не вариант.
if get_values ​​(); может возвращать переменную другого типа, это значение, конечно, нужно проверить.

Более краткое расширение кода @ Kris

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

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

особенно для использования внутри кода шаблона

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

Если вы используете php7 и хотите обрабатывать только неопределенные ошибки, это самое чистое ИМХО

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

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

Это не проверяет, является ли это массивом, но пропускает цикл, если переменная равна нулю или пустой массив.

Я не уверен, так ли это, но эта проблема, по-видимому, возникала несколько раз при миграции WordPress сайтов или вообще динамических сайтов. В этом случае убедитесь, что хостинг, на который вы переходите, использует ту же версию PHP, что и ваш старый сайт.

Если вы не мигрируете свой сайт, и это просто проблема, попробуйте обновить до PHP 5. Это решит некоторые из этих проблем. Может показаться глупым решением, но сделал свое дело для меня.

Исключительный случай для этого уведомления происходит, если вы устанавливаете массив в нуль внутри цикла foreach

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

Как насчет этого решения:

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

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

Предупреждение неверный аргумент предоставлен для foreach()отображения твитов. перейти к /wp-content/plugins/display-tweets-php. Затем вставьте этот код в строку номер 591, он будет работать отлично.

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

Кажется также, что есть отношение к окружающей среде:

У меня была ошибка «неверный аргумент предоставлен foreach ()» только в среде разработчика, но не в prod (я работаю на сервере, а не на localhost).

Несмотря на ошибку, var_dump показал, что с массивом все в порядке (в обоих случаях app и dev).

Во if (is_array($array))всем foreach ($array as $subarray)решена проблема.

Извините, что я не могу объяснить причину, но, поскольку мне потребовалось некоторое время, чтобы найти решение, я подумал о том, чтобы лучше поделиться этим в качестве наблюдения.

Используйте функцию is_array, когда вы передадите массив в цикл foreach.

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

Как насчет определения пустого массива как запасного, если get_value()он пуст?
Я не могу придумать кратчайшего пути.

$values = get_values() ?: [];

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

Я буду использовать комбинацию empty, issetи is_arrayкак

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

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

я бы сделал то же самое, что и Энди, но я бы использовал «пустую» функцию.

вот так:

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