Quando eseguo le revisioni del codice tendo ad avere solo un monologo in esecuzione, così come sto dando un senso a ciò che sto leggendo ci sarà un sacco di "Ok, vedo cosa fa .. Buono si connette a questo e lo chiama, va bene ... e quel pezzo dipende da entrambi. ".
Penso che in questo modo non sia "oo la la è così bello!", potrebbe essere un codice noioso e banale, ma ascoltare qualcun altro effettivamente analizzare e mostrare la comprensione di ciò che hai scritto è una forma di feedback positivo dentro e di per sé, il feedback è "Questo codice ha senso", quando mi imbatto in parti che non capisco chiedo spiegazioni e quando lo capisco esclamano "Ah, ho capito".
Penso che il semplice trasferimento di comprensione sia un elogio per un altro ingegnere perché tutti vogliamo che il nostro codice sia compreso dagli altri, fornisce una forma di convalida implicita.
Detto questo, se vedi parti del codice che sono caratteristiche positive o positive (anche il noioso codice banale può essere buono se è la forma minima di se stesso), io tendo decisamente a indicare quelle caratteristiche, di nuovo non le attribuisco come "Wow fantastico!" così come "Vedo che questa è un'implementazione minima" o "Ok, questo algoritmo complesso ha molti commenti", si concentra sugli attributi del codice non tanto per la sua intrinseca bontà o cattiveria.
Ogni volta che attribuisci "bontà" o "cattiveria" al codice in una revisione del codice per evitare di far sentire l'ingegnere guardato dall'alto in basso o tenuto su un piedistallo, non dire che qualcosa è buono o cattivo, ma piuttosto parlare attraverso la causa ed effetto del loro codice.
"Ok questa parte ha senso, ah c'è un numero magico qui, il significato di quel valore potrebbe non essere ben compreso dal prossimo ingegnere per toccare questo"
"Vedo che hai un contenitore DI qui ok, quindi avrai un accoppiamento lento con quel repository"
"Ah c'è un dizionario statico qui, se più thread stanno toccando quel dizionario potremmo imbattersi in alcune condizioni di gara"
Notate che non sto dicendo che tutto sia buono o cattivo, ma se l'ingegnere dovrebbe cambiarlo o no sarà capito dall'ingegnere il cui codice è in fase di revisione. Ovviamente devi terminare la revisione del codice con un yay o un nay, ma l'accumulo di queste affermazioni nel corso di esso ammorbidirà le noia come la spiegazione è già stata fatta sotto forma di dichiarazioni di causa ed effetto quando dici loro "Vorrei quei numeri magici sono stati risolti prima di verificarlo in ".