Использование устаревшего атрибута

Шаг 1. Отметьте участника или класс как [Устаревший]

Шаг 2. Обновите все внутренние варианты использования члена или класса, чтобы либо использовать новый подход, который заменяет устаревший подход, либо пометить этот член или класс как [Устаревший].

Шаг 3. Если вы пометили новый материал как [Устаревший] на шаге 2, повторите этот шаг по мере необходимости.

Шаг 4. Удалите все устаревшие члены и классы, которые не являются общедоступными и не используются устаревшим открытым членом или классом.

Шаг 5. Обновите документацию, чтобы дать более четкое описание подхода, рекомендуемого для замены любых общедоступных устаревших членов или классов.

В конце этого у вас не будет устаревшего кода, который используется исключительно внутренним кодом. Однако нельзя сказать, что вы должны сделать все это за один раз; на каждом этапе вы добились прогресса. Время между началом шага 1 и завершением шага 5 может составлять 5 секунд или 5 лет, в зависимости от многих факторов (большинство из них связано со сложностью).

Между прочим, если кому-то легко игнорировать предупреждения компилятора, проблема не в [Obsolete]. Тем не менее, одна из причин не оставлять такие вызовы в коде надолго (то есть делать до шага 2 как можно скорее) состоит в том, чтобы убедиться, что люди не привыкнут к предупреждениям компилятора, поскольку они являются частью кода. привычная реакция на компиляцию кода.

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

Однако у меня также была ситуация, когда соответствующий рефакторинг занимал намного больше времени (много вызовов по всей кодовой базе), и в этом случае я использовал атрибут [Obsolete]. В этом случае новая разработка будет использовать новые функции, а те, у кого есть время, выполнят рефакторинг, пока не исчезнут все вызовы, а это значит, что метод можно будет удалить.

По-разному. Да, вы МОЖЕТЕ реорганизовать код. МОГЛИ БЫ ВЫ?

Проблема в том, что вы МОЖЕТЕ провести рефакторинг ВНУТРИ ОДНОЙ ПРОГРАММЫ. Это намного сложнее, если API находится в открытом доступе, и вы просто НЕ МОЖЕТЕ провести рефакторинг кода, используя свой API. Это то, для чего создан Obsolete.

если API является внутренним для вашего кода, то рефакторинг — это путь. Очистите код, не оставляйте беспорядок.

Но если общедоступный API изменится, это следует, по возможности, делать медленно.

Остальное пока субъективно. Мне не нравится «Устаревший» для внутренних API.

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

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

Действительно ли это хорошо или плохо, довольно субъективно. Это инструмент, как и любой другой.

Это не прямой случай. Если вы удаляете методы из работающего приложения и реорганизуете код, вы создаете вероятность того, что вы внесете новые ошибки и сломаете работающее приложение. Если это приложение является критически важным, влияние может быть огромным и стоить компании больших денег, такие изменения необходимо тщательно планировать и тестировать. В этом случае может быть целесообразно пометить методы как устаревшие, это должно помочь предотвратить их использование людьми в дальнейшей разработке, что облегчит возможный рефакторинг. Однако, если приложение не критично для выполнения задач или вероятность возникновения ошибок невелика, возможно, лучше провести рефакторинг, если у вас есть время. В конечном итоге добавление атрибута [Obsolete] немного похоже на задачу, и его использование зависит от многих факторов.