In base alla mia esperienza, la maggior parte dei team dell'azienda esamina il codice del membro del team con una piccola quantità di codice, sempre meno di centinaia di righe. È opportuno rivedere una grande quantità di codice, ad esempio un modulo, quando il codice è completo e pronto per essere riesaminato una volta per tutte?
La ragione delle piccole revisioni del codice è quella di massimizzare l'efficacia.
Gli studi che riguardano il Processo software personale hanno rilevato che i revisori sono i più efficaci, in relazione alla massimizzazione dei difetti identificati nel revisione, quando riesaminano non più di 150-200 righe di codice sorgente all'ora. Dato che i dati empirici, diventa una questione di determinare per quanto tempo le persone possono stare attenti. Non ho dati empirici, ma so che la mia mente inizia a vagare dopo circa 1 ora di lettura di qualcosa. Per me, questo indica che le revisioni del codice dovrebbero rivedere meno di 150 righe di codice alla volta.
È appropriato se aiuta il tuo team a creare un prodotto migliore.
Alcuni motivi per cui le revisioni del codice si concentrano spesso su blocchi più piccoli:
Esaminare correttamente una grande quantità di codice richiede che ogni partecipante impieghi molto tempo a prepararsi, e che può essere costoso.
Più a lungo una riunione di revisione (qualsiasi riunione, davvero), meno persone prestano attenzione. Due ore equivalgono a quasi quanto la maggior parte della gente può sopportare, e tenere riunioni più brevi di quelle (ad esempio, un'ora o 90 minuti in alto) è una lunga strada per assicurarsi che tutti siano al loro meglio.
Spesso non è necessario rivedere ogni riga. Una ragione per rivedere il codice è assicurarsi che tutti si attengano fondamentalmente allo stesso insieme di linee guida per la codifica. Una revisione dettagliata di una piccola quantità di codice funziona meglio a tale scopo rispetto a una revisione meno dettagliata di più codice.
Detto questo, se trovi che rivedere più codice è utile, e se la tua squadra è in grado di sopportarlo, prova a provarlo. Aiuterà ad assicurarsi che ci siano alcune regole di base e qualcuno che agisca da facilitatore per mantenere la riunione in movimento. Inoltre, considera che i revisori devono inviare i loro commenti in anticipo in modo da dedicare meno tempo a cose pignoli come l'uso improprio dello spazio bianco e più tempo a discutere su problemi importanti e interessanti.
Se rivedi una grande quantità di codice una volta che è già completo (e presumibilmente funzionante), questo si trasformerà più in una sessione di "avremmo dovuto fare ..." che "risolviamo / cambia questo da fare. .. "
Quindi, se lo scopo principale dell'attività è come un esercizio di apprendimento post-mortem, allora la revisione del codice di grandi dimensioni ha senso. Se è per cercare di catturare i difetti, probabilmente non funzionerà molto bene. È probabile che manchino piccoli difetti poiché tutti nella revisione si annoiano e sono stanchi, e grandi difetti strutturali probabilmente verranno saltati da quando, beh, "troppo tardi ora - il codice è scritto e il cliente lo vuole ieri."
Leggi altre domande sui tag code-reviews