2010-09-09 3 views
11

Je suis en train de mettre en œuvre un processus de révision de code, parmi les choses à examiner est javascript. On m'a demandé de faire une sorte de liste de contrôle des choses à rechercher lors de l'examen du code. Jusqu'à présent ma liste de contrôle est la suivante:Ce qui est sur votre liste de contrôle de révision de code javascript

  • aucune fonction ou variable dans l'espace de noms global .
  • écrire du code efficace, en particulier dans les boucles
  • ne pas ajouter à dom dans une boucle.
  • garder le style en CSS, utiliser les classes. Utilisez le meilleur sélecteur jQuery possible.
  • Ne sélectionnez pas un objet dans jquery plus d'une fois, utilisez le chaînage ou placez-le dans une variable.
  • minimiser les requêtes http.
  • n'utilisez pas jQuery $ .each.
  • valider l'entrée de l'utilisateur avant de faire demandes http.
  • gérer des erreurs spécifiques.
  • fournir à l'utilisateur assez d'informations pour remédier à la situation en cas d'erreur se produire (si possible)

Quelqu'un at-il tout ce qu'ils vous recommande une liste de contrôle de révision générale javascript (jQuery inclus).

Répondre

1

Je voudrais ajouter quelque chose comme "distinguer entre les erreurs non fatales et fatales". Cela améliore beaucoup l'expérience de l'utilisateur. Une autre chose à laquelle je pense est l'utilisation de fermetures pour minimiser la pollution des espaces de noms. Mon troisième point: la détection des fonctionnalités sur le reniflage du navigateur.

1

Aucune erreur de JSLint en utilisant les paramètres que vous souhaitez. Cela peut vérifier les choses que vous avez mentionnées, comme ne rien ajouter à l'espace de noms global.

Questions connexes