Quelle est la plus maintenable - l'affectation booléenne via if / else ou l'expression booléenne?

Laquelle serait considérée comme plus maintenable?

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

ou

 c = (a == b);

J'ai essayé de chercher dans Code Complete, mais je ne trouve pas de réponse.

Je pense que le premier est plus lisible (vous pouvez littéralement le lire à haute voix), ce qui le rend également plus facile à maintenir. Le second est certainement plus logique et réduit le code, mais je ne suis pas sûr qu'il soit aussi maintenable pour les développeurs C # (je m'attends à voir cet idiome plus dans, par exemple, Python).

La deuxième option est meilleure.

Il y a une raison certaine de se méfier des raccourcis de programmation intelligents qui nuisent à la maintenabilité en obscurcissant l'intention du code. Donc, je ne vous blâme pas d'avoir posé la question.

Cependant, je ne considère c = (a == b);pas être un exemple d'une astuce intelligente. Il s'agit d'une représentation simple d'un concept simple. Aussi simple que possible.

Un bon « maintenable » mise en forme de votre premier exemple (sans les accolades manquantes et la construction d' une ligne que je fais envisager un raccourci intelligent) donnerait ce code:

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

D'après mon expérience, l'écriture d'une logique booléenne simple d'une manière aussi prolixe et sujette aux erreurs est un signe de code "incertain". Cela me ferait me demander comment une logique plus complexe est gérée dans cette base de code.

Tout d'abord, réalisez que vos deux formulaires ne sont pas équivalents.

if (a == b) c = true;

csera mis à true si aest égal à b, et sinon, sa valeur restera ce qu'elle est déjà.

c = (a == b);

csera défini sur true si aest égal à b, et sinon, il sera défini sur false.

Si vous voulez l'équivalent du deuxième formulaire, dans le style du premier formulaire, vous devez l'écrire comme ceci:

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

Maintenant, il est clair lequel des deux est plus lisible, plus facile à gérer et moins susceptible d'introduire des bogues si quelque chose est changé. Restez avec la deuxième forme.

Je ne suis pas d'accord que votre premier formulaire soit plus lisible - ce n'est certainement pas idiomatique C # d'avoir deux déclarations sur une seule ligne, et il n'est pas recommandé d'avoir une ifdéclaration sans utiliser d'accolades.

Deuxièmement, je ne vois pas comment la deuxième forme est moins maintenable - il n'y a rien à maintenir. C'est une simple déclaration de la relation entre aet bet cela ne pourrait pas être exprimé plus simplement.

Une autre raison de préférer le deuxième formulaire est que vous pouvez le déclarer cet l'assigner dans une seule déclaration, c'est-à-dire

bool c = (a == b);

La modification des variables peut facilement conduire à des erreurs, donc je l'éviterais. L'utilisation d'une ifinstruction nécessite que la variable soit déclarée avant le conditionnel puis modifiée.

«plus maintenable» pourrait être très subjectif.

Je préfère généralement la lisibilité et l'intention à la réduction de code. Je pense que vous enregistrez 8 caractères dactylographiés en utilisant le formulaire réduit.

Prendre la langue et la culture autour de la langue est une caractéristique de la «lisibilité» à mon avis.

Il y a des moments où les performances peuvent être à l'origine de la réduction du code pour optimiser le code d'octet résultant, mais cela doit être fait avec soin après un certain profilage.

La deuxième. Il a moins de répétition (DRY) et est plus facile à comprendre ce qui se passe, qui cvaut si oui ou non aet bsont égaux.

À mon humble avis, encore mieux serait

c = a == b

Tout comme j'écrirais

• 1 + 2 + 3 au lieu de ((1 + 2) + 3)
• 5 + 3 * 7 au lieu de (5 + (3 * 7))

De toute évidence et trivialement, le code inutile n'est pas une vertu. C'est encombré.

Votants, veuillez expliquer ce qui ne va pas avec ma réponse révisée.

Oui, c = (a == b);peut être difficile à lire (pire encore, StyleCop se plaint de parenthèses inutiles), mais j'aime toujours la simplicité de a == b. Par conséquent, voici ce que j'aime utiliser lorsque les deux aet bsont les mêmes:

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

Et puis vous pouvez faire: this.c = this.NoPeriodau lieu de:

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