Prioriser les changements dans le code JavaScript

Question

J'ai acquis la tâche de mettre à jour un code JavaScript horriblement obsolète que je n'ai pas écrit, plein de hacks et pièges, éparpillés dans les commentaires. En plus de faire fonctionner le changement, j'aimerais rendre le script plus facile à comprendre et plus robuste (par exemple, ne pas utiliser de variables globales).

Je pense que j'ai été capable de me concentrer sur la plupart de son exécution, donc je pense que je suis prêt à commencer à faire des changements dessus.

Ma question est: quel temps limité, quelles pièces devrais-je prioriser de changer? Dois-je d'abord essayer de tout encapsuler dans les espaces de noms? Dois-je créer des classes pour tous mes objets? Devrais-je remplacer tous les appels .innerHTML par leurs équivalents DOM? Dois-je "utiliser jQuery"? Selon vous, quelles sont les pièces les plus importantes pour maximiser les améliorations du code?

Answer 1

1. Obtenez Hacks Cross navigateur de votre code. Peu importe la bibliothèque que vous utilisez pour cela, mais ils vont le faire mieux que vous. Le plus important ici est probablement la gestion des événements inter-navigateurs, mais il y a d'autres problèmes de DOM, et l'utilisation de quelque chose comme jQuery peut vraiment se débarrasser de certaines choses standard.
2. Si le code utilise innerHTML, il génère probablement aussi des chaînes html. innerHTML n'est vraiment pas une mauvaise chose autant que générer le code HTML car il peut être très laid. Regardez dans une simple solution de modélisation JS.
3. En chemin vers les deux premiers, vous trouverez probablement qu'il est logique de rendre le code plus orienté objet/namespaced, et vous pouvez probablement le faire comme vous allez. S'il reste quelque chose, vous pouvez le faire quand les autres auront fini.

Answer 2

Personnellement, je commencerais par namespacing et éventuellement des classes. De cette façon, si votre temps est limité et que vous devez partir et revenir au code, il sera mieux organisé et donc plus facile pour vous ou quelqu'un d'autre de le ramasser et de le nettoyer plus avant.

Answer 3

Refactor le fruit lowhanging: http://c2.com/cgi/wiki?RefactorLowHangingFruit