1TBS para expresiones condicionales largas

Question

Nota importante: esta pregunta no se trata de la superioridad de un estilo de arriostramiento sobre otro. Actualmente estoy cambiando de estilos para evaluar por mí mismo cuál creo que funciona mejor en mi situación, y me gusta Allman hasta 1TBS.

Usuarios del estilo de refuerzo de 1TBS, ¿cómo formatear las condiciones largas en una declaración if y el código subsiguiente?

if ((this_is_the_first_part_of_a_long_condition) 
    && (the_second_part_is_shorter__wait_no_it_is_not) 
    && (and_one_more_for_the_road)) { 
    here_comes_the_block_code(); 
} 

Creo que debe haber una manera mejor. Mi forma actual es agregar una línea vacía antes de la primera línea del bloque de código. Allman tampoco se ve muy bien en este caso, aunque es más legible en mi opinión.

Otro ejemplo con for bucles:

for (int relevant_counter_variable_name = START_VALUE; 
    intelligent_expression_that_may_include_the_counter_variable; 
    relevant_counter_variable_update) { 
    first_code_line_inside_the_block(); 
} 

No tan agradable ...

KNF (8) espacios guión ayudarían aquí, pero me gustaría evitar eso. Tengo algunas otras opciones, pero me gustaría saber si hay algún tipo de forma estándar.

Answer 1

if (
    (this_is_the_first_part_of_a_long_condition) 
    && (the_second_part_is_shorter__wait_no_it_is_not) 
    && (and_one_more_for_the_road) 
) { 
    here_comes_the_block_code(); 
} 

Answer 2

Mix-n-partido

---

Estoy de acuerdo en que los estilos de mezcla es a menudo mal visto.
Pero, me atrevo a decir que siempre que sea posible, la regla puede doblarse para facilitar la lectura.

En los casos en que el estilo es estrictamente forzada, (empresa de codificación-políticas)
que suelo hacer esto:

if ( (this_is_the_first_part_of_a_long_condition) &&  
     (the_second_part_is_shorter__wait_no_it_is_not) && 
     (and_one_more_for_the_road)) { 
       here_comes_the_block_code(); 
} 

Sólo tiene que utilizar un nivel de guiones para todas las condiciones,
y otro nivel adicional para el código dentro de las llaves.
Esto es lo más legible posible, sin ofender a ningún purista.

Answer 3

I líneas continuas de doble guión:

if ((this_is_the_first_part_of_a_long_condition) 
     && (the_second_part_is_shorter__wait_no_it_is_not) 
     && (and_one_more_for_the_road)) { 
    here_comes_the_block_code(); 
} 

Answer 4

espacio único para cada nivel de sangría, utilizan paréntesis para cada condición si necesita o no.

Para condiciones complicadas, los paréntesis de estilo Allman pueden funcionar bien.

El enfoque general funciona para la continuación del código que no cabe en una línea, o para listas de argumentos de función.

Cada elemento de cierre está indentado al mismo nivel que el elemento de apertura, de ahí el "));" para "Trace.WriteLine (String.Format (" y el independiente ";" para "devolver".

YMMV.

if (
    (
    (this_is_the_first_part_of_a_long_condition) && 
    (the_second_part_is_shorter__wait_no_it_is_not) && 
    (and_one_more_for_the_road) 
    ) || 
    (
    (this_is_the_first_part_yet_another) && 
    (
     (the_second_part_yet_another) || 
     (val < 22) 
    ) 
    ) 
    ) { 
     here_comes_the_block_code(); 

     int bits = 0 
     | O_DEF 
     | CONFIG_THIS 
     | CONFIG_THAT 
     ; 

     FILE *OUPT = fopen(
     "/tmp/oupt.txt", 
     "a+" 
    ); 

     Trace.WriteLine(String.Format(
     "format {0} 0x{1:x8}" 
     ,(eGenericDeviceFeatureEnum)val 
     ,val & 0x7ffffc00 
    )); 

     return 
     (CurrentWort != null) && 
     (CurrentWort.IsFeatureSupported(
     eGenericDeviceFeatureEnum.SupportsTriBromoCarcinogen 
     )) 
     ; 
    } 

Answer 5

simplemente que perdería unas pocas variables para absoluta claridad y legibilidad:

cond1 = this_is_the_first_part_of_a_long_condition; 
cond2 = the_second_part_is_shorter__wait_no_it_is_not; 
cond3 = and_one_more_for_the_road; 
if (cond1 && cond2 && cond3) { 
    here_comes_the_block_code(); 
} 

No! 1TBS en todo su esplendor Sin estilo de mezcla. Sin fealdad La sangría (1) puede ocuparse de ella sin /* *INDENT-OFF* */ hacer trampa.

Incluso se puede dar a las condiciones nombres significativos, como

guidance = this_is_the_first_part_of_a_long_condition; 
navigation = the_second_part_is_shorter__wait_no_it_is_not; 
surgeon = and_one_more_for_the_road; 
if (guidance && navigation && surgeon) { 
    capcom_we_are_go_for_powered_descent(); 
} else { 
    agc_alarm(1202); 
}