Любые полезные метрики подойдут





Одна из вещей, которые я ищу в коде, - это модульные тесты. Это даст свободу его рефакторингу. Поэтому, если в коде нет тестов, я считаю его устаревшим.
Термин «устаревший код» субъективен и, вероятно, является загруженным термином. но в целом я разделяю мнение о том, что унаследованный код не подлежит модульному тестированию и поэтому его сложно рефакторировать.
Майкл Фезерс, автор прекрасной книги «Эффективная работа с устаревшим кодом», определяет его как любой код, не имеющий тестов.
Да. По словам Фезерса, если он написан без модульных тестов, он становится унаследованным кодом в ту же минуту, когда он написан.
Да ладно тебе! Да, модульные тесты важны, но это определение заходит слишком далеко. Наследие означает устаревшее. Код без тестов не устарел, это просто головная боль.
Я не читал Feathers, но могу догадаться о его рассуждениях: если у вас нет модульных тестов, то у вас нет возможности проверить, что какие-либо изменения в коде не нарушили его, поэтому вам не следует его менять, поэтому он наследие.
Я опаздываю, но некоторые языки применяют «контракты кода», которые делают некоторые аспекты модульного тестирования устаревшими.
Возможно, я ошибаюсь, но я не думаю, что для этого есть установленная метрика.
Обычно фрагмент кода считается устаревшим, если он прошел не менее 5-6 циклов выпуска (может быть, больше). Чаще всего оригинального исполнителя больше нет, а код поддерживается.
Я не знаю никаких реальных показателей, которые можно было бы использовать, чтобы определить, является ли что-то «устаревшим кодом» или нет, но все, что старше только что написанного, можно было бы считать устаревшим. Унаследованный код означает разные вещи для разных людей / организаций, поэтому на самом деле это несколько субъективно.
Если код:
Разве это не то, что вы назвали бы вздором.
Я не верю, что есть однозначный ответ, но я верю, что вероятность того, что код является устаревшим, увеличивается с увеличением количества людей, которые не хотят его трогать, и вероятностью того, что его изменение приведет к его поломке.
Почти через секунды после того, как разработчики покинут помещение. :)
Если...
нет денег в банке на новые функции
вы не можете найти никого, кто допустил бы работу над проектом, который требует исправления
исходный код вашего проекта прошел MIA
... то вы работаете над устаревшим кодом.
Лучшим вопросом, вероятно, будет то, что помечает часть кода как не унаследованную.
Для меня наследие означает неизменное. Так что, как только вы больше не в состоянии его изменить, это наследство.
В значительной степени не имеет значения, устранена ли эта способность фиксированными требованиями, страхом поломки, потерей знаний или каким-либо другим воздействием.
Еще одно замечание: я не думаю, что когда-либо буду использовать точное слово «наследие», поскольку оно вызывает слишком много эмоций, чтобы быть полезным.
Мы используем термин «унаследованный» для обозначения любого кода, который все еще используется и разработан с использованием технологий, в которых мы прекратили активную разработку.
Это код, который мы бы предпочли переписать с использованием более свежих инструментов, чем изменять его текущее состояние.
На мой взгляд, весь написанный код - это устаревший код. Может пройти некоторое время, прежде чем первоначальное намерение и все решения, принятые в отношении кода, будут забыты, но рано или поздно вы не сможете представить, о чем они думали, когда его писали. Вы же сами никогда не пишете устаревший код, верно?
Использование модульных тестов или некоторой меры, такой как секунды, с тех пор, как разработчик покинул здание, на самом деле не позволяет определить, является ли код устаревшим кодом. Унаследованный код может иметь хороший набор модульных тестов и комментариев, и он может быть подвергнут строгой проверке кода и другому анализу. Это не означает, что код по-прежнему актуален для рассматриваемой программы. Это просто говорит о том, что код может быть написан сравнительно хорошо. И если это больше не актуально, код фактически усложнит решение проблемы, для которой разработана программа.
Я сам предпочитаю такой подход. После того, как код написан, он остается «наследником», независимо от того, есть ли тесты, покрывающие код, или нет. У вас могут быть тесты, которые охватывают код, который был написан много лет назад, никого не знаком с этим кодом и / или использующие «старые» технологии ...
Обычно люди называют что-то унаследованным кодом, когда вокруг нет никого, кто знаком с кодом или чувствует себя комфортно с ним.
Модульные тесты позволяют людям, незнакомым с кодом, вникать в него, поэтому теория такова, что они помогают предотвратить превращение кода в «устаревший».
Часто, когда код является устаревшим, он изменяется по-другому. Люди боюсь, чтобы изменить это, но также изменения, как правило, быстро и грязно, потому что никто не понимает всех последствий. Могут возникнуть проблемы с дублированием кода, потому что люди не хотят идти на риск, связанный с более глубокими изменениями.
Итак, в таких обстоятельствах ситуация может стать хуже, с возрастающей скоростью.
Унаследованный код во многих местах определяется как «код без тестов». Я не думаю, что они специфичны для разных типов тестов, но в целом, если вы не можете внести изменения в свой код, не опасаясь чего-то неизвестного, что ж, это быстро меняется.
Я вижу здесь прагматичного программиста.