nyann

Для начала скажите зачем Вы вообще хотите рефакторить код? Моральное удовлетворение?! Технический долг?! Вам что, приятнее трахаться с функциями в коде, чем со своей девушкой?

Работает код - не трогайте его, пусть дальше работает.

А вот если клиенту нужно сделать какую-то специальную фишку, для чего тебе нужно влезть в N*100 классов, добавить обработку новых входных параметров, прописать формирование и контроль выходных данных... Вообщем создает много гемороя, вот тогда ты говоришь клиенту, что это встанет в столько-то часов твоего времени и вот такой будет ценник и, если тот соглашается, уже начинаешь рефакторить.

Вообще где-то я видел мнение(тут на тостере или на codenet) что клиенту вообще не надо знать, что ты занимаешься рефакторингом, с чем лично я согласен. Ты показываешь ему свои почасовые отчеты, клиент видит что ты работаешь, все. Остальное уже те мелочи, от которых ты его избавляешь как специалист.

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

А если видно, что код и структура чисты, то конечно, костыли неприемлемы.

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