Samedi 13 août 2016 à 00:00
Lexpage n'est pas toujours celui que l'on croit...
—
Nightm4re
Discussions
Billet - Pourquoi faire les codes review ?
Cette discussion est en relation avec le billet suivant :
Reprise automatique du message précédent.
Merci Tchou. J'ai galéré toute la journée avec mes promesses en JS mais je crois que tu as eu une pire journée que moi 
MerleBon après en .Net tu ne peux pas faire de if(variable = true) (pas d'assignation dans un if) donc le problème ne se posera pas...C'est mes deux derniers jours à relire le code de ma boîte, pour trouver pourquoi des formulaires qui auraient du être préremplis étaient vides…
Un
variable = true au lieu de variable === true 
Un bon petit test unitaire du genre 
(oui, je lis souvent du code de tests où les gens passent leur temps à faire des assertTrue au lieu de profiter du "langage" d'assertions pour que (1) ça soit plus lisible, et que (2) ça casse pas bêtement
assertTrue(variable = value) (oui, je lis souvent du code de tests où les gens passent leur temps à faire des assertTrue au lieu de profiter du "langage" d'assertions pour que (1) ça soit plus lisible, et que (2) ça casse pas bêtement
PetitCalgonTest unitaires !L'application est tellement grosse et vieille que bon, je suis pas sûr que ce soit si simple.
On fait ça à l'ancienne avec des testeurs et des serveurs de recette.
Tant qu'on est dans le thème de la code review, s'il y a un motivé pour me relire je suis preneur :
github.com/adyxax/b98/bl…
github.com/adyxax/b98/bl…