强调否定

我只是在编写带有相当长的属性名称的if语句，并且遇到了这个问题。

假设我们有一个if语句，如下所示：

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

第二个属性是布尔类型，像

if(boleanValue == true)

有没有更好的方法来强调否定然后放在!前面。在我看来，在阅读代码时可以很容易地对此进行监督，并且可能会导致调试问题

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

然后，在视图模型对象中

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

（假设someValue是一个字符串，并且模型对象不知道）

这不仅强调了！运算符，但通常使其更具可读性。现在，在调用方法中，我可以看到一个条件，应该对它进行适当的命名以在调用对象的上下文中描述该条件。在模型对象中，我可以看到在模型对象的上下文中这意味着什么。

在评估次要条件之前，请将其单独放入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，则应该没问题。