Здравствуйте, fmiracle, Вы писали:
F>Код-ревью — это когда члены команды просматривают изменения в коде друг друга и дают замечания, если видят проблемы в них.
Почему? Ничто не мешает под этим термином понимать и ревью существующего кода.
Да, чаще про это говорят "аудит", но формально противоречия нет.
Вероятно, ТС просто не знает, какие тут стандартные термины.
F>Оно может внедряться при вообще любом качестве кода системы. Имеет смысл заранее проговорить что и как проводится на код-ревью. F>Т.е. замечания "тут надо бы переписать все вот это и еще полсистемы старого, и тогда будет хорошо" они бессмысленны. Замечания должны быть про то, как можно лучше сделать то, что нужно сделать, с имеющимися ресурсами на задачу.
Вполне осмысленны, если из них следует или что надо перед изменениями поставить рефакторинг, или что он уйдёт потом в отдельную задачу. Дальше весь вопрос в том, кто что успевает.
F>На мой взгляд, код-ревью это одна из самых простых и при этом чуть ли не самая эффективная из методик улучшения кода. F>Когда будешь проводить рефакторинг ключевых проблем — в чем беда, что проведенный рефакторинг посмотрит кто-то из коллег? А он может заметить существенную проблему ведь. Взгляд замыливается и со стороны частенько видно лучше.