1

Привык работать с защищенными полями, начинающимися с _ , переехал на 2017 версию студии с 2015, и теперь она мне рекомендует добавлять ReadOnly для таких свойств.

Зачем студия требует вместо

Private _property As New Hashtable()

это

Private ReadOnly _property As New Hashtable()

1 ответ 1

3
+50

Для приватных полей компилятор проверяет, изменяются ли они где-нибудь после вызова конструктора. Если нет, то рекомендует явно пометить их модификатором readonly в C# или ReadOnly в VB.

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

А тут можно почитать на английском

Проверил поведение на Visual Studio 2017 и 2019, и там есть разница. В VS2017 при вызове каких-либо методов/свойств изменяемого объекта сообщение, предлагающее добавить модификатор readonly/ReadOnly, пропадает. Я так полагаю, это сделано для того, чтобы не вводить разработчика в заблуждение модификатором 'только для чтения', который не запрещает изменение экземпляра объекта, а только лишь препятствует изменению самой ссылки на объект. В VS2019 сообщение присутствует вне зависимости от наличия последующих вызовов методов/свойств, по принципу, который я описал выше. По-видимому, в VS2019 доработали анализатор кода.

введите сюда описание изображения

6
  • Боюсь к конструктору это ни какого отношения не имеет, ели есть методы в которых идет работа с этой переменной, включая вызовы методов если это объект, то рекомендация не показывается. 14 фев 2020 в 6:30
  • Я немного не это имел в виду. Каждое поле/приватное свойство вы либо изменяете после вызова конструктора (в других методах), либо нет. Если изменяете, у компилятора нет к вам никаких вопросов. Если же вдруг значение поля/свойства нигде не меняется, компилятор предполагает, что оно и не должно меняться, и предлагает указать это явно с помощью модификатора ReadOnly, чтобы в дальнейшем никто необдуманно не изменил это значение.
    – aa_talanin
    14 фев 2020 в 8:41
  • Например, если у вас в качестве поля/приватного свойства выступает List<T>, и вы нигде не меняете значение поля, работая исключительно с методами самого объекта List<T>, то есть смысл пометить его как readonly, чтобы кто-нибудь из членов вашей команды не решил очищать этот список с помощью list = new List<T>(); вместо list.Clear(); и не вызвал тем самым лишней нагрузки на сборщик мусора. Это первый пример, который пришел в голову.
    – aa_talanin
    14 фев 2020 в 8:44
  • В том то и дело что нет разницы, присваивание или вызов метода. c2n.me/460d5ru clip2net.com/s/460dddi 14 фев 2020 в 12:23
  • А вот это, на самом деле, уже интересно. Приеду домой, посмотрю.
    – aa_talanin
    14 фев 2020 в 12:46

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.