Должен ли я код рефакторинга, если он скрывает поток программы?

Question

Я создал спокойные веб-службы. Конечные точки служб имеют аналогичную логику:

Если проверка ввода прошла успешно, выполните операцию базы данных и верните результат.

Если операция db завершилась неудачно, верните ошибку db.

Если проверка не получится, возвращает ответ с информацией о неудачных проверках

Моего кода было явно в том смысле, вы можете прочитать код и понять поток программы.

Кооператор реорганизовал его так, чтобы этот базовый поток был реорганизован в класс утилиты, основная функция которого принимает тело запроса, операцию db и обработчики.

Теперь код не выражает поток, но основная логика реорганизуется.

Какая практика? Вы пожертвовали бы читабельностью для рефакторинга?

Да, единичный тест также будет уменьшен.

Спасибо.

Answer 1

Как вы думаете, что класс утилита, которая содержит переработан код, будет повторно использоваться во многих других подобных сценариях для операций, например,

• подтверждения ввода,
• работает операции с базой данных,
• возвращая результат

Если он будет использоваться повторно в других подобных сценариях, тогда это нормально для рефакторинга.

Я думаю, что было бы лучше рефакторировать, как вы можете видеть с точки зрения основного кода, рефакторинг серверов - лучшая цель. (Если позволяет время, я думаю, вы могли бы сделать новый код также читаемым, чтобы сделать поток явным.)