r
по мне так код надо исправлять, если он не выполняет свою задачу, а не от того, что он отлежался
Рефакторинг делают на коде, который выполнят свою задачу. Т.е. поведение системы после него не меняется. Причина рефакторинга совсем в другом.
Size: a a a
2019 June 12
r
Vyacheslav
Имхо, код лучше улучшать сразу. В плане – увидил и сразу исправил. Но для этого процессы должны быть поставлены, чтобы на это время можно было закладывать.
Правило бойскаута.
r
И как то странно звучит - отрефакторили, лобавив тесты.
почему странно? я конечный результат описал
r
был легаси код (код без тестов) - стал отрефакторенный с тестами
r
почему странно? я конечный результат описал
Странно, потому что ваша фраза очень неоднозначно звучит! Я ее понял: был сделан рефакторинг, смысл которого - покрытие тестами старой кодовой базы.
r
В нашем деле предпочитаю докапываться до каждого слова, ведь даже с такой дотошностью всеравно есть постоянные расхождении в понимании не только внутри команды.
r
Странно, потому что ваша фраза очень неоднозначно звучит! Я ее понял: был сделан рефакторинг, смысл которого - покрытие тестами старой кодовой базы.
ну, в принципе, бывает старый код, который трудно покрыть тестами без предварительных ласк. про это целая книга есть - working with legacy code
r
Да, жаль только книга очень убого переведена. :(
r
Но там да, описаны «ласки» :)
V
по мне так код надо исправлять, если он не выполняет свою задачу, а не от того, что он отлежался
Код устаревает всегда. Если его не редактировать, то рано или поздно это бомбанет.
N
Vyacheslav
Имхо, код лучше улучшать сразу. В плане – увидил и сразу исправил. Но для этого процессы должны быть поставлены, чтобы на это время можно было закладывать.
одно другому не мешает. через год-два после написания теста полюбому можно увидеть как лучше его написать, чтобы он был чище и понятнее. Если взять код пятилетней давности, то его зачастую проще переписать проще, поскольку очень многое за это время могло появиться - удобные либы, которые можно взять и заюзать вместо велосипедов, какие-то конструкции языка, которые стали более предпочтительными.
Цель в общем шарить экспертизу и вырабатывать подходы к написанию. Те же самые юнит-тесты начать внедрять на старом коде. И тут возникает две проблемы - поиск дурнопахнущего кода и обсуждение, почему код плохой и какие существуют способы его улучшения.
Цель в общем шарить экспертизу и вырабатывать подходы к написанию. Те же самые юнит-тесты начать внедрять на старом коде. И тут возникает две проблемы - поиск дурнопахнущего кода и обсуждение, почему код плохой и какие существуют способы его улучшения.
V
А зачем переписывать код?
V
Какое этому экономическое обоснование ?
r
Легче поддерживать, модифицировать, расширять
V
Легче поддерживать весь код? Тебе приходится поддерживать, модифицировать и расширять весь код?
V
Насколько легче?
Настолько же, сколько времени потратили на переписывание?
Настолько же, сколько времени потратили на переписывание?
V
Просто потому что нельзя сразу взять и написать хорошо
V
А что такое хорошо?
V
Код, который удовлетворит бизнес требованиям. И в нем легко исправлять ошибки
И добавлять новые фичи
И добавлять новые фичи
V
Да, нового человека в проект легко добавить