echoの出力を一つにしたい場合

Question

前提・実現したいこと

##出力結果を"nice"だけにしたい場合

該当のソースコード

##問題配列の中に1があれば、niceだけを出力し、無い場合はbadを出力しろ

$a=[3,5,1,7];

foreach($a as $b) {
if($b==1){
echo "nice ";
}else {
echo "bad ";
}
}
出力結果
bad bad nice bad

と出力されてしまいます

試したこと

補足情報（FW/ツールのバージョンなど）

Comments

[m.ts10806]

コードはマークダウンの機能を利用してご提示ください。
また、「試したこと」が「ボールドテキスト」って何でしょう。
質問は編集できますので適宜ご対応ください。

[m.ts10806]

あと「出力されるなら」を何を以て判断されますか？
「そこまでは出力してそれ以降出力しない」のか「そこだけ出力する」のかでは違います。
もっとケースパターンを出してください。

[m.ts10806]

コードはマークダウンの機能を利用してご提示ください。
https://teratail.com/questions/238564

Answer 1

解決済みですが、参考までに

$a = [3,5,1,7];
$c = array();
foreach ($a as $b) $c[$b] = $b;
if (isset($c[1])) {
    echo 'nice';
} else {
    echo 'bad';
}

Answer 2

何かの課題っぽいので解説はしませんし、そのまま提出してここで質問しているのがバレて怒られても誰も責任取りません。
1つ言えるのは既に指摘されているようにforeachは悪手です。
かなり冗長な記述になるからです。

<?php
function valid($array){
    foreach($array as $n){
        if($n === 1){
            return true;
        }
    }
    return false;
}

$a=[3,5,1,7]; 
echo valid($a)?'nice':'bad';

Comments

[m.ts10806]

大喜利ですねぇ。

Answer 3

配列に1があるかないかということを判断するだけであれば、in_array関数で判断すればいいかと思います

https://www.php.net/manual/ja/function.in-array.php

Comments

[oitama]

in_array を使わずに私が書いたコードをベースにはできませんか？

[akkman]

であれば一つのやり方として、

$cnt=0;
foreach($a as $b) {
　if($b==1){
$cnt++;
　}
}
if($cnt==0){
echo "bad";
}else{
echo "nice";
}

というやり方もあります

[oitama]

ありがとうございます。
なのですがそのコードでやってみたのですが、
if($b==1){
$cnt++;
}
のところでsyntax error unexcepeted " ; "とエラーが起きます。

[oitama]

大変すみません。自分の誤字だったみたいです。
お騒がせしてすみませんでした。