PHPのif文の書き方について

Question

初歩的な質問ですみません。

<?php
$r1 = [ここは 1 か ""];
$r2 = [ここは 1 か ""]; 
        if($r1=='1' and $r2=='1'){
          echo "<!--";
        }elseif($r1=='1' or $r2=='1'){
          echo "<!--";
        }else{
          echo "";
        } 
    ?>
    <h1>aaa</h1>
    <?php 
      if($r1=='1' and $r2=='1'){
        echo "-->";
      }elseif($r1=='1' or $r2=='1'){
        echo "-->";
      }else{echo "";} 
    ?>

<?php 
if($r1!=='1' and $r2!=='1'){
echo "<!--";
}elseif($r1!=='1' or $r2!=='1'){
echo "<!--";
}else{
echo "";
} ?>
<h1>bbb</h1>
<?php 
if($r1!=='1' and $r2!=='1'){
echo "-->";
}elseif($r1!=='1' or $r2!=='1'){
echo "-->";
}else{echo "";} 
?>

$r1 か $r2　に1が入っているときまたは両方に1が入ってるときbbbを表示させる。
それ以外はaaa表示させたいのですが、

これだと$t1か$r2どちらだけに1が入っていた場合aaaもbbbどちらも表示されません。
どこが違うのでしょうか。
というか、すごく効率の悪い書き方をしている気がするのですが…
ご教授よろしくお願いいたします。

Answer 1

すでに答え自体は、他の方に回答されてしまったので、考え方を言いますね。
IF文の分岐が複雑になるのは、頭の中で仕様とコードが一致してないからです。

まず、図解すると分かりやすくなるので、慣れるまでは面倒でも、
フローチャートを書くのは有効です。

あるいは、表を書いて、変数の内容と結果を対応させると、重複に気づきます。

表示	$r1	$r2
aaa	0	0
bbb	1	0
bbb	0	1
bbb	1	1

ここから、一行のORを使ったIF文だけで済むことが分かります。
（もしくはメソッドからANDでリターンする）

---

$r1 か $r2　に1が入っているときまたは両方に1が入ってるときbbbを表示させる。
それ以外はaaa表示させたい

直接読み下す方法も教えましょう。
上の仕様を、擬似コードで段階的詳細化していきます。

if($r1 か $r2　に1が入っているときまたは両方に1が入ってるとき){bbbを表示}
else{aaa表示}

if($r1 か $r2　に1が入っているとき || 両方に1が入ってるとき){bbbを表示}
else{aaa表示}

if(($r1 == 1) || ($r2 == 1) || ($r1 == 1) && ($r2 == 1)){bbbを表示}
else{aaa表示}

if((($r1 == 1) || ($r2 == 1)) || ($r1 == 1) && ($r2 == 1)){echo "<h1>bbb</h1>";}
else{echo "<h1>aaa</h1>";}

if((($r1 == 1) || ($r2 == 1)) || ($r1 == 1) && ($r2 == 1)){
echo "<h1>bbb</h1>";}
} else {
echo "<h1>aaa</h1>";}

if(($r1 == 1) || ($r2 == 1)){
echo "<h1>bbb</h1>";}
elseif(($r1 == 1) && ($r2 == 1)){
echo "<h1>bbb</h1>";}
} else {
echo "<h1>aaa</h1>";}

if(($r1 == 1) || ($r2 == 1)){
echo "<h1>bbb</h1>";}
} else {
echo "<h1>aaa</h1>";}

慣れて来れば、ステップを飛ばして一気に理解できます。
英文の翻訳とかと同じことで、普段はこれくらいなら無意識で読み書きします。

でも最初のうちは逐次で追いましょう。上は冗長ですがわざとやっています。
最後の方でif文が長いのでリファクタリングして、
二行に分けた結果、条件の重複に気づいたので、
さらにリファクタリングする、という過程を再現しています。

仕様とコードを近づけていく呼吸です。
両者をバラバラにして考えてしまうと、
ご質問のサンプルコードのように、カオスになっていきます。

仕様を忘れてコードだけこねくり回してはダメで、
面倒なようでもステップを踏んでください。「急がば回れ」です。

Answer 2

大分悩んだコードですね。こういう悩み方、好きです！

やりたいことを簡潔に表現すると、以下のようになります。

<?php
$r1 = 1;
$r2 = 1;
?>

<?php if($r1 === 1 || $r2 === 1):?>
<h1>bbb</h1>
<?php else: ?>
<h1>aaa</h1>
<?php endif; ?>

$r1 か $r2　に 1 が入っているときまたは両方に1が入ってる

両方にという条件は、どちらかが 1 であれば満たしているので省略可能です。
つまり

$r1 か $r2　に 1 が入っている

に包含されます。

その為、どちらかが 1 であるかどうかを確認する if 文を書きます。
で、条件に合致すれば、<h1>bbb</h1>をそうでなければ<h1>aaa</h1>を表示します。

Comments

[MeB]

ありがとうございます！…アホらしくなってきましたw
同様に$r1、$r2が3,4...$r7まであり
<h1>ccc</h1>があった場合も同様に
足して行く感じでよろしいのでしょうか？

[退会済みユーザー]

それは実際に動かしてみるほうが良いと思いますよ。
せっかくここまで試してみてるので。

ちなみにですが、提示したスクリプトは「phpでは極力HTMLタグを出力しない。」というコンセプトで記述しています。
これをすると、ロジック部分と表示部分を分けることが容易になり、結果、可読性が高まります。意識して記述すると、コードがきれいになります。

サクッとコードを試したいときは、以下のサイトが使いやすいです。
https://paiza.io/

[MeB]

できました！
サイトもありがとうございます！

Answer 3

$r1 か $r2　に1が入っているときまたは両方に1が入ってるときbbbを表示させる。

それ以外はaaa表示させたいのですが、

この条件を満たすif文なら下記だけで良いでしょう

if ($r1 === 1 || $r2 === 1) {
	echo '<h1>bbb</h1>';
} else {
	echo '<h1>aaa</h1>';
}

Comments

[MeB]

自分やってたことが無駄すぎて笑っちゃいました。
同様に$r1、$r2が3,4...$r7まであり
<h1>ccc</h1>があった場合も同様に
足して行く感じでよろしいのでしょうか？

[退会済みユーザー]

他にも条件が増えるようなら
} else {
の前に
} else if (条件) {
// 処理
}
を追加していけば良いですよ

[Equestria]

そうです。
読みやすいように改行などに気を付けるといいかと思います。

[MeB]

できました。
ありがとうございます。

Answer 4

$r1 か $r2　に1が入っているときまたは両方に1が入ってるときbbbを表示させる。
それ以外はaaa表示させたいのですが、

<?php if($r1=='1' || $r2=='1'){ echo "<h1>bbb</h1>"; }else{ echo "<h1>aaa</h1>"; } ?>