Come gestivi le situazioni in cui non eri d'accordo con la valutazione della revisione del codice?

Quando sorgono disaccordi nella valutazione di una revisione del codice, applico il seguente approccio:

1. Studio attentamente il commento del revisore: Cerco di capire la ragione principale della sua osservazione e la proposta di miglioramento. Potrei aver trascurato qualcosa o il suo punto di vista si basa su un’esperienza diversa.

2. Analizzo la mia posizione: Rivedo la mia decisione iniziale e argomento perché ho scelto quell’approccio. È importante formulare chiaramente i miei argomenti.

3. Discuto direttamente l’osservazione: Invece di essere passivamente in disaccordo, avvio una discussione con il revisore. Questo può essere un messaggio sincronizzato, un incontro o un commento nel sistema di revisione del codice. Il mio obiettivo non è "vincere" la discussione, ma arrivare a una soluzione ottimale.

4. Propongo soluzioni alternative: Se la mia soluzione non soddisfa, sono disposto a considerare altre opzioni proposte dal revisore o a sviluppare congiuntamente un nuovo approccio che tenga conto di entrambi i punti di vista.

5. Se necessario, coinvolgo una terza persona: Se dopo aver discusso non si raggiunge un consenso, posso consultare un altro membro del team o il team leader per un’opinione indipendente.

Esempio di discussione nel sistema di revisione del codice:

// Il mio commento sull’osservazione
// Ciao! Grazie per la revisione. Capisco il tuo punto di vista sull’uso di this.findViewById.
// Ho scelto questo approccio per motivi di prestazioni, dato che qui la gerarchia delle viste è piccola.
// Puoi spiegare perché, in questo caso specifico, suggerisci di usare ViewBinding?

Esempio di discussione se il revisore propone un’ottimizzazione che sembra eccessiva:

Presenterò i dati in forma tabellare, illustrando l’impatto dell’ottimizzazione proposta sulle prestazioni in questo caso specifico rispetto al tempo dedicato alla sua implementazione.

Ottimizzazione	Potenziale aumento delle prestazioni	Tempo di implementazione
Soluzione iniziale	Base	0
Ottimizzazione proposta	Minimo (circa 1-2 ms)	1 ora

In definitiva, il mio obiettivo principale è migliorare la qualità del codice e renderlo migliore, non semplicemente difendere il mio punto di vista. Sono aperto a critiche costruttive e disposto a cambiare la mia decisione se vengono presentati argomenti convincenti.