Мое эмпирическое правило: оставляйте код не хуже, чем вы его нашли .
Идея состоит в том, чтобы работать в направлении улучшения , не пытаясь достичь идеального результата или идти до конца.
Индивидуальные рефакторинги иногда имеют сомнительную выгоду, и - как крайний пример - это действительно может быть аргумент, если m_Pi
лучше, чем m_PI
. Однако чаще всего один выбор является более последовательным и менее удивительным, даже если он явно не «лучше».
Одной из ситуаций, в которой я, как правило, оказываюсь рефакторингом автоматически, является до реализации функции на фрагменте кода.
Зачастую TODO ждут подачи, некоторые несоответствия или иногда пользовательские функции, которые в последнее время получили лучшую поддержку библиотек. Выполнение этих изменений до того, как я реализую фактический запрос функции, дает мне некоторое понимание кода, и я проверяю функциональность «до».
Еще один момент - после исправления ошибок. После, так что пред-репро не затрагивается, а исправление ошибки и рефакторинг - это две отдельные фиксации.