È comune per chiunque ottenere più di 100 commenti nelle proprie recensioni di codice su base regolare? 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. Ognuno ha le proprie idee su come qualcosa avrebbe dovuto essere fatto. Se il tuo processo di revisione del codice consente ai commenti di essere nel formato "Dovresti farlo in questo modo invece che in quel modo", probabilmente otterrai MOLTI commenti anche per un codice adeguato. Se il tuo processo ha lo scopo di trovare "difetti", il numero di commenti dovrebbe essere molto più piccolo.
Nella mia esperienza, le recensioni che consentono "suggerimenti" per metodi alternativi sono perditempo. Tali "suggerimenti" dovrebbero essere gestiti uno a uno al di fuori del processo di revisione. Le recensioni di difetti sono più utili in quanto inducono le persone a concentrarsi sugli errori invece di "perché non l'hai fatto come l'avrei fatto?". È anche più utile perché non si può negare un bug se qualcuno lo trova. Quindi, non ci sono sentimenti feriti ma probabilmente gratitudine.
AGGIORNAMENTO: Detto questo, alcuni codici sono semplicemente cattivi, anche se privi di difetti. In tal caso, il commento della recensione dovrebbe essere un singolo commento che dice qualcosa di simile. "Questo codice deve essere ripulito. Ti preghiamo di rimandare la revisione fino a quando il codice non verrà discusso con [il tuo nome qui]." In tal caso, un'ulteriore revisione del codice dovrebbe interrompersi fino alla correzione del commento.
UPDATE2: @User: Discuti il tuo codice / progetto con uno di essi mentre lo stai sviluppando in modo da poter implementare ciò che stanno cercando prima di arrivare lontano facendo 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 vada bene? Stai imparando qualcosa dai loro commenti?
Quando sono il capofila di un progetto, è mio compito essere responsabile di TUTTI i prodotti di lavoro. Se approvo un prodotto di lavoro, allora sto sostenendo che il prodotto è accettabile. Voglio avere una reputazione per la costruzione di prodotti di qualità. Quindi, ho aspettative e non accetterò meno che soddisfacente. Allo stesso tempo, cerco di insegnare e spiegare i motivi delle mie preferenze. Tali preferenze potrebbero non essere sempre ideali (in particolare agli occhi degli altri), ma la maggior parte di tali preferenze deriva dall'esperienza. Di solito una reazione per evitare di ripetere quelli cattivi. Quindi, ci sono alcuni dei miei "stickler" personali che sono necessari per ottenere la mia approvazione, indipendentemente dal pushback.
Dall'altro lato, è necessario conoscere le aspettative necessarie per ottenere l'approvazione dei prodotti di lavoro. Puoi non essere d'accordo, ma poiché non sembri avere l'autorità di sovrastare, allora impara cosa ci si aspetta. Dubito che il team stia cercando di farti fallire. Poiché ciò li rende anche cattivi. A tal proposito, dimostra solo che sei desideroso di imparare (anche se non lo sei), prendi quello che dicono e fai del tuo meglio per adattarti alle loro preferenze e probabilmente li vedrai indietreggiare un po '. Forse trova quello che puoi almeno tollerare e vedere se faranno un po 'di mano per insegnarti i loro modi. Chissà, nel frattempo potresti imparare qualcosa che potrebbe davvero portare le tue abilità al livello successivo.