Strumenti di revisione del codice: da usare o no? [chiuso]

0

Nel mio team di sviluppo, stiamo eseguendo revisioni di codice, tuttavia non in modo corretto, credo.

I problemi di cui il nostro processo soffre:

  1. Non viene assegnato tempo sufficiente per una corretta revisione del codice.
  2. Fare recensioni non è obbligatorio - molte volte semplicemente non viene fatto.
  3. Gli sviluppatori siedono insieme per le recensioni, a causa della mancanza di un altro meccanismo semplice per farlo "offline" senza passare il tempo di entrambi gli sviluppatori.

La mia domanda è: l'integrazione di uno strumento per le revisioni del codice può migliorare i punti sopra menzionati? Non è necessario?

Mi piacerebbe sentire da esperienze positive / negative.

    
posta liortal 17.10.2013 - 23:29
fonte

2 risposte

4

Applicando effettivamente le revisioni del codice con strumenti come gerrit o Crogiolo è possibile risolvere la parte 2 e 3 del problema. La prima parte è più un problema di processo / sociale che non dovrai mai risolvere con il software.

    
risposta data 17.10.2013 - 23:44
fonte
4

No, non stai facendo correttamente le revisioni del codice (ovviamente!). Quindi la domanda rimane se c'è qualcosa di sbagliato nel tuo codice che ha bisogno di sessioni di revisione? (cioè se stai spedendo codice funzionante, le revisioni del codice sono una soluzione alla ricerca di un problema).

Se pensi che sarebbe utile, allora inizierei a lavorare con uno strumento di revisione del codice che puoi utilizzare in modalità "come e quando" - quindi carichi una richiesta di revisione del codice ma non si ferma il resto il processo di commit e costruzione di codice da accadere. Penso che se hai aggiunto la recensione al processo del flusso di lavoro che ha impedito i commit fino a dopo la revisione, finiresti per frustrarti e aggirare le recensioni. Sii onesto con te stesso in questo caso.

Ma se riesci a leggere il codice quando hai tempo, diventa un buon modo per mostrare cosa hai fatto per risolvere un compito di lavoro e puoi sempre correggere retroattivamente i difetti di revisione del codice sollevando un errore successivo.

Gli strumenti per la revisione del codice sono ReviewBoard - un sito Web di base Python. Carichi diffs al sito e puoi quindi assegnarli e commentarli. VMWare lo ha sviluppato per i propri scopi. Gerrit funziona bene se usi git. Redmine ha un buon plug-in di revisione del codice che consente recensioni basate su qualsiasi commit, basta creare una nuova recensione come solo un altro ticket tracker . È piacevole da usare e non ti ostacola.

    
risposta data 17.10.2013 - 23:46
fonte