Что вы делаете с избыточным кодом?

Question

У меня есть класс, который является частью проекта библиотеки кода, который был написан для определенной цели, которая больше не требуется. Итак, вопрос в том, что вы делаете с таким кодом? Вы просто удаляете его или оставляете это, имея в виду, что будущие разработчики могут столкнуться с ним и не осознавать, что могут его игнорировать, или у вас есть какая-то архивная система, существует ли признанный «шаблон», который находится в использовать ...

Answer 1

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

Answer 2

Как сказал Нил, удали его. Если меня нанимают поддерживать ваш проект спустя годы после того, как вы закончите с ним, и он все еще полон мертвого кода ... Я буду преследовать вас. И не таааааааааааааааааааааааааааааа, а ... ЧАСТНАЯ ЗАПИСЬ !!!!!!!!!!!!!!!!!!!!!! раздражающий вид преследования.

Answer 3

Это зависит.

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

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

Answer 4

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

Обновление: хорошо, теперь я думал о .NET и C #, но я уверен, что многие другие языки имеют схожие функции ...

Answer 5

Если он нигде не используется и больше не требуется, его следует удалить, чтобы избежать путаницы.

Вы не сказали, какой код используете, но в C # / VisualStudio вы можете использовать атрибут Obsolete, чтобы запретить другим разработчикам использовать код, вы можете установить для аргумента errors значение true, и это повредит сборку где угодно что код используется.

Answer 6

Просто удалите его. Если оно больше не требуется, нет смысла его хранить.

Answer 7

Установите GIT затем:

cd <code repo>
git init .
git add .
git commit -m 'inital import for my old code'

... Рефакторинг кода ...

git add <path/to/file/with/changes/>
git commit -m 'that feels much better... :)'

... Создать учетную запись на GitHub или настроить GitServer

git remote add origin <remote git repo>
git push origin master

И все готово ...:)

Answer 8

Если много, многократно используемого кода и / или кода трудно воспроизвести, я обычно помещаю его в файл с именем <projectname>_rubbish.<ext>. Не очень элегантно, но я могу легко проигнорировать это, а также легко найти его, когда оно мне снова понадобится.

Answer 9

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

Answer 10

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