if(!value)
C'est toujours plus clair à mon avis.
if(value == false)
Je déteste dire cela, parce qu'il semble un peu de la moyenne, mais cela montre que la personne qui écrit le code n'a pas vraiment de comprendre l'utilisation de valeurs booléennes. Vous n'avez pas besoin de valider à nouveau ce qu'est un booléen dans une instruction if. C'est redondant.
(Personnellement, je serais gêné à la personne trop si ils ont nommé la variable value au lieu de quelque chose de plus significatif. J'ai le sentiment que ce que vous avez posté est juste pseudo code, je serais certainement ding que sur un examen.)
Modifier (en réponse à un commentaire ci-dessous):
Il peut sembler trivial, mais c'est souvent un signe de beaucoup plus de choses. Honnêtement, la plupart des gens qui ne l'utilisent var == true etc. ne pas comprendre. C'est juste un fait. Je ne dis pas que leurs stupides ou ils ne devraient pas être les programmeurs juste qu'il y a probablement quelque chose qu'ils ont besoin de revoir et d'apprendre. Le problème est que lorsque la logique devient beaucoup plus complexe, de ne pas comprendre des concepts tels que cela peut conduire à beaucoup plus de problèmes sur la route. Certaines personnes disent que "c'est un style." C'est très bien. La vraie question dans ce cas est, "Comment est-il avantageux pour moi de faire de cette façon? Que dois-je (ou d'autres personnes en tirer?" Si vous ne pouvez pas solidement répondre à cette question, alors vous devez vous demander "Pourquoi est-ce une bonne idée?"
