2010-09-09 4 views
11

Ich bin ein Code-Review-Prozess implementieren, unter den zu überprüfenden Dingen ist Javascript. Ich wurde gebeten, eine Art Checkliste mit Dingen zu erstellen, nach denen ich beim Überprüfen des Codes suchen sollte. Bis jetzt meine Checkliste ist wie folgt:Was ist auf Ihrem Javascript-Code Überprüfung Checkliste

  • keine Funktionen oder Variablen im globalen Namensraum.
  • schreiben Sie effizienten Code, vor allem in Schleifen
  • nicht an die Dom in einer Schleife anhängen.
  • Stil in CSS beibehalten, Klassen verwenden.
  • Verwenden Sie den besten jQuery-Selektor möglich.
  • Wählen Sie ein Objekt in jquery nicht mehr als einmal aus, verwenden Sie eine Verkettung oder fügen Sie es in eine Variable ein.
  • HTTP-Anfragen minimieren.
  • Verwenden Sie nicht jQuery $ .Each.
  • Benutzereingaben überprüfen, bevor HTTP-Anfragen erstellt werden.
  • behandeln spezifische Fehler.
  • bieten dem Anwender genügend Informationen, um Abhilfe die Situation, wenn Fehler (wenn möglich) auftreten

Hat jemand alles, was sie für eine allgemeine javascript Bewertung Checkliste empfehlen würde (jQuery enthalten).

Antwort

1

Ich würde etwas wie "zwischen nicht-tödlichen und fatalen Fehlern unterscheiden" hinzufügen. Dies verbessert die Benutzererfahrung sehr. Eine andere Sache, über die ich nachdenke, ist die Verwendung von Schließungen, um die Namensraumverschmutzung zu minimieren. Mein dritter Punkt: Feature-Erkennung über Browser-Sniffing.

1

Keine Fehler von JSLint mit den Einstellungen, die Sie entscheiden möchten. Dies kann nach Dingen suchen, die Sie erwähnt haben, z. B. dass Sie dem globalen Namespace nichts hinzufügen.