Неправильний аргумент, наданий для 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. Загортання передпліччя в межах, якщо
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){
  ...
}

Проблема завжди є нульовою, а лиття - насправді очищенням.

Перш за все, кожна змінна повинна бути ініціалізована. Завжди.
Кастинг - це не варіант.
якщо 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 і хочете обробляти лише невизначені помилки, це найчистіший IMHO

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