条件構文のスマートな使い方

Question

言語はC#
Windows　Formを使用してアプリを作成中です
SQLサーバー、VisualStudio2017という環境です。

今現在テキストボックスは4つあります。仮に
aTextBox　
bTextBox　
cTextBox
dTextBox
とします。

それぞれのエラーメッセージを代入する変数も
A,B,C,Dという感じで用意しました。

a,b,c,dのテキストボックスがすべて空欄の場合にしたい処理を

if(
a.textBox.Text.Length == 0 &&
a.textBox.Text.Length == 0 &&
a.textBox.Text.Length == 0 &&
a.textBox.Text.Length == 0)
{ 処理 }

と記載したのですが
aが記載ありでb,c,d入力なしの場合の処理
a,b記載ありでc,d入力なしの場合の処理

とそれぞれ記載のありなしで処理をわけたいんですが
if..else ifでわけていくとものすごい長さになっていますんですが、、、なにかいい方法はないでしょうか

Comments

[Zuishin]

それだけの情報なら「長いのを書いてください」としか言えません。もう少し条件がはっきりすれば別の答えもあると思います。

Answer 1

それぞれの状態によってintを返す判定用のメソッドを作るという手もあります。
（すべて空なら0　aのみ空なら1を返すなど）
そうして、処理はswitch文で分ければいいです。

ですが・・・
他の人に見せたり、自分が見たりしてもわかりやすいようにするのが基本だと思いますので
先の方が回答している、boolに逃がしてやる方法が何をしているかわかりやすいと思います。

Answer 2

bool aIsEmpty = a.textBox.Text.Length == 0;
という具合で各々のテキストボックスをboolに逃がしてあげて、次のように条件分岐をすると見やすいです。
また、変数は人が見て何の値が入っているのか分かるような命名をしてあげましょう。

boolean isEmpty = aIsEmpty && bIsEmpty && cIsEmpty && dIsEmpty;

if(isEmpty){
    //処理
}
else if(!aIsEmpty && bIsEmpty && cIsEmpty && dIsEmpty){
    //aが記載ありでb,c,d入力なしの場合の処理
}
else if(!aIsEmpty && !bIsEmpty && cIsEmpty && dIsEmpty){
    //a,b記載ありでc,d入力なしの場合の処理
}

余談ですが、abcdはコレクションで持っておくとLINQを使用して簡潔に書けるかもしれません。
例えば全てのテキストボックスが空かどうかは次のように書けます。
boolean isEmpty = textBoxes.All(v => v.Text.Length == 0);