if文のifの中に入る条件が複雑なときの書き方

Question

C#初心者です。
今カードを書いているのですが、if文の書き方で少し疑問があったので質問しました。

条件として指定したいのが、
①A=1かつB=2の時か、
②C=5かつD=6の時
みたいなかんじです。

この場合の書き方って
if((A==1 && B==2) || (C==5 && D==6))
でいいんでしょうか？
初歩的な質問ですみません。中々該当する答えを見つけられなかったのでお願いします。

Answer 1

はい。本件はそれで正しいです。が、もっと複雑な場合も多々あるので、状況次第では整理しないとわけがわからなくなります。

例えば、条件演算結果をbool変数に代入して、これをifの中で確認するとか、ifブロックの入れ子化とかが例です。

逆に簡易な分岐ならば三項演算子で行数を節約するとか、switch構文を使って書くという方法もあります。もっとも、この辺りはかなりプログラマのセンスに依るものがあるので一概には言えません。

ついでですが、「変数と定数が等しい」という条件式を書くならば、定数を左辺に置くことが良いと言われています。＝と＝＝を書き間違えたとしても、コンパイル時にエラーメッセージが出るからです。

Answer 2

書き方はそれで問題ないです
ただ、if文が長くなるということは可読性の低下に繋がります

もしできるのであればコメントもしくは変数による命名によって
読みやすいコードにした方がよいです

例えば

①A=1かつB=2の時か、

②C=5かつD=6の時

の場合で、仮にゲームで考えると

①ジャンプ中かつ上からか、

②スター取得中かつ全方向

よりも

①ジャンプ攻撃か、

②無敵状態

の方が分かりやすくなります
コードで示すと

bool jumpAttack = (isJump && direction = 1);
bool invincible = (isStar && direction = 15);
if(jumpAttack || invincible){}

というような感じで実装すると
見やすいし、後々のデバックや仕様変更に対応しやすく良いかと思います

正直自分も初心者なので間違っているかもしれませんが
参考にしてみてください

Answer 3

if((A==1 && B==2) || (C==5 && D==6))

良いと思います。
不安なときは、デバッガで確認するなり、小さいプログラムを書いて確認するのが近道と考えます。

Answer 4

それでいいですよ。

VisualStudioを使ってるなら、ブレークポイントで実行を止めて、変数のナカミを確認したり、１行づつステップ実行したりできますんで、それで動作を確認してはどうでしょう