¿Qué es más fácil de mantener: asignación booleana a través de if / else o expresión booleana?

¿Cuál se consideraría más mantenible?

if (a == b) c = true; else c = false;

o

 c = (a == b);

He intentado buscar en Code Complete, pero no puedo encontrar una respuesta.

Creo que el primero es más legible (literalmente puedes leerlo en voz alta), lo que también creo que lo hace más fácil de mantener. El segundo ciertamente tiene más sentido y reduce el código, pero no estoy seguro de que sea tan fácil de mantener para los desarrolladores de C # (esperaría ver más este modismo en, por ejemplo, Python).

La segunda opción es mejor.

Hay una razón definitiva para desconfiar de los atajos de programación inteligentes que perjudican el mantenimiento al ocultar la intención del código. Entonces, no te culpo por hacer la pregunta.

Sin embargo, no considero c = (a == b);que sea un ejemplo de un truco inteligente. Es una representación directa de un concepto simple. Tan sencillo como puedas.

Una adecuada, "mantenible" formato del primer ejemplo (sin las llaves que faltan y construcción de una línea, que yo hago considerar un atajo inteligente) produciría este código:

if (a == b)
{
    c = true; 
}
else 
{
    c = false;
}

En mi experiencia, escribir una lógica booleana simple en una forma tan propensa a errores es un signo de código "dudoso". Me haría preguntarme cómo se maneja una lógica más compleja en esta base de código.

Primero, tenga en cuenta que sus dos formas no son equivalentes.

if (a == b) c = true;

cse establecerá en verdadero si aes igual a b, y si no, su valor seguirá siendo lo que sea.

c = (a == b);

cse establecerá en verdadero si aes igual a b, y si no, se establecerá en falso.

Si desea el equivalente de la segunda forma, en el estilo de la primera forma, debe escribirlo así:

if (a == b) {
  c = true;
} else c = false;

Ahora está claro cuál de los dos es más legible, más fácil de mantener y menos propenso a introducir errores si se cambia algo. Quédate con la segunda forma.

No estoy de acuerdo con que su primer formulario sea más legible: ciertamente no es idiomático C # tener dos declaraciones en una sola línea, y no se recomienda tener una ifdeclaración sin usar llaves.

En segundo lugar, no veo cómo la segunda forma es menos mantenible: no hay nada que mantener. Es una simple declaración de la relación entre ay by no podría expresarse más sencillamente.

Otra razón para preferir la segunda forma es que puede declararla cy asignarla en una sola declaración, es decir

bool c = (a == b);

Modificar variables fácilmente puede conducir a errores, por lo que lo evitaría. El uso de una ifdeclaración requiere que la variable se declare antes del condicional y luego se modifique.

"más mantenible" podría ser muy subjetivo.

Por lo general, prefiero la legibilidad y la intención sobre la reducción de código. Creo que guarda 8 caracteres escritos usando la forma reducida.

Tomar el idioma y la cultura alrededor del idioma es una característica de 'legibilidad' en mi opinión.

Hay momentos en que el rendimiento puede ser la causa de la reducción del código para optimizar el código de bytes resultante, pero eso debe hacerse con cuidado después de algunos perfiles.

El segundo. Tiene menos de repetición (DRY) y es más fácil de entender lo que está pasando, que cmantiene al valor de si o no ay estar bson iguales.

En mi humilde opinión, aún mejor sería

c = a == b

Justo como escribiría

• 1 + 2 + 3 en vez de ((1 + 2) + 3)
• 5 + 3 * 7 en vez de (5 + (3 * 7))

Obviamente y el código trivialmente innecesario no es una virtud. Está abarrotado.

Votantes bajos, por favor explique qué está mal con mi respuesta revisada.

Sí, c = (a == b);puede ser difícil de leer (peor aún, StyleCop se queja del paréntesis innecesario), pero todavía me gusta la simplicidad de a == b. Por lo tanto, esto es lo que me gusta usar cuando ambos ay bson los mismos:

private static bool NoPeriod
{
    get
    {
        return this.MyWaveLength == this.HerWaveLength;
    }
}

Y luego puedes hacer: en this.c = this.NoPeriodlugar de:

this.c = this.MyWavelength == this.HerWaveLength;