2010-09-09 8 views
11

Sto implementando un processo di revisione del codice, tra le cose da rivedere è javascript. Mi è stato chiesto di fare una sorta di lista di controllo delle cose da cercare durante la revisione del codice. Finora la mia lista di controllo è la seguente:Cosa c'è nella lista di controllo per la revisione del codice javascript

  • nessuna funzione o variabile nello spazio dei nomi globale .
  • codice di scrittura efficiente, in particolare nei loop
  • non aggiungere al dom in un ciclo.
  • mantieni lo stile in css, usa le classi.
  • utilizza il miglior selettore jQuery possibile.
  • Non selezionare un oggetto in jQuery più di una volta, utilizzare il concatenamento o inserirlo in una variabile.
  • minimizza le richieste http.
  • non utilizzare jQuery $ .each.
  • convalidare l'input dell'utente prima di effettuare le richieste http .
  • gestisce errori specifici.
  • fornire all'utente abbastanza informazioni per porre rimedio alla situazione in cui gli errori verificare (se possibile)

Qualcuno ha qualsiasi altra cosa che vi consiglio per un generale javascript recensione lista di controllo (jQuery inclusa).

risposta

1

vorrei aggiungere qualcosa come "distinguere tra errori fatali non-fatale e". Ciò migliora molto l'esperienza dell'utente. Un'altra cosa a cui sto pensando è l'uso di chiusure per ridurre al minimo l'inquinamento dello spazio dei nomi. Il mio terzo punto: Rilevamento delle funzionalità rispetto allo sniffing del browser.

1

Nessun errore da JSLint utilizzando qualsiasi impostazioni che si decide che si desidera. Questo può verificare la presenza di cose che hai citato, ad esempio aggiungere nulla alla namespace globale.

Problemi correlati