Почему resharper предлагает поля только для чтения

Question

Почему ReSharper предлагает поле для чтения для 'настроек' в моем примере ниже?

Если я правильно понимаю, вы должны использовать модификатор readonly, если вы изменяете это поле только в конструкторе, но в моем примере я также изменяю его другим методом в том же классе.

Что мне не хватает?

public partial class OptionsForm : Form 
{ 
    private Settings settings; 

    public OptionsForm(Settings s) 
    { 
     settings = s; 
    } 

    private void SaveData() 
    { 
     settings.ProjectName = TextBoxProject.Text; 
    } 
} 

Answer 1

Если ссылочный тип объявлен как readonly, указатель является неизменным, но не объектом, на который он указывает. Это означает, что:

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

Читайте подробную статью об этом

Mark a C# class data member as readonly when it’s read only

Answer 2

Вы не изменяете настройки вне конструктора, объект тот же, что и в SaveData. Свойства объекта могут меняться, но не ссылки на объекты, поэтому, похоже, это имеет смысл с точки зрения Resharper.

Answer 3

Метод SaveData() не изменяет переменную настроек, она меняет один из ее свойств. Содержимое настроек (к чему это относится) устанавливается только в конструкторе.

Answer 4

Помните главную причину стандартов кодирования и шаблонов проектирования, это сделать, чтобы облегчить для людей, чтобы понять ваш код.

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

Однако, поскольку объект, на который указывает поле readonly, может иметь свое состояние, обозначение поля, которое может быть введено в заблуждение, может время от времени вводить в заблуждение. Так что думайте о погоде, это помогает читателю (например, человеку) вашего кода понимает ваш дизайн или нет.

Если значения в объекте, которое поле указывает на изменения в течение жизни объекта, я не думаю, что поле должно быть помечено только для чтения. (Например, указанный объект должен вести себя так, как если бы он был неизменным к тому времени, когда был вызван подрядчик в вашем классе)

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

Answer 5

На самом деле, вы правы, а Resharper ошибается. Поле следует указывать только для чтения, если оно является неизменным во всей полноте.В вашем примере, если вы сделаете это только для чтения и включите анализ кода Microsoft, он предупредит вас, что настройки имеют изменяемые свойства.

Answer 6

Это, кажется, немного странно, я не могу себе представить, что Эрик Липперт и др. Не считали очевидным фактом, что создание ссылки неизменяемой не приводит к тому, что экземпляр, на который указывает эта ссылка, неизменен, хотя упомянутый Правило анализа кода поддерживает представление выше (http://msdn.microsoft.com/en-us/library/ms182302(v=VS.100).aspx).

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

Я вижу, как использование ссылки является неизменяемой, а не экземпляром, на который указывает.

Answer 7

ReSharper предлагает сделать «настройка» только для чтения:

readonly private Settings settings; 

public OptionsForm(Settings s) 
{ 
    settings = s; 
} 

потому, что при сканировании кода он делает вывод, что ваше поле «Настройка» происходит только в конструкторе для этого же класса.

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

После отмечен только для чтения компилятор пометит как предупреждения различных злоупотреблений на местах, таких как:

The left-hand side of an assignment must be an l-value 

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

также использование ref и out Ограничители параметров ограничены.

Следуя предложению ReSharpers, вы будете предупреждены компилятором, если вы попытаетесь неправильно использовать поле, которое вы действительно не собираетесь изменять после инициализации.