Aujourd'hui, on m'a fait une remarque sur le code en considérant la façon dont je vérifie si une variable est vraie ou fausse dans un devoir scolaire.
Le code que j'avais écrit était quelque chose comme ceci :
var booleanValue = true;
function someFunction(){
if(booleanValue === true){
return "something";
}
}
Ils ont dit que c'était mieux/beaucoup mieux de l'écrire comme ça :
var booleanValue = true;
function someFunction(){
if(booleanValue){
return "something";
}
}
La remarque que j'ai reçue au sujet de la partie "=== true" était qu'elle n'était pas nécessaire et pouvait créer une confusion.
Cependant, mon idée est qu'il est préférable de vérifier si la variable est un booléen ou non, d'autant plus que Javascript est un langage loosetypé.
Dans le deuxième exemple, une chaîne de caractères renverrait également "quelque chose" ;
Ma question est donc la suivante : est-il plus propre de perdre la partie "=== true" à l'avenir, ou est-ce une bonne pratique de vérifier également le type de la variable ?
Editar: Dans mon "vrai" code, le booléen représente si une image a été supprimée ou non, donc les seules valeurs que boolValue devrait avoir sont true ou false.
0 et 1 par exemple ne devraient pas être dans cette variable.