Size: a a a

2019 June 12

r

ref1exxx in devleads chat
Стася
по мне так код надо исправлять, если он не выполняет свою задачу, а не от того, что он отлежался
Рефакторинг делают на коде, который выполнят свою задачу. Т.е. поведение системы после него не меняется. Причина рефакторинга совсем в другом.
источник

r

ref1exxx in devleads chat
Vyacheslav
Имхо, код лучше улучшать сразу. В плане – увидил и сразу исправил. Но для этого процессы должны быть поставлены, чтобы на это время можно было закладывать.
Правило бойскаута.
источник

r

rokrbek in devleads chat
ref1exxx
И как то странно звучит - отрефакторили, лобавив тесты.
почему странно? я конечный результат описал
источник

r

rokrbek in devleads chat
был легаси код (код без тестов) - стал отрефакторенный с тестами
источник

r

ref1exxx in devleads chat
rokrbek
почему странно? я конечный результат описал
Странно, потому что ваша фраза очень неоднозначно звучит! Я ее понял: был сделан рефакторинг, смысл которого -  покрытие тестами старой кодовой базы.
источник

r

ref1exxx in devleads chat
В нашем деле предпочитаю докапываться до каждого слова, ведь даже с такой дотошностью всеравно есть постоянные расхождении в понимании не только внутри команды.
источник

r

rokrbek in devleads chat
ref1exxx
Странно, потому что ваша фраза очень неоднозначно звучит! Я ее понял: был сделан рефакторинг, смысл которого -  покрытие тестами старой кодовой базы.
ну, в принципе, бывает старый код, который трудно покрыть тестами без предварительных ласк. про это целая книга есть - working with legacy code
источник

r

ref1exxx in devleads chat
Да, жаль только книга очень убого переведена. :(
источник

r

ref1exxx in devleads chat
Но там да, описаны «ласки» :)
источник

V

Vyacheslav in devleads chat
Стася
по мне так код надо исправлять, если он не выполняет свою задачу, а не от того, что он отлежался
Код устаревает всегда. Если его не редактировать, то рано или поздно это бомбанет.
источник

N

Nekt in devleads chat
Vyacheslav
Имхо, код лучше улучшать сразу. В плане – увидил и сразу исправил. Но для этого процессы должны быть поставлены, чтобы на это время можно было закладывать.
одно другому не мешает. через год-два после написания теста полюбому можно увидеть как лучше его написать, чтобы он был чище и понятнее. Если взять код пятилетней давности, то его зачастую проще переписать проще, поскольку очень многое за это время могло появиться - удобные либы, которые можно взять и заюзать вместо велосипедов, какие-то конструкции языка, которые стали более предпочтительными.

Цель в общем шарить экспертизу и вырабатывать подходы к написанию. Те же самые юнит-тесты начать внедрять на старом коде. И тут возникает две проблемы - поиск дурнопахнущего кода и обсуждение, почему код плохой и какие существуют способы его улучшения.
источник

V

Vitaly in devleads chat
А зачем переписывать код?
источник

V

Vitaly in devleads chat
Какое этому экономическое обоснование ?
источник

r

rokrbek in devleads chat
Легче поддерживать, модифицировать, расширять
источник

V

Vitaly in devleads chat
Легче поддерживать весь код? Тебе приходится поддерживать, модифицировать и расширять весь код?
источник

V

Vitaly in devleads chat
Насколько легче?
Настолько же, сколько времени потратили на переписывание?
источник

V

Vyacheslav in devleads chat
Просто потому что нельзя сразу взять и написать хорошо
источник

V

Vitaly in devleads chat
А что такое хорошо?
источник

V

Vyacheslav in devleads chat
Код, который удовлетворит бизнес требованиям. И в нем легко исправлять ошибки
И добавлять новые фичи
источник

V

Vyacheslav in devleads chat
Да, нового человека в проект легко добавить
источник