2010-11-05 12 views
14

Quali sono le mie migliori opzioni per condurre una revisione del codice poiché sono un 'esercito di 1' al lavoro? Il meglio che ho fatto finora è eseguire Sonar sui miei progetti, ma sto pensando che si potrebbe fare di più.Revisione del codice quando si lavora da solo

Mi piacerebbe andare oltre la pubblicazione e chiedere feedback su piccoli blocchi di codice. Grazie.

+0

check out http://area51.stackexchange.com/proposals/11464/code-review possiamo ottenere la palla rotolante se otteniamo abbastanza commit per questo. – greatwolf

risposta

9

Una tecnica che si potrebbe provare è eseguire un diff su ciascuno dei file di origine prima di controllarli. Basta attivare lo strumento di differenziazione su ciascun file sorgente e scorrere le modifiche al codice. Prendetevi un momento per riflettere su ogni modifica proprio come si farebbe se si stesse rivedendo il codice di un altro sviluppatore, e brevemente chiedetevi:

  • Che cosa significa questo cambiamento?
  • Come si riferisce al bug/miglioramento/richiesta originale?
  • Il nuovo codice fa realmente ciò che era previsto?
  • Vedo bug?
  • ecc ...

Dopo un po 'si otterrà l'abitudine di correre attraverso queste recensioni veloci codice, e in grado di garantire che almeno un'occhiata a ciascuno di modifiche più di una volta.

Concesso, questo metodo funziona meglio quando si mantiene il codice e si apportano piccole modifiche: è più difficile quando vengono modificati grandi blocchi di codice, ma questo livello di disciplina è ancora valido.

+1

Sono d'accordo sul fatto che seguire quella raccomandazione può essere utile. Potrebbe forse diventare qualcosa di più di una cosa che fai come parte di una seconda natura se hai il codice sotto controllo di versione e hai l'abitudine di eseguire i tuoi cambiamenti attraverso un programma diff, prima di commetterli. So che per me, anche quando so che cosa ha apportato esattamente i cambiamenti, ho anche verificato almeno una volta l'output del diff prima di eseguire le modifiche. – ayaz

1

Soluzione non tecnica: forse potresti cercare un altro "esercito di 1" e un codice di scambio?

In alternativa, rendere generali le parti open source, caricarle e attendere segnalazioni di bug (offrire risultati migliori).

+1

Dubito del "probabilmente": la tua "parte generale" deve essere significativamente utile per altri codificatori, e questo non è naturalmente il caso per la maggior parte del codice. – gimpf

+0

Vorrei se potessi, qualche suggerimento? – vector

+0

Secondo me l'approccio open source funzionerebbe meglio per progetti indipendenti con qualche valore per gli altri. Al lavoro è molto un tipo di "ho-ham" :-( – vector

1

Potrebbe utilizzare uno strumento come Resharper di FxCop.

+0

... fantastici strumenti, per me sebbene sia Java o PHP per ora, grazie comunque – vector

4

Ci sono comunità di revisione del codice là fuori, vedere il mio vecchio question per alcuni esempi.

+0

+1 Per salvare l'apertura della 'domanda', c'è una sola risposta con questo sito: http: //www.refactormycode Com'è bello –

+0

@Chris Knight - Ma ovviamente inventa comunque la mia domanda :) – willcodejavaforfood

+0

... anzi, refactormycode potrebbe farlo :-), ci proveremo sicuramente! – vector

1

Nelle domande non è stato specificato, ma se si sta utilizzando Java, una configurazione Checkstyle controllata ti aiuterà a rimanere dritta e stretta, oltre a identificare le aree in cui il tuo design è debole e il refactoring sarebbe vantaggioso.

+0

... è Java e sto usando NB ma il plugin checkstyle è davvero vecchio, pensavo che Sonar facesse la stessa cosa. – vector

1

Basta rivedere le differenze, dopo un giorno come un giorno, utilizzando uno strumento di revisione del codice può aiutare a identificare, tracciare e risolvere i problemi.

Quando si sta esaminando è necessario mettere il cappello "revisore" e cercare di criticare le modifiche nel miglior modo possibile.

Nella mia esperienza personale, la revisione del codice può essere efficace quanto il test dell'unità di auto nell'identificazione dei problemi.

Nota: sono associato all'azienda che crea il prodotto collegato sopra.

+1

Si prega di consultare [FAQ] (http://stackoverflow.com/faq#promotion). "Inoltre, se un'enorme percentuale dei tuoi post include una menzione del tuo prodotto, sei chiaramente qui per i motivi sbagliati." 13 su 14 è piuttosto alto, in particolare per un nuovo utente. I tuoi post sono già contrassegnati come spam, quindi prova a contribuire in altri modi. –

Problemi correlati