Come puoi ottenere critiche costruttive per il tuo codice?

8

La mia squadra raramente esegue la revisione del codice, principalmente perché non abbiamo abbastanza tempo e le persone non hanno l'energia e la volontà di farlo. Ma mi piacerebbe davvero sapere cosa pensa la gente del mio codice quando lo leggono. In questo modo, ho una migliore comprensione di come le altre persone pensano e personalizzano il mio codice di conseguenza, quindi è più facile da leggere.

Quindi la mia domanda è: come posso ottenere critiche costruttive per il mio codice? Il mio intento è capire come pensano le persone in modo da poter scrivere un codice più leggibile.

    
posta burnt1ce 31.10.2012 - 16:14
fonte

4 risposte

8

Avendo lavorato in posti con recensioni di codice e senza, è diventato uno dei miei problemi di make-or-break nella ricerca di nuova occupazione. Il tempo risparmiato evitando le emergenze perché i problemi non sono emersi fino a quando non si arriva a prod è molto più alto del tempo speso nella revisione del codice. E questo non menziona quanto sia meno stressante trovare un problema nella revisione del codice.

Potresti iniziare in piccolo se la squadra ha bisogno di essere convincente. Dal momento che desideri rivedere il codice, inizia da lì. Chiedi a uno o più tuoi colleghi di incontrarti per un'ora o giù di lì e di esaminare alcuni frammenti di codice che ritieni desideri ricevere. Se il feedback è in gran parte negativo, non metterti sulla difensiva. Prendere appunti e prendere in considerazione le modifiche suggerite. Ma fallo su qualcosa che non hai ancora inviato a prod (o francamente non apporterai le modifiche). Puoi anche farlo in modo informale alla tua scrivania, basta chiamare qualcuno e dire "hey non sono sicuro di avere la soluzione migliore qui, cosa ne pensi?"

Un altro modo per convincere gradualmente le persone a iniziare a vedere il valore della revisione del codice è di avere una sessione di una volta alla settimana in cui ognuno deve presentare un pezzo di codice per la revisione (o si ruota attraverso ogni persona ma solo una settimana a seconda della complessità del tipo di codice che necessita di revisione). Portare ciambelle o bagel la prima volta! Se le persone si sentono a disagio nel dire a qualcuno di persona o se pensi che le persone siano troppo difensive, chiedi loro di mandare una email al capo e fargli confermare i commenti in modo che la persona sottoposta a revisione non sappia chi ha detto il codice. Francamente, preferisco sapere di persona chi ha detto cosa, perché la mia valutazione delle proprie capacità di codifica mi aiuterà a decidere quanto seriamente dovrei prendere le critiche.

Se non riesci a trovare qualcuno che controlli il tuo lavoro, siediti con te stesso e prova a spiegare il codice e perché stai facendo quello che stai facendo come se ci fosse qualcuno. Sono stupito di quanto spesso sia la persona che ha creato il codice che ha trovato il problema nel corso della spiegazione del codice. Aiuta anche a sedersi con il documento dei requisiti come una sorta di lista di controllo e assicurarsi che non manchi qualcosa che è richiesto.

    
risposta data 31.10.2012 - 19:06
fonte
5

Nessun tool di fantasia, a mio parere, potrebbe sostituire la seduta con uno sviluppatore o architetto senior e camminare attraverso il codice (o il loro codice, e vedere cosa hanno fatto di prima mano). Detto questo, ho trovato che il codice di test delle unità ti costringe a pensare in modo riutilizzabile, ti costringe anche a incorporare schemi che potrebbero rendere il tuo codice più leggibile (con l'ulteriore vantaggio di un codice migliore).

    
risposta data 31.10.2012 - 16:19
fonte
2

Hai buoni strumenti per le revisioni del codice? Abbiamo avuto un problema simile nel mio gruppo. Ho impiegato del tempo per installare un server Reviewboard e la partecipazione alle revisioni del codice è saltata. Ho scoperto che alle persone piace commentare il codice quando è stato semplificato.

    
risposta data 31.10.2012 - 16:17
fonte
2

Dovresti davvero prendere il tempo di fare revisioni del codice, perché nella mia esperienza vale la pena per la qualità del tuo codice. Se la revisione del codice non è un'opzione del tuo team e vuoi capire come pensa la gente, forse la programmazione della coppia è un'opzione per te. Ottieni un feedback immediato su ciò che stai digitando - qualcosa come una recensione immediata.

    
risposta data 31.10.2012 - 16:24
fonte

Leggi altre domande sui tag