「今回は複数の比較を1回でやらない」というテーマで解説していきます。
BAD:複数の比較を1回にやらない
次のような例を見てください。
private void 複数の比較を1回のif文でやらない(int valueA, int valueB) { //BAD:複数の比較を1回にやらない if (valueA == 1 && valueB == 1) { //処理 } }
valueAとvalueBを「&&」でつないで比較しています。極力こういった書き方はしない方がいいと思います。今回の例では2個の比較ですが,3個4個とある場合や,ANDとORが混ざっていたり,小確固で優先される比較を囲ったりしている場合は,どんどんわかりづらくなり,非常にバグの元となります。
ANDの場合は階層を分ける
複数の比較をANDでつないで判定する場合は,階層を分けることで対処できます。
//複数の比較を1回のif文でやらない private void 複数の比較を1回のif文でやらない(int valueA, int valueB) { //BAD:複数の比較を1回にやらない if (valueA == 1 && valueB == 1) { //処理 } //ANDの場合 //NOTBAD:階層を分ける if (valueA == 1) { if (valueB == 1) { //処理 } } }
このように,valueAの判定を通過した場合のみvalueBの判定をするように,if文をネストすれば,if文の階層は深くなりますが,判断は1回ずつしか行わないため,&&で連結するよりもバグりにくいでしょう。
ORの場合
NOTBAD:複数の比較を1回にやらない
2つの判定をOR判定する場合は,先ほどのようにif文をネストしてやるわけにはいきませんね。valueAの判定がFalseでもvalueBの判定は行いたいわけですから,次のように「||」で連結して複数の判定をしなくてはなりません。
//ORの場合 //NOTBAD:複数の比較を1回にやらない if (valueA == 1 || valueB == 1) { //処理 }
ただ,この場合は,前述の通り,バグの温床になるので,次に紹介するメソッド化を検討するほういいでしょう。
GOOD:メソッド化を検討する
次に例は,IsWriteLosgというメソッドを作成して対応しています。
//複数の比較を1回のif文でやらない private void 複数の比較を1回のif文でやらない(int valueA, int valueB) { ・・・ //GOOD:メソッド化を検討する if (IsWriteLosg(valueA, valueB)) { //処理 } } private bool IsWriteLosg(int valueA, int valueB) { if (valueA == 1) { return true; } if (valueB == 1) { return true; } return false; }
IsWriteLosgメソッドの中では,valueAが1ならTrue,valueBが1でもTrueが返却されるようになっており,どちらかが1ならTrueになるため,最初の「valueA == 1 || valueB == 1」と同じ動作をします。動作は同じですが,メソッドの中で,1つずつ判定を行い,どちらかがTrueの場合はすぐにメソッドを抜けることで,1つの判定のみの実施になっており,読みやすさとバグの混入を防ぐ効果があります。
GOOD:ANDでもメソッド化を検討する
先ほどはORの場合はメソッド化で対応するというお話をしましたが,ANDで比較する場合もif文のネストではなく,メソッド化を検討した方がより良いといえます。
ANDとORが混ざっていたらなおさらメソッド化する
さらにANDとORが混ざっていたり,小括弧が付いていて,比較の優先順位が複雑になっているようなものは,なおさら,メソッド化を検討してください。
#02_プロジェクトの作成
#03_右に長いコードを書かない_隣のとなりまでしか訪ねない
#04_隣のとなりまで_右スクロールより縦スクロールの方がいい
#05_IFとELSEがある時は肯定系をIF否定形をELSEにする
#06_比較する時は変数を左_定数を右にする
#07_複数の比較を1回のif文でやらない
#08_booの比較でTrueやFalseを書かない
#09_否定の否定はしない
#10_型チェックはasを使う
#11_メソッドはできるだけ早く抜ける_返却する値を無駄に変数に入れない
#12_対象外の時はすぐに抜ける
#13_都合が悪いケースはガードする
#14_必ずやりたい処理はfinallyを使う
#15_比較演算子はできるだけクラスにさせる
#16_ifの中括弧の省略はしない
#17_if文のリーダブルコードまとめ
#18_名前の付け方
#19_意図が明確な名前を付ける
#20_名前は素直に付ける_連想ゲーム的な名前を付けない
#21_1つの事しかしていなければ短い名前でも理解できる
#22_長いクラス名の扱い方
#23_単数形と複数形で表現する
#24_対になる言葉の組み合わせを決めておく
#25_業務で使う名前は統一する
#26_名前を統一するための辞書ツール作成
#27_メンバー変数にアンダーバーを付ける
#28_ハンガリアン記法を使わない
#29_メソッド内の変数をメソッド最初に全部宣言しない
#30_メソッド内の変数は直前に宣言する
#31_ループの変数はループ内で宣言する
#32_変数を使いまわさない
#33_boolの戻り値はどちらがTrueかをわかるようにする
#34_解放が必要なオブジェクトにはusingを使う
#35_varを推奨する場合
#36_メソッド名の付け方
#37_voidとFunctionを意識する
#38_インテリセンスを意識した名前にする
#39_生成メソッドはCreate_型変換はToを使う
#40_無駄に変数に入れて返却しない
#41_重複をなくす
#42_リージョンで区切らない
#43_アクセス修飾子とsealedを付ける
#44_クラス名はソリューションエクスプローラーで並べることを意識する
#45_クラス名は名詞か名詞句で命名する
#46_クラス名で継承元や特性を表現する
#47_メソッド内にコメントを書かない
#48_分かりづらい部分はメソッド化をしてメソッド名で想いを伝える
#49_コードを読んだ人が「えっ?」と思うことが予想される場所にだけコメントを付ける
#50_コメントで悪いコードを取り繕うことはできない
#51_未実装機能はTODOコメントを書く
#52_リーダブルコードまとめ
下記URLは、ベネッセが提供する教育用プラットフォームにて
私が公開しているプログラミング学習コースです。
無料コースもあるので、よかったらご覧になってみてください。
【無料動画】「C#を勉強する順番」