Discussion :

[JulioLaSalamandre]

Bonjours,

On m'a récemment posé une colle: Certaines personnes posent leurs conditions de la façon suivante:

if (condition == true)

et d'autres :

if (true == condition)

J'aimerai connaitre les avantages de la deuxième méthode.
C'est peut être une question bateau mais je me permet de la poser quand même.

Merci d'avance

[ternel]

L'intérêt de la seconde méthode est de poser une erreur de compilation si on oublie un =.

Cependant, il est encore plus simple d'écrire if (condition), et il ne peut pas y avoir d'erreur.
Et si ce n'est pas lisible, c'est probablement parce que la variable est mal nommée.

Par exemple, tester si une chaine de caractère (bidule) est vide se dirait en anglais "if bidule is empty, then say sorry"

En code, on a trois choix:

• if (bidule.empty() == true) say("sorry");, "if the fact that bidule is empty is true, (then) say sorry", ou en français "si le fait que bidule est vide est vrai, dire désolé"
• if (true== bidule.empty()) say("sorry"); "if true is the fact that bidule is empty, (then) say sorry", ou en français "si vrai est le fait que bidule est vide, dire désolé"
• if (bidule.empty()) say("sorry"); "if bidule is empty, (then) sey sorry", ou en français "si bidule est vide, dire désolé"

Autant dire que la troisième solution est plus claire. Au passage, empty() est régulière critiquée, car elle aurait pû ou dû s'appeler is_empty().

[stendhal666]

Aujourd'hui, ton compilateur t'avertira toujours s'il voit: if ( a = true ). La véritable bonne pratique c'est de toujours lire les avertissements du compilateur et de mettre le niveau d'avertissement le plus élevé.

C'est une bonne pratique d'un autre temps de mettre if (true == a) à mon avis. C'est bien de la connaître pour lire le code des autres, mais if (a == true) est quand même plus naturel et plus agréable à lire.

Pour le cas particulier où on teste si un bool renvoie vrai, supprimer true est une bonne idée. Pour le bool dont on veut savoir s'il est false, il y a: if (!empty), pourquoi pas. Mais la pratique est valable pour toute constante:
if (2 == nb_joueurs).

En revanche si on compare deux variables non constantes : if (nb_max ==current) elle ne permet pas de détecter l'erreur if (nb_max = current). Donc la seule sauvegarde vraiment générale ce sont les avertissements du compilateur.