Avvio del processo di revisione del codice Dove prima c'era nessuno? [duplicare]

20

Qualche suggerimento per un gruppo che ha bisogno di iniziare le revisioni del codice? Al momento non abbiamo processi di ingegneria del software reali. Ho lavorato come software engineer per 7 anni e non ho mai visto processi reali implementati nei vari luoghi in cui ho lavorato, sebbene molti libri sposino tutti i tipi di metodologie ingegneristiche. La direzione supporta l'idea di attivare nuovi processi, anche se c'è una certa resistenza da parte dell'ingegnere senior che ha oltre 20 anni di esperienza nell'ingegneria del software.

Siamo un gruppo di due programmatori, senza alcun responsabile dell'ingegneria del software. Segnaliamo direttamente all'amministratore delegato della Società. Ma lavoriamo su numerosi progetti software. Il vero motivo per cui dobbiamo iniziare ad eseguire le revisioni del codice è a causa dei requisiti PA-DSS. Pertanto, sto spingendo per noi a fare revisioni del codice in tutto - non solo sul software di applicazione di pagamento.

    
posta Brian 11.11.2010 - 15:26
fonte

3 risposte

6

Dato che ci sono solo due di voi, in teoria non avete davvero bisogno di un processo di revisione formale. Quello che dovrebbe accadere è che quando uno di voi finisce un pezzo di codice, dovrebbe semplicemente dire all'altro: "Ecco cosa ho fatto. Per favore, date un'occhiata".

Questo, naturalmente, presuppone che entrambi siete ragionevoli, sufficientemente disciplinati e che entrambi avete una presa sufficiente sul proprio ego. Se questi presupposti sono falsi, allora è necessario un processo formale applicato dall'alto. La cosa più semplice da fare è fare una regola che tutti i commit o le fusioni nel bagagliaio devono essere guardati e approvati dall'altra persona, o almeno da un'altra persona, nel caso in cui la tua squadra cresca.

Questo non si applica ancora a te, ma nella mia esperienza le revisioni formali del codice da parte di un team di 4 persone finiscono per richiedere troppo tempo e il 90% dei commenti dei singoli revisori è lo stesso. Ecco perché siamo passati a questo sistema con un solo revisore.

    
risposta data 11.11.2010 - 15:44
fonte
7

Vuoi che le recensioni del codice siano un esercizio positivo! Ecco alcuni suggerimenti:

  • Esame del codice anticipato dalla programmazione di coppie! Anche se questo è solo per 30 minuti al giorno.

  • Usa strumenti di analisi del codice statico, controlli di stile e modelli per sbarazzarti del fluff. È quindi possibile concentrarsi sui problemi più complessi (SRP, schemi di progettazione, gestione delle eccezioni, codifica su interfacce, ecc.), Che è ciò che le revisioni del codice dovrebbero riguardare (IMHO).

  • Ottieni un elenco di controllo concordato tra te su ciò che è importante e ciò che non lo è

  • Una sorta di approccio Test Driven (costringe a migliorare la progettazione iniziale) è raccomandato più in basso nella traccia

  • Ci sono alcuni buoni strumenti per la revisione del codice - utilizzo Crucible personalmente

  • Assicurati che l'atteggiamento di 'migliorare il codice' contrapponga a 'crticise the coder'. Questo è fondamentale per mantenere felice l'ego di tutti.

  • Fatelo regolarmente, non lesinare su di esso. O dopo un gruppo di commit o 1 / giorno o qualsiasi altra cosa su cui sei d'accordo.

Buona fortuna!

    
risposta data 11.11.2010 - 16:55
fonte
4

Il modo più semplice per iniziare la revisione del codice: basta iniziare a farlo con uno strumento di revisione open source. link Puoi iniziare da un processo semplice: ogni giorno un codice di valutazione per sviluppatori del secondo sviluppatore e viceversa. Se la recensione ti aiuterà a migliorare il tuo codice, allora potresti pensare a "processo tecnico reale"

    
risposta data 11.11.2010 - 15:45
fonte

Leggi altre domande sui tag