否定を強調する

かなり長いプロパティ名でifステートメントを書いていたところ、この問題に遭遇しました。

次のようなifステートメントがあるとします。

if(_someViewModelNameThatIsLong.AnotherPropertyINeedToCheck == someValue &&
   !_someViewModelNameThatIsLong.ThisIsABooleanPropertyThatIsImportant)
{
    //Do something
}

2番目のプロパティはブール型であり、次のようなステートメントを持つことは意味がありません

if(boleanValue == true)

否定を強調するより良い方法はあり!ますか？私には、コードを読み取るときにこれが簡単に監視され、デバッグで問題が発生する可能性があるようです

if(_someViewModelNameThatIsLong.NeedsMeToDoSomething(someValue))
{
    //Do something
}

そして、ビューモデルオブジェクトで

public bool NeedsMeToDoSomething(string someValue)
{
    return AnotherPropertyINeedToCheck == someValue &&
        !ThisIsABooleanPropertyThatIsImportant;
}

（someValueが文字列であり、モデルオブジェクトによって認識されていない場合）

これは、！演算子ですが、一般的に読みやすくします。これで、呼び出し元のメソッドに1つの条件が表示されます。これは、呼び出し元オブジェクトのコンテキストで条件を説明するために名前を付ける必要があります。また、モデルオブジェクトでは、モデルオブジェクトのコンテキストでそれが何を意味するかを確認できます。

重要度の低い条件を評価する前に、それを独自のifブロックに入れます。他の条件の混乱なしに読みやすくなるだけでなく、プログラマーが最初に読む条件でもあります。これを、@ scrwtpで既に述べたアイデアと組み合わせて、意味のある名前の変数に割り当てて、次のようにします。

var isValid = !_someViewModelNameThatIsLong.ThisIsABooleanPropertyThatIsImportant;
if( isValid ) 
{
    if( _someViewModelNameThatIsLong.AnotherPropertyINeedToCheck == someValue ) 
    {
        //Do something
    }
}

コンパイラ言語でプログラミングしている場合、外側のifと内側のifの間にコードを挿入しない限り、ほとんどの場合、これらのネストされたifブロックは最終的に結合されます。ケース。

C / C ++を使用している場合は、プリプロセッサで読みやすくすることができます。

#define NOT !

if(_someViewModelNameThatIsLong.AnotherPropertyINeedToCheck == someValue &&
    NOT _someViewModelNameThatIsLong.ThisIsABooleanPropertyThatIsImportant)
{
    //Do something
}

私はちょうど抽出したい

`!_someViewModelNameThatIsLong.ThisIsABooleanPropertyThatIsImportant`

これを返すメソッド内。このメソッドにNotThisIsABooleanPropertyThatIsImportantという名前を付けた場合、問題ありません。