Enfatice la negación

Estaba escribiendo una declaración if con nombres de propiedad bastante largos y encontré este problema.

Digamos que tenemos una declaración if como esta:

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

La segunda propiedad es de tipo booleano y no tiene sentido tener el stetement como

if(boleanValue == true)

¿Hay una mejor manera de enfatizar la negación que poner el !frente? Para mí, parece que esto se puede supervisar fácilmente al leer el código y puede causar problemas de depuración.

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

Y luego, en la vista del objeto modelo

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

(suponiendo que someValue es una cadena y el objeto modelo no la conoce)

¡Esto no solo enfatiza el! operador, pero generalmente lo hace más legible. Ahora, en el método de llamada, puedo ver una condición, que debería estar bien nombrada para describir la condición en el contexto del objeto de llamada. Y en el objeto modelo, puedo ver lo que eso significa en el contexto del objeto modelo.

Póngalo solo si bloque antes de evaluar las condiciones menos importantes. No solo sería más fácil de leer sin el desorden de las otras condiciones, sino que también es la primera condición que leerá un programador. Combine esto con la idea ya mencionada por @scrwtp para asignar a una variable con un nombre significativo y obtendrá:

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

Si está programando en un lenguaje de compilación, la mayoría de las veces estos bloques anidados si se combinan al final de todos modos, siempre que no inserte código entre el if externo y el if interno, por lo que no debería afectar el rendimiento en estos casos.

Si está utilizando C / C ++, entonces el preprocesador podría proporcionarle legibilidad.

#define NOT !

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

Solo extraeria

`!_someViewModelNameThatIsLong.ThisIsABooleanPropertyThatIsImportant`

En un método que devuelve esto. Si nombra este método NotThisIsABooleanPropertyThatIsImportant, debería estar bien.