Хороший метод для обработки большого количества неиспользуемых или устаревших реализаций функций в коде - PullRequest
Новая
       3

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

3 голосов
/ 17 декабря 2010

В настоящее время мы сталкиваемся со следующей проблемой в нашем приложении:

Приблизительно 40% кода, который находится в приложении, никогда не используется.Это означает, что код был бы там, и, возможно, функционировал, но функция Frontend была отключена , поэтому пользователи больше не могут достичь функциональности, или другие методы заменяют старые, теперь устаревшие методы .

Что я сейчас делаю, так это удаляю весь старый код, не пытаясь ничего сломать, вручную.

Вопрос в следующем:

  • Не могли бы вы удалить старый Код или надеяться, что он может проснуться когда-нибудь ... зомби - как
  • Как вы думаете,Стоит приложить усилия для удаления Кодекса (меньше работы по поиску вещей в беспорядке, лучшее тестовое покрытие, проще для других людей найти свой путь)
  • Должны ли мы где-то хранить Код в качестве ссылки?(Мы используем Контроль версий, но я нахожу, что довольно сложно найти старый код в джунглях Ревизии ... есть ли у вас какие-нибудь советы для этого?)
  • У вас есть аргументы, чтобы убедить команду / руководство / разработчиков в том, чтоВы написали указанный код?
  • Причины не использовать устаревший код, а затем удалить код?

TL; DR: Удалить неиспользуемый код или оставить его как есть?Обсудить!

u

Ответы [ 2 ]

4 голосов
/ 17 декабря 2010

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

Удаление неиспользуемого кода облегчит обслуживание проекта, и ваша команда, вероятно, в конечном итоге сэкономит время (никто не перечитает код, чтобы попытаться понять, для чего он использовался, никто не будет в итоге изменяя указанный код, думая, что он все еще может быть использован ...)

Однако, если ваш код содержит общедоступный API-интерфейс, который распространяется, вы, вероятно, захотите пометить классы / методы как устаревшие на некоторое время перед эффективным удалением кода, чтобы у вызывающих абонентов было время для адаптации (или для информирования вас выпуск).

2 голосов
/ 17 декабря 2010
  • Вы удалите старый Код или надеетесь, что он может проснуться когда-нибудь ... зомби - как

Я бы определенно удалил это. Ненавижу работать, если когда-нибудь будут вызваны функции.

  • Как вы думаете, стоит ли пытаться удалить Кодекс (меньше работы по поиску вещей в беспорядке, лучшее тестовое покрытие, проще для других найти свой путь)

Да, определенно стоит усилий.

  • Должны ли мы хранить код где-нибудь, в качестве ссылки?

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

...