e
Как раз для того, чтобы не похерить то, что было, пока ты рефакторишь.
ну если вкратце, то да
Size: a a a
2020 April 25
YM
Как раз для того, чтобы не похерить то, что было, пока ты рефакторишь.
а рефакторить не равно херить, избавляться
?
?
I
Нет.
I
Рефакторинг не должен изменять поведение.
YM
Рефакторинг не должен изменять поведение.
всмысле херить код а не поведение
e
всмысле херить код а не поведение
какая разница, ты на 100 процентов не можешь быть уверен, что после слепого рефакторинга твоя система будет работать как раньше
YM
какая разница, ты на 100 процентов не можешь быть уверен, что после слепого рефакторинга твоя система будет работать как раньше
почему слепого? глаза же есть
I
Тесты пишутся обычно на поведение. Соответственно интерфейс даже скорей всего никак не изменится. Изменится внутренняя логика объекта за ним, тот самый код, который мы рефакторим.
e
почему слепого? глаза же есть
глаза есть, а если код писал кто то другой? ты тоже все предсказать сходу сможешь когда рефакторить будешь?
YM
глаза есть, а если код писал кто то другой? ты тоже все предсказать сходу сможешь когда рефакторить будешь?
я всегда внимательно читаю чужой код, чтобы понять что он делает
e
:)
I
А тесты подтвердят, что все работает после рефактора так, как изначально и работало.
YM
как будто изначально оно работало хорошо и правильно
зачем тогда рефакторить
зачем тогда рефакторить
e
как будто изначально оно работало хорошо и правильно
зачем тогда рефакторить
зачем тогда рефакторить
в том и прикол, что да
YM
все же лучшее основание - знание домена, а не чужой код
I
как будто изначально оно работало хорошо и правильно
зачем тогда рефакторить
зачем тогда рефакторить
Пользователь твоего интерфейса не видит тех процессов, при помощи которых и достигается это "работает хорошо и правильно". И вот именно эти процессы могут быть очень сложны в поддержке и понимании. Их мы и рефакторим.
I
А пользователь даже и не подозревает о том, что мы что-то сделали проще, ведь для него ничего не изменилось.
YM
ну это когда интерфейс уже есть
понятно что покрыть его тестами и рефакторить нутряк
понятно что покрыть его тестами и рефакторить нутряк
YM
а бывает что мазня намазана в 10 местах а в сумме на целый модуль набегает
I
Может быть и такое, конечно.