И
зачем с нуля. Если простыня кода, начни в выделения сервисов и переноса функций туда.
там сервисы тоже на 3-4к строк
Size: a a a
2020 June 04
AS
какая-то прям мощная диаграма
И
вообщем мне дали ее, сказав сделай пожалуйста все красиво, чтоб было максимально читабельно и тд, если понадобится перепиши с нуля
AS
такие вещи часто появляются как кривой форк какой-нибудь либы приправленный костылями
И
какая-то прям мощная диаграма
не, я вот первым этапом, разделил по меньшим компонентам, и в функциях где видел странный код, типа больших ифов или не понятную фигню, переписывал ее, теперь следующий этап уже полностью прочитстить
И
такие вещи часто появляются как кривой форк какой-нибудь либы приправленный костылями
код начинали писать в далеких 80х, и с тех пор это кажется первый крупный рефактор, много работяг полегло под этой диаграммой
AS
Исмаил
код начинали писать в далеких 80х, и с тех пор это кажется первый крупный рефактор, много работяг полегло под этой диаграммой
это он на чем написан? о.о
И
это он на чем написан? о.о
ну тут я чуть перегнул конечно) но проект с тех временен кочевал по разным языкам фреймворкам и тд, и это просто большой кусок мусора)
IB
Исмаил
вообщем мне дали ее, сказав сделай пожалуйста все красиво, чтоб было максимально читабельно и тд, если понадобится перепиши с нуля
Любой рефакторинг начинается с того что пишешь тесты и фиксируешь текущее поведение
И
Любой рефакторинг начинается с того что пишешь тесты и фиксируешь текущее поведение
проблема в том что я незнаю как она должна работать, тоесть нет представления о правильном поведении, получается прокликать все варианты?
IB
Исмаил
проблема в том что я незнаю как она должна работать, тоесть нет представления о правильном поведении, получается прокликать все варианты?
Да, подаёшь на вход какие то данные, смотришь что на выходе, это и фиксируешь. Таким образом в конце если все тесты прошли - ты сохранил текущее поведение. Там были ошибки? Поправь неверный тест и все пофикси
И
Да, подаёшь на вход какие то данные, смотришь что на выходе, это и фиксируешь. Таким образом в конце если все тесты прошли - ты сохранил текущее поведение. Там были ошибки? Поправь неверный тест и все пофикси
и после уже начинаю переписывать уже сам компонент по тестам получается?
IB
Исмаил
и после уже начинаю переписывать уже сам компонент по тестам получается?
Сначала все зафиксировать тестами - потом только начинать рефакторинг
И
Сначала все зафиксировать тестами - потом только начинать рефакторинг
идея понятна, и хороша вроде, попробую
IB
Тем более когда мыглворим про что то в тысячи строк и годы разработки
Вキ
Вキ
А россия тебя разве выпустит?
Мне как раз очень надо в рф и назад
Мне как раз очень надо в рф и назад
не понятно
]🌶
Тем более когда мыглворим про что то в тысячи строк и годы разработки
Годы тестов перед рефакторингом
И
кстати тестов там ровно ноль)
AS
Если код хаотичен, и там канвас, то тесты замучаешься писать, проще наверно выкинуть весь код и сделать свою диаграмму по мотивам