Bene, tendo a fare commenti in diverse aree generali e ogni tipo potrebbe essere gestito in modo diverso.
Modifiche richieste. Questi sono i tipi di modifiche in cui sottolineo che il codice non soddisfa i requisiti funzionali o non funziona e deve essere risolto prima di essere trasferito alla produzione. Tendo ad essere molto semplice in questi commenti. I requisiti dicono ..., questo non lo fa. O questo fallirà se il valore inviato è nullo (specialmente quando saprai che il caso si verificherà in base ai dati che verranno inviati).
Poi ci sono i "questo funziona ma qui c'è un modo migliore per realizzare quel" commento. Devi essere più gentile in questi e fare più di un passo di vendite. Potrei dire che lo farei invece perché è probabile che sia più performante (in genere riesamina il codice SQL in cui le prestazioni sono molto importanti). Potrei aggiungere alcuni dettagli sul perché è una scelta migliore, proprio come farei per rispondere a una domanda su Stack Overflow. Potrei sottolineare che non è necessario cambiarlo per questo particolare codice, ma considerare il cambiamento nella codifica futura. Fondamentalmente con questi tipi di commenti sto educando le persone con meno esperienza su cosa potrebbe funzionare meglio.
Poi ci sono i commenti "questo funziona ma facciamo le cose in questo modo". Probabilmente saranno necessari anche questi cambiamenti. Includerebbero commenti sugli standard aziendali o sull'architettura che ci aspettiamo che usino. Vorrei fare riferimento al documento standard o di architettura e dire loro di fissare lo standard. Il commento sarebbe semplice ma neutrale, manca così e così o i nomi delle variabili non sono conformi al nostro standard di denominazione o alle cose simili. Ad esempio, la nostra architettura per i pacchetti SSIS richiede che il pacchetto utilizzi il nostro database di metadati per archiviare particolari informazioni sul pacchetto e richiede una registrazione particolare. Il pacchetto funzionerebbe senza queste cose, ma sono necessarie per motivi aziendali (dobbiamo riportare il tasso di successo delle importazioni ad esempio o analizzare i tipi di errori che riceviamo).
L'unica cosa che non vuoi fare nei commenti di revisione del codice è attaccare personalmente qualcuno. Può anche aiutare se trovi qualcosa che hanno fatto bene e fai notare che era buono. A volte imparo qualcosa di nuovo da una revisione del codice e se lo facessi lo dico alla persona.