CodeSOD : Renforcer l'argument Note

CodeSOD : Renforcer l'argument

David a rencontré une demande de tirage (pull request) déroutante avec des commentaires étendus axés sur un changement apparemment simple : l'ajout d'un seul paramètre à une fonction JavaScript. La fonction d'origine avait déjà sept paramètres, ce qui suggérait potentiellement une opportunité de refactoring pour utiliser un objet. La révision initiale a introduit une instruction conditionnelle liée à un nouveau paramètre, notArg8. Si notArg8 était vrai, le code passait arg8: !notArg8 au contexte. L'auteur ne pouvait pas comprendre cette approche compliquée, car il aurait suffi d'ajouter simplement le paramètre. David a commenté le mauvais code, mais le développeur d'origine a cessé de répondre. Un développeur junior a approuvé le changement de code confus, mais il n'a jamais été fusionné. Un autre développeur a ensuite simplifié la solution, en ajoutant simplement arg8 et en le passant au contexte. Le code simplifié a fonctionné, a passé les tests et a rendu tout le monde heureux. L'intention du développeur d'origine et la raison derrière l'instruction conditionnelle compliquée restent un mystère. La fin suggère que la vision du développeur d'origine reste irréalisée.