¿Cómo sangría long conditionals para declaraciones 'if'?

Question

Mi pregunta se relaciona con this previous question, pero las soluciones ofrecidas no abordan el problema que he descrito a continuación. Después de una búsqueda en Google, no he encontrado ninguna guía de estilo de código que aborde el problema específico de los condicionalmente largos en una declaración if como esta.

if(isNull(value1) || 
    isToLong(value1) || 
    hasBadFormat(valule1)){ 
    doSomething(); 
}else{ 
    doSomethingElse(); 
} 

O:

if(isNull(value1) || isToLong(value1) || hasBadFormat(valule1)){ 
    doSomething(); 
}else{ 
    doSomethingElse(); 
} 

El problema que tengo con estos dos estilos es que hace que sea difícil para el ojo para encontrar el código en el bloque verdadero y lo separan de los condicionales, o es es demasiado difícil para el ojo determinar la siguiente línea correcta después de una larga condicional en una sola línea, especialmente si la instrucción if ya tiene sangría algunas pestañas dentro de una función u otras declaraciones if.

¿Sería preferible hacer algo como esto:

if( isNull(value1) || 
     isToLong(value1) || 
     hasBadFormat(valule1)){ 
    doSomething(); 
}else{ 
    doSomethingElse(); 
} 

o sería este estilo será mejor para sangrar cada nueva condición en cualquiera de estas maneras:

if(isNull(value1) || 
     isToLong(value1) || 
      hasBadFormat(valule1)){ 
    doSomething(); 
}else{ 
    doSomethingElse(); 
} 

if(isNull(value1) 
     || isToLong(value1) 
      || hasBadFormat(valule1)){ 
    doSomething(); 
}else{ 
    doSomethingElse(); 
} 

¿Alguien tiene una codificación guía de estilo (tal vez una política de estilo de codificación de la empresa) que aborda este problema de una manera diferente o mejor de lo que he propuesto? ¿Cuál es preferible y puede encontrar inconvenientes o ventajas en las soluciones que he mencionado?

Answer 1

La solución obvia es mover la abrazadera abierta a la siguiente línea, ¡tal como Dios quiso!

</flamebait>

Answer 2

if(isNull(value1) || 
    isToLong(value1) || 
    hasBadFormat(valule1)) 
{ 
    doSomething(); 
} 
else 
{ 
    doSomethingElse(); 
} 

Ahora aparece el verdadero bloque fácilmente que pienso.

Por supuesto, prefiero:

if(isNull(value1) 
    || isToLong(value1) 
    || hasBadFormat(valule1)) 
{ 
    doSomething(); 
} 
else 
{ 
    doSomethingElse(); 
} 

:-)

Answer 3

Realmente depende de la preferencia y la convención de las personas con las que trabaja, pero los dos primeros son las dos formas más comunes que he visto . Tiendo a preferir mover el condicional a líneas múltiples solo si es tan largo que requiere desplazamiento de izquierda a derecha en mi ide.

Esto es como lo escribiría:

if(isNull(value1) ||  
    isToLong(value1) || 
    hasBadFormat(valule1)) 
{  
    doSomething(); 
} 
else 
{  
    doSomethingElse(); 
} 

A menos que condicional no es tiempo suficiente para obligarme a desplazarse para verlo todo. Si no es así, me gustaría hacer esto:

if(isNull(value1) || isToLong(value1) || hasBadFormat(valule1)) 
{  
    doSomething(); 
} 
else 
{  
    doSomethingElse(); 
} 

Y en este caso, ya que parece lo suficientemente corto, lo haría este último.

Answer 4

¿Qué tal algo así?

bool isValid = isNull(value1) || isToLong(value1) || hasBadFormat(valule1); 
if(isValid) 
{ 
    doSomething(); 
} 
else 
{ 
    doSomethingElse(); 
} 

El condicional se mueve a otra línea, lo que podría facilitar la lectura.

Answer 5

Tiendo a poner los operadores al principio de la línea para que todos suban.

tanto, aquí está una sugerencia:

if(isNull(value1) 
    || isTooLong(value1) 
    || hasBadFormat(valule1)) 
{ 
    doSomething(); 
} /* if */ 
else 
{ 
    doSomethingElse(); 
} /* else */ 

Aquí hay otro:

if(0 
    || isNull(value1) 
    || isTooLong(value1) 
    || hasBadFormat(valule1)) 
/* ...etc... */ 

(Para & &, lo que sería si (1 & & un & & b), etc.)

O esto:

if 
(
    isNull(value1) 
    || isTooLong(value1) 
    || hasBadFormat(valule1) 
) 
/* ...etc... */ 

Answer 6

Probablemente soy el único que lo hace de esta manera. Se llama Horstmann Style, pero lo hago de forma ligeramente diferente.

if (bool)  // comment 
{ dothis;   
    andthis;  
} else if (bool) // comment 
{ dothis;  
    andthis;   
} else   // comment 
{ dothis;   
    andthis;  
} 

Answer 7

Esta es la alternativa que prefiero:

if(
    isValid = isNull(value1) || 
    isToLong(value1) || 
    hasBadFormat(valule1) 
) { 
    doSomething(); 
} 
else { 
    doSomethingElse(); 
}