Переглядаючи код, я зазвичай намагаюся дати конкретні рекомендації щодо вирішення проблем. Але через обмежений час, який можна витратити на огляд, це не завжди працює добре. У цих випадках я вважаю його більш ефективним, якщо розробник сам знайде рішення.
Сьогодні я ознайомився з деяким кодом і виявив, що клас, очевидно, не добре розроблений. Він мав ряд необов’язкових атрибутів, які були призначені лише для певних об'єктів і залишені порожніми для інших. Стандартним способом вирішити це було б розділення класу та використання успадкування. Однак у цьому конкретному випадку це рішення здавалося надто складним. Я сам не брав участі в розробці цього програмного забезпечення і не знайомий з усіма модулями. Тому я не відчував себе достатньо обізнаним, щоб прийняти конкретне рішення.
Ще один типовий випадок, який я переживав багато разів, - це те, що я знаходжу очевидно безглузду або навіть оманливу функцію, ім'я класу чи змінної, але не можу сам придумати гарне ім’я.
Отже, як рецензент, як добре сказати, "цей код хибний, тому що ..., зробіть це по-іншому" або вам потрібно придумати конкретне рішення?