PHPのループ内の条件分岐による配列の比較

Question

ループ内の条件分岐で一致した条件の項目と一致しなかった項目を分けたくて次の様にしています。
しかし、求める表示結果とは異なってしまいます。
ループをネストしているからだとは思うのですが、どうなっているのか理屈がいまいちよく分からず…。
よろしければ求める表示結果を得る方法と、下記コードがどう間違えているのかの解説を頂けないでしょうか…。
尚、どちらも比較しているのは文字列です。

foreach ( $items as $item ) { // 連想配列を持つオブジェクトが複数格納された配列
    foreach ( $key as $value ) { // 連想配列
        if ( $item -> name == $value ) { // name と $value が一致した時は次を表示
            echo '<li>Success: ' . $item -> name . '</li>';
        } else { // 一致しない時は次を表示
            echo '<li>Fail: ' . $item -> name . '</li>';
        }
    }
}

求める表示結果

• Success: A
• Success: B
• Fail: C

実際の表示結果

• Success: A
• Fail: A
• Fail: B
• Success: B
• Fail: C
• Fail: C

ちなみに次の様にした時は意図した通りの結果になりますが、複数の値に対応出来ません

foreach ( $items as $item ) { // 連想配列を持つオブジェクトが複数格納された配列
    if ( $item -> name == $key[0] ) { // name と $value が一致した時は次を表示
        echo '<li>Success: ' . $item -> name . '</li>';
    } else { // 一致しない時は次を表示
        echo '<li>Fail: ' . $item -> name . '</li>';
    }
}

表示結果

• Success: A // 最初に一致した項目だけSuccess
• Fail: B
• Fail: C

Answer 1

単純に下記のようにしてはだめなんでしょうか

foreach($items as $item){
    if(in_array($item->name, $key)){
        echo '<li>Success: ' . $item->name . '</li>' . "\n";
    }else{
        echo '<li>Fail: ' . $item->name . '</li>' . "\n";
    }
}

$keyの要素数が多ければ下記

$key = array_flip($key);
foreach($items as $item){
    if(isset($key[$item->name])){
        echo '<li>Success: ' . $item->name . '</li>' . "\n";
    }else{
        echo '<li>Fail: ' . $item->name . '</li>' . "\n";
    }
}

Comments

[flat]

回答して下さりありがとうございます。
どちらもシンプルで分かりやすいですね。
ちなみに後者は「配列に値があるかどうか」をチェックするよりも「反転した$keyの配列にあるキーに同名のnameがあるかどうか」でチェックする方が要素が多い時には速いという理解で良いのでしょうか？

[yokoyam49]

はい。
in_arrayは要素を走査していくので、要素数が多い場合は遅くなっていきます。
キーと値を反転してダイレクトにissetで存在するか確認したほうが速い…はずです。
大量のデータから判定する、もしくは判定する可能性がある場合はissetの方がよいと思います。

[flat]

お返事が遅くなりました…。
こちらの質問にも答えて下さりありがとうございます。
大事な知識の一つとして頭に入れておきます。

Answer 2

データが何なのか不明なのでよくわかりませんが、こんな感じでしょうか？

foreach ( $items as $item ) { // 連想配列を持つオブジェクトが複数格納された配列
    $flat = false;
    foreach ( $key as $value ) { // 連想配列
        if ( $item -> name === $value ) {
            $flag = true;
            break;
        }
    }
    if ( $flag ) { // name と $value が一致した時は次を表示
        echo '<li>Success: ' . $item -> name . '</li>';
    } else { // 一致しない時は次を表示
        echo '<li>Fail: ' . $item -> name . '</li>';
    }
}

Comments

[flat]

その様に処理を振り分けたら良いのですね。
試してみたところ求めた通りの結果が得られました。
ありがとうございます。