Что делать со старыми кусками кода в проекте?

Question

[beginer123]

Заметил такую вещь, что много оставляю за собой кода, на всякий случай.
Допустим старую функцию addToCart переименовываю в addToCart7777, или внутри функций комментирую старые куски кода.
Вдруг пригодятся.
Однако понял что они как то усложняют читаемость кода.
Но совсем удалить не могу.
удаляют только то что 100% больше не нужно.
Придумал такое решение: в отдельный файл например old.js, old.php и тд записывать все вещи которые могут пригодиться.
Хорошее решение?так делают сеньер разработчики?
Если нет то как тогда надо?

Comments

[Толстый Лорри]

Хорошее решение?так делают сеньер разработчики?
Если нет то как тогда надо?

Лишний код не нужен вообще. Ибо открыв через некоторое время нельзя быть уверенным, насколько он актуален, исправен и т.п. И лишь мозолит глаза.
Насчет ценности. Мелкие куски кода по озвученным выше причинам несложно написать заново (а независимые сущности обязаны быть маленькими, ибо SOLID), а большие подсистемы теряют актуальность в связи с рефакторингами и прочим. Достаточно абстрактные кандидаты на "библиотечность" можно записать себе куда-то. Но мертвый код в проекте держать не нужно, тем более со всяким трешем в названии.

Answer 1

[tapokshot]

Удаляю сразу как только вижу что не где не используется, всегда можно по git истории восстановить.

Comments

[beginer123]

А если не используется git?

[beginer123]

А как на счет моего решения?

[Александр Титов]

beginer123, если не испоьзуется git (или другая vсs), то вообще обсуждать нечего :)

[sim3x]

не где
нигде

[tapokshot]

beginer123, ты спросил про хорошее решение, а оно включает в себя git =)

Answer 2

[Сергей delphinpro]

Заметил такую вещь, что много оставляю за собой кода, на всякий случай.

Человек, начни использовать систему контроля версий, не страдай фигнёй =)

Answer 3

[Demian Smith]

Kill them with fire. Удаляйте их с гомерическим хохотом.

Но вобще в огромных коммерческих проектах типа booking.com не разрешается удалять старый код слёту и по прихоти. Что там делают? На входе в функцию ставят счётчик, который увеличивается на единицу при каждом вызове функции и пишет это дело в лог. Так же фиксируется когда функция была вызвана в последний раз. Если в течении трёх месяцев функция не была вызвана ни разу, то её сносят.

Answer 4

[Максим Тимофеев]

Для этого есть TODO, что бы не забывать где напачкал. И 2 версии проекта, в разработке - это допустимо, а на продакшен надо только с пустым TODO

Answer 5

[Михаил]

Удалять код, может не сразу, но при следующей ревизии чистить.
Если хотите хранить какие то куски кода для себя ( по принципу how to ) то используйте wiki, например https://www.dokuwiki.org/DokuWiki или аналог