Комментарии 6
httperr.Unauthorised("no-user-found", err, w, r)
За такие дегенеративные сообщения об ошибках надо выкидывать из профессии!
Личный опыт показывает, что хороший рефакторинг - когда становится меньше кода. Если стало больше кода - это не есть хорошо. Больше кода - больше шансов возникновения ошибки. Больше кода - гругу придётся читать больше кода.
Странный опыт у вас, у меня вот обратный. Метод на 1000 строк будет всегда меньше чем выделение этого ада в разные классы. Более того, довольно часто приходится писать всякий скаффолд, чисто ради того, что бы к примеру разделить ответственности или зависимости. Но это конечно если переписывается конкретный кусок. В общем приложение скорее всего станет меньше, потому что код можно будет легче переиспользовать. Да и скаффолд я не считаю увеличением кода, как правило он легко автоматизируется. В любом случае я лучше напишу 200 строк бездумного кода, чем 100 строк транзакшн скрипта, который умеет всё.
Здесь нет прямой связи между раздутыми методами и общим уменьшением кода в следствии хорошего рефакторинга. Возможно стоит уточнить, что уменьшение кода не является достаточным признаком, но с большой вероятностью является необходимым. Это позволяет сохранить баланс между использованием, скажем, абстракций и реализацией конкретной бизнес логики.
От wtf-кода к чистой архитектуре