¿Cuál es la mejor práctica con respecto a la evaluación de cortocircuito C#?

Question

Una respuesta y posterior debate in the comments en otro hilo me hizo preguntar:

En C# || y & & son las versiones en cortocircuito de los operadores lógicos | y & respectivamente.

Ejemplo de uso:

if (String.IsNullOrEmpty(text1) | String.IsNullOrEmpty(text2) | String.IsNullOrEmpty(text3)) 
{ 
    //... 
} 

frente:

if (String.IsNullOrEmpty(text1) || String.IsNullOrEmpty(text2) || String.IsNullOrEmpty(text3)) 
{ 
    //... 
} 

En términos de práctica, que es el mejor utilizar y por qué codificación?

Nota: Me doy cuenta de que esta pregunta es similar a this question, pero creo que garantiza un debate específico del idioma.

Answer 1

En cuanto a la práctica, que es el mejor utilizar la codificación y por qué?

Respuesta simple: utilice siempre las versiones en cortocircuito. Simplemente no hay razón para no hacerlo. Además, hace que su código sea más claro porque expresa su intento : evaluación lógica. Utilizar las operaciones bit a bit (lógicas) implica que solo desea eso: operaciones de bits, no evaluación lógica (aunque MSDN las llama también "operadores lógicos" cuando se aplica a valores booleanos).

Además, dado que un cortocircuito en sólo evalúa lo que se necesita evaluar, a menudo es más rápido, y permite escribir dicho código como

bool nullorempty = str == null || str.Length == 0; 

(Tenga en cuenta que para resolver este problema en particular una mejor función ya existe, es decir, string.IsNullOrEmpty que también usó en su pregunta.) Este código no sería posible con las operaciones lógicas bit a bit porque incluso si str fuera null, se evaluaría la segunda expresión, lo que daría como resultado NullReferenceException.

EDITAR: Si desea efectos secundarios que se producen en un contexto lógico favor todavía no utilizan operaciones bit a bit. Este es un ejemplo típico de ser demasiado inteligente. El próximo mantenedor del código (o incluso usted mismo, después de algunas semanas) verá que este código pensará "hmm, este código se puede borrar para usar operadores condicionales", rompiendo inadvertidamente el código. Lamento a quien sea que esté a cargo de arreglar este error.

En cambio, si usted tiene que confiar en el lateral, efectos, hacerlas explícitas:

bool hasBuzzed = checkMakeBuzz(); 
bool isFrobbed = checkMakeFrob(); 
bool result = hasBuzzed || isFrobbed; 

sentado, tres líneas en lugar de uno. Pero un código mucho más claro como resultado.

Answer 2

Utilice && y || cuando solo le importa el resultado y desea saber ese resultado lo antes posible y ninguna de sus expresiones tiene efectos secundarios que deben ocurrir incluso si no se cumple la condición booleana. Es decir, casi siempre.

Utilice & y | cuando se debe evaluar cada expresión (por ejemplo, si tiene efectos secundarios de sus expresiones). Pero dado que nunca debe tener efectos secundarios de los que su programa depende que deben ocurrir incluso si no se cumple la condición booleana, probablemente no deba usar & y |.

Por ejemplo, esto probablemente sería excepcionalmente tonto:

if (false & somethingThatUpdatesTheDatabase()) { /* ... */ } 

Answer 3

Voy a responder a esta pregunta a la inversa: cuando es el momento única utilizo los operadores lógicos?

A veces usaré las comparaciones lógicas cuando tengo una serie de condicionales (de bajo costo) que deben cumplirse todos. Por ejemplo:

bool isPasswordValid = true; 

isPasswordValid &= isEightCharacters(password); 
isPasswordValid &= containsNumeric(password); 
isPasswordValid &= containsBothUppercaseAndLowercase(password); 

return isPasswordValid; 

En mi opinión, lo anterior es más fácil de leer que:

return (isEightCharacters(password) && 
     containsNumberic(password) && 
     containsBothUppercaseAndLowercase(password)); 

La desventaja es que es un poco más esotérica.