CodeSOD : Compter tout Note

CodeSOD : Compter tout

Aujourd'hui, c'est le jour de l'élection aux États-Unis et le décompte est dans l'esprit de beaucoup de gens. Le code en discussion n'est pas lié au décompte des votes, mais il est néanmoins problématique. Le code provient d'une ancienne application publiée en 2007 et fait référence à la limite de lignes d'Excel de 65 536, qui a été portée à 1 000 000 dans Excel 2007. Cependant, le code lit en réalité un fichier CSV et non un fichier Excel. Le code construit le nom du fichier deux fois, ce qui est inutile. Le véritable problème réside dans la façon dont le code compte les lignes, ce qui est inefficace et fonctionne mal sur les grands fichiers. Le code lit l'ensemble du fichier en mémoire, le divise en lignes et compte ensuite la longueur. Cette approche est terrible pour les grands fichiers, dont il y en avait beaucoup. Malheureusement, personne ne veut corriger le code en raison de la règle "ne pas y toucher" pour les anciens codes hérités. Au lieu de cela, la direction promet de lancer un nouveau projet de remplacement, mais pour l'instant, le code reste non corrigé.