Salut à tous,
C'est quelque chose que je vois très très souvent dans les codes pendant les séries, donc je me permets une petite remarque.
Voici la simplification progressive du code suivant:
bool wesh() {
bool retour;
if (condition == true) { retour = true; }
else { retour = false; }
return retour;
}
Où condition est une expression booléenne (comme p. e.x. `x < 5` ou `str == "hello world"`)
Déjà la condition VAUT true ou false, donc pas la peine de checker sa valeur contre un bool !
bool wesh() {
bool retour;
if (condition) { retour = true; }
else { retour = false; }
return retour;
}
Ensuite, ce que font beaucoup c'est passer le retour dans la branche if-else, on se passe alors de la variable temporaire (c'est bien pour les petites fonctions, mais des fois c'est plus pratique de la garder (p. ex. fonction de sanitisation d'un string : retour de string mais sans les caractères non-alphanumériques !))
bool wesh() {
if (condition) { return true; }
else { return false; }
}
On y est presque ! Rappel: toute expression FAIT et VAUT qqc ! En l'occurence, condition VAUT une valeur booléenne ! Donc tout ceci se réduit à:
bool wesh() {
return condition;
}
Et voilà ! C'est plus propre non ?
(Modifié par Loïs Bilat. Écrit initialement le samedi, 23 novembre 2019, 14:46)