foreach（）に無効な引数が指定されました

配列またはnull変数のいずれかである可能性のあるデータを処理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]またはとして含む1つの要素を持つ配列になり[0=>1]ます。

私はこの問題を提示する簡単なテストを書きました。（これは、最初のテストURLが失敗した場合のバックアップテストです。）

：のためのテストです含めnull、false、true、class、arrayとはundefined。

foreachで使用する前に、必ず入力をテストしてください。提案：

1. クイックタイプチェック：$array = is_array($var) or is_object($var) ? $var : [] ;
2. foreachを使用して戻り値の型を指定する前に、メソッドにヒント配列を入力する
3. if内でforeachをラップする
4. try{}catch(){}ブロックの使用
5. 適切なコードの設計/製品版リリース前のテスト
6. 正しい形式に対して配列をテストするarray_key_existsには、特定のキーで使用するか、配列の深さをテストします（それが1の場合！）。
7. 重複するコードを減らす方法で、常にヘルパーメソッドをグローバル名前空間に抽出します。

これを試して：

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

;）

$values = get_values();

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

問題は常にnullであり、キャスティングは実際にはクリーニングソリューションです。

まず、すべての変数を初期化する必要があります。常に。
キャストはオプションではありません。
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を使用していて、未定義のエラーのみを処理したい場合、これは最もクリーンなIMHOです。

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

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

これは配列かどうかはチェックしませんが、変数がnullまたは空の配列の場合はループをスキップします。

これが当てはまるかどうかはわかりませんが、ワードプレスサイトや一般的な動的サイトを移行するときに、この問題が何度も発生するようです。この場合は、移行先のホスティングで、古いサイトが使用しているものと同じバージョンのPHPを使用していることを確認してください。

サイトを移行しておらず、これが問題となっているだけの場合は、PHP 5に更新してみてください。これにより、これらの問題のいくつかが処理されます。ばかげた解決策のように見えるかもしれませんが、私のためにトリックをしました。

foreachループ内で配列をnullに設定すると、この通知の例外的なケースが発生します

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)問題を解決しました。

申し訳ありませんが、原因を説明することはできませんが、解決策を見つけるのにしばらく時間がかかったので、これを観察としてよりよく共有することを考えました。

foreachループに配列を渡す場合は、is_array関数を使用します。

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