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.¿Cómo sangría long conditionals para declaraciones 'if'?
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?
Esta es la única cosa que me avisó de la llave-en-en-su-propia abierta - línea de campamento. – DaveE
En realidad, me encontré con esta respuesta porque estaba marcada como de baja calidad. Pero simplemente no puedo recomendar la eliminación. :) – helmbert