Come integrare le recensioni del codice in una squadra [duplicato]

1

Lavoro in una squadra che continua a crescere. Tuttavia, non ci sono revisioni del codice e vedo questo come un problema ogni giorno. Ci sono persone con capacità veramente limitate che commettono codice nel nostro repository che credo non dovrebbe impegnare.

Non sono il caposquadra né sono in posizione dirigenziale. Ho già proposto le revisioni del codice circa 6 mesi fa e non sono stati ricevuti feedback. Né ci sono state recensioni di codice.

So che le recensioni del codice potrebbero avvantaggiare tutti, compresi i più esperti. Qualcuno ha avuto qualche esperienza nel tentativo di integrare le recensioni del codice in una squadra? Che cosa hai proposto e come l'hai fatto?

    
posta DogDog 30.04.2013 - 19:03
fonte

3 risposte

3

Guida con l'esempio, anche se non sei il leader. Il primo passo potrebbe essere per te iniziare a chiedere agli altri membri del team di rivedere il tuo codice. Anche se non esiste una politica ufficiale della squadra, inizia a farlo. Se ci sono persone che hanno esperienza pari o inferiore a te, offri di rivedere il loro codice per loro. Anche se la squadra nel suo complesso non adotta la pratica, almeno raccoglierà personalmente i benefici della revisione tra pari.

Potresti anche esaminare alcuni degli strumenti di revisione degli scaffali e provare a configurarne e installarne uno sulla scatola o su un server comune. Quindi, potresti offrire di dimostrarlo a tutti durante il pranzo.

    
risposta data 30.04.2013 - 19:19
fonte
0

In qualità di contributore finale, le tue opzioni sono limitate. Il meglio che puoi fare è cercare di capire cosa è importante per la gestione e mostrare come la tua soluzione (revisioni del codice) ha un impatto. Vedi anche "Troppo orientato agli oggetti" e Che cosa fare quando i tuoi colleghi non valutano la manutenibilità del codice [duplicato] .

Potresti considerare di cercare in giro per un'altra posizione. La vita è troppo breve per sprecare nei negozi di cianfrusaglie se non sei in grado di migliorare le cose.

    
risposta data 30.04.2013 - 19:18
fonte
0

Tutti sono d'accordo sul loro valore, ma è difficile ottenere un consenso su come ottenerli.

Se non ci sono standard di codifica sul posto, ti rimane uno standard de facto o POV.

Potresti avere uno sviluppatore che fa molto caldo sui modelli di design, mentre un altro scrive la sua nuova lingua nello stile di un vecchio con cui si è sviluppato.

Inoltre - il cui codice viene revisionato? Se è solo uno staff junior, può sentirsi offeso, come lo è il personale più anziano che deve prendersi del tempo per farlo. Il vero valore arriva quando è 360. Gli sviluppatori junior possono evitare alcune trappole mentre gli sviluppatori senior possono offrire la loro ricchezza di esperienza.

Mi rendo conto che nessuna delle risposte precedenti risponde direttamente "come iniziare". Ma se non puoi venderlo o suggerire un quadro di base, non hai alcuna possibilità che venga preso.

    
risposta data 30.04.2013 - 21:28
fonte

Leggi altre domande sui tag