Чи звичайно комусь регулярно отримувати 100 коментарів у своїх оглядах коду? Я б сказав, що ні. Чи загально для людей, чия якість коду "залишає бажати кращого", отримують багато коментарів, абсолютно.
Однак це також залежить від "правил" процесу перегляду коду. У ВСЕГО є свої уявлення про те, як щось треба було зробити. Якщо ваш процес перегляду коду дозволяє коментарям мати форму "Ви повинні зробити це так, а не таким чином", ви, швидше за все, отримаєте багато коментарів навіть за відповідний код. Якщо ваш процес призначений для пошуку «дефектів», то кількість коментарів повинна бути значно меншою.
На мій досвід, огляди, які дозволяють "запропонувати" альтернативні методи, є витратами часу. Ці "пропозиції" повинні бути розроблені один на один поза процесом огляду. Відгуки про дефекти є більш корисними, оскільки вони змушують людей зосереджуватися на помилках, а не "чому ти цього не зробив так, як я це зробив би?". Це також корисніше, тому що не можна заперечувати помилку, якщо хтось її знайде. Таким чином, немає ніяких пошкоджених почуттів, а ймовірно, вдячність.
ОНОВЛЕННЯ: З урахуванням сказаного, деякий код є просто поганим, навіть якщо дефект є вільним. У такому випадку коментар до огляду повинен бути єдиним коментарем, який говорить щось подібне. "Цей код потрібно очистити. Будь ласка, відкладіть огляд, поки код не буде обговорено з [ваше ім'я тут]." У такому випадку подальший перегляд коду повинен припинитися, поки коментар не буде виправлений.
UPDATE2: @User: Чи обговорюєте ви свій код / дизайн з одним із них, поки ви розробляєте його, щоб ви могли реалізувати те, що вони шукають, перш ніж дійти до того, як це зробити ваш шлях? Чи змінюєте ви щось про те, як ви розробляєте код на основі їх пропозицій чи продовжуєте думати, що ваш шлях добре? Ви щось дізнаєтесь з їхніх коментарів?
Коли я є керівником проекту, моя відповідальність за ВСІ робочі продукти. Якщо я схвалюю робочий продукт, то я стверджую, що продукт є прийнятним. Я хочу мати репутацію в створенні якісної продукції. Таким чином, я маю очікування і не прийму менш задовільно. У той же час я намагаюся навчити та пояснювати причини своїх уподобань. Ці вподобання не завжди можуть бути ідеальними (особливо в очах інших), але більшість цих переваг випливає з досвіду. Зазвичай це реакція, щоб уникнути повторення поганих. Таким чином, є кілька моїх особистих «наклейок», які необхідні для отримання мого схвалення, незалежно від відштовхування.
З іншого боку, вам потрібно вивчити очікування, необхідні для затвердження вашої роботи. Ви можете погодитись, але оскільки ви, здається, не маєте повноважень надмірно управляти, то дізнайтеся, що очікується. Я сумніваюся, що команда намагається зробити вас невдачею. Тому це також виглядає погано. У цьому плані просто продемонструйте, що ви прагнете вчитися (навіть якщо ви цього не робите), прийміть те, що вони говорять, і зробіть все можливе, щоб адаптуватися до їхніх уподобань, і ви, ймовірно, побачите їх зовсім небагато. Можливо, знайдіть той, якого ви можете принаймні терпіти, і подивіться, чи вони зроблять трохи рукою, щоб навчити вас їх дорогам. Хто знає, у процесі ви можете навчитися чомусь, що насправді може перенести ваші навички на наступний рівень.