8Fermer10
Pen^2Le 19/04/2017 à 02:50
Kevin Kofler (./6) :
j'évite les variables intermédiaires inutiles qui n'apportent rien
Elles apportent qu'elle permettent de construire le raisonnement, de raccourcir les expressions (les lignes), et aussi de débugguer. Et ça favorise aussi la factorisation, je ne compte pas les fois où je lis 15 fois de suite le même
getTruc().getMachin().doThis()[/pre] et [pre]getTruc().getMachin().doThat()
if ( funcBlip() != funcBloup() ) { ... est nettement moins clair pour moi que de nommer le booléen avec un nom bien choisi qui indique ce qu'il est vraiment dans l'algo. Parfois les tests portent sur des variables qui n'ont pas de lien évident entre elles. Si c'est pour finir avec un commentaire à côté du test, autant nommer un booléen.


Kevin Kofler (./6) :
indice: si tu dois passer du temps à chercher un nom à ta variable, c'est une variable artificielle qui n'apporte rien à la lisibilité
Qui a parlé de passer du temps ? Si tu ne sais pas ce que tu es en train de récupérer c'est qu'il y a un petit problème à mon avis cheeky


edit/cross
Kevin Kofler (./8) :
Il y a un seul if-else dans le code:
Tu peux appeler ça comme tu veux, c'est clairement un else perdu au fond du code, et justement pour éviter ça je l'aurais placé au début.