È comune per chiunque ricevere più di 100 commenti nelle loro revisioni del codice su una base di routine? Direi di no È comune per le persone la cui qualità del codice "lascia molto a desiderare" per ottenere molti commenti, assolutamente.
Tuttavia, dipende anche dalle "regole" del processo di revisione del codice. TUTTI hanno le loro idee su come qualcosa dovrebbe essere fatto. Se il tuo processo di revisione del codice consente ai commenti di avere il formato "Dovresti farlo in questo modo invece che in quel modo", allora probabilmente otterrai MOLTISSIMI commenti anche per un codice adeguato. Se il tuo processo è destinato a trovare "difetti", il numero di commenti dovrebbe essere molto più piccolo.
Nella mia esperienza, le recensioni che consentono "suggerimenti" per metodi alternativi sono perdite di tempo. Questi "suggerimenti" dovrebbero essere gestiti uno a uno al di fuori del processo di revisione. Le revisioni dei difetti sono più utili in quanto portano le persone a concentrarsi sui bug invece di "perché non l'hai fatto come avrei fatto io?". È anche più utile perché non si può negare un bug se qualcuno lo trova. Quindi, non ci sono sentimenti feriti, ma probabilmente gratitudine.
AGGIORNAMENTO: Con tutto ciò che detto, un certo codice è semplicemente cattivo, anche se privo di difetti. In tal caso, il commento della recensione dovrebbe essere un singolo commento che dice qualcosa di simile. "Questo codice deve essere ripulito. Si prega di posticipare la revisione fino a quando il codice non viene discusso con [il tuo nome qui]". In tal caso, un'ulteriore revisione del codice dovrebbe interrompersi finché il commento non viene corretto.
UPDATE2: @User: discuti il tuo codice / design con uno di essi mentre lo stai sviluppando in modo da poter implementare ciò che stanno cercando prima di arrivare a molto a fare a modo tuo? Stai cambiando qualcosa su come stai sviluppando il codice in base ai loro suggerimenti o continui a pensare che la tua strada stia bene? Stai imparando qualcosa dai loro commenti?
Quando sono il responsabile di un progetto, è mio compito essere responsabile di TUTTI i prodotti di lavoro. Se approvo un prodotto di lavoro, dichiaro che il prodotto è accettabile. Voglio avere una reputazione per la costruzione di prodotti di qualità. Quindi, ho aspettative e non accetto meno che soddisfacente. Allo stesso tempo, cerco di insegnare e spiegare le ragioni delle mie preferenze. Queste preferenze potrebbero non essere sempre ideali (in particolare agli occhi degli altri), ma la maggior parte di queste preferenze deriva dall'esperienza. Di solito una reazione per evitare di ripetere quelle cattive. Pertanto, ci sono alcuni miei "stickler" personali che sono necessari per ottenere la mia approvazione, indipendentemente dal pushback.
Dall'altra parte, devi imparare le aspettative che sono necessarie per ottenere l'approvazione dei tuoi prodotti di lavoro. Non si può essere d'accordo, ma dal momento che non sembra avere l'autorità per sovrastimare, si impara cosa è previsto. Dubito che la squadra stia cercando di farti fallire. Anche questo li fa sembrare cattivi. A tal proposito, dimostra semplicemente che sei desideroso di imparare (anche se non lo sei), prendi ciò che dicono e fai del tuo meglio per adattarsi alle loro preferenze e probabilmente li vedrai arretrare un po '. Forse trova quello che puoi almeno tollerare e vedere se faranno un po 'di tenere in mano per insegnarti i loro modi. Chissà, nel processo potresti imparare qualcosa che potrebbe davvero portare le tue abilità al livello successivo.