Обычно ли кто-то получает более 100 комментариев в своих обзорах кода на регулярной основе? Я бы сказал нет. Обычно люди, чье качество кода «оставляет желать лучшего», получают много комментариев, абсолютно.
Однако это также зависит от «правил» процесса проверки кода. У ВСЕХ есть свои идеи о том, как что-то должно было быть сделано. Если ваш процесс проверки кода позволяет комментариям иметь форму «Вы должны делать это таким образом, а не таким образом», то вы, вероятно, получите МНОГО комментариев даже для адекватного кода. Если ваш процесс предназначен для поиска «дефектов», то количество комментариев должно быть намного меньше.
По моему опыту, обзоры, которые позволяют «предложения» для альтернативных методов, являются пустой тратой времени. Эти «предложения» должны обрабатываться один на один вне процесса рассмотрения. Обзоры дефектов более полезны, так как заставляют людей сосредоточиться на ошибках, а не на том, «почему вы не сделали это так, как я бы сделал?». Это также более полезно, потому что нельзя отрицать ошибку, если кто-то ее находит. Таким образом, нет никаких обид, а скорее всего благодарности.
ОБНОВЛЕНИЕ: С учетом всего вышесказанного, некоторый код просто плохой, даже если без дефектов. В этом случае комментарий должен быть одним комментарием, который говорит что-то вроде. «Этот код необходимо очистить. Пожалуйста, отложите проверку до тех пор, пока код не будет обсужден с [вашим именем здесь]». В этом случае дальнейшее рассмотрение кода должно быть прекращено до исправления комментария.
ОБНОВЛЕНИЕ2: @User: Обсуждаете ли вы свой код / дизайн с одним из них во время его разработки, чтобы вы могли реализовать то, что они ищут, прежде чем вы продолжите делать это по-своему? Вы что-то меняете в том, как вы разрабатываете код, основываясь на их предложениях, или продолжаете думать, что у вас все в порядке? Вы изучаете что-нибудь из их комментариев?
Когда я отвечаю за проект, моя работа - отвечать за ВСЕ рабочие продукты. Если я одобряю рабочий продукт, то я утверждаю, что продукт приемлем. Я хочу иметь репутацию производителя качественной продукции. Таким образом, у меня есть ожидания, и я не приму меньше, чем удовлетворительное. В то же время я стараюсь учить и объяснять причины своих предпочтений. Эти предпочтения не всегда могут быть идеальными (особенно в глазах других), но большинство этих предпочтений основано на опыте. Обычно это реакция, чтобы избежать повторения плохих. Таким образом, есть некоторые из моих личных "сторонников", которые необходимы, чтобы получить мое одобрение, независимо от откатов.
С другой стороны, вам нужно узнать ожидания, необходимые для утверждения ваших рабочих продуктов. Вы можете не согласиться, но, поскольку у вас нет полномочий для чрезмерного управления, вы должны узнать, что ожидается. Я сомневаюсь, что команда пытается заставить вас потерпеть неудачу. Как это заставляет их выглядеть плохо тоже. В связи с этим просто продемонстрируйте, что вы стремитесь к обучению (даже если вы этого не делаете), примите то, что они говорят, и сделайте все возможное, чтобы приспособиться к их предпочтениям, и вы, скорее всего, увидите, что они немного отступят. Может быть, найти тот, который вы, по крайней мере, можете терпеть и посмотреть, будут ли они немного держаться за руки, чтобы научить вас своим путям. Кто знает, в процессе вы можете узнать что-то, что действительно может поднять ваши навыки на следующий уровень.