Проблема с get и set. Почему не работает?

Question

[YoppaLotus]

Почему так не работает?

{ get 
            {
                if (this == null) return " ";
                return this.ActNumber;
            }
            set
            {
                this.ActNumber = value;
            }
        }

При использовании геттера вываливается ошибка Access Violation

Comments

[Foggy Finder]

У вас ошибка где-то еще. Покажите больше кода.

Здесь, конечно, проверка лишняя:

if (this == null) return " ";

Но к Access Violation отношения иметь не должно.

[YoppaLotus]

Foggy Finder, дело в том, что при замене на дефолтные геттеры и сеттеры(т.е. { get; set; }) все работает. И почему проверка лишняя? Класс, для которого определен геттер поля сам является полем другого класса, т.е. геттер был написан для того, чтобы при попытке обращения к неинициализированному полю дочернего класса из родительского программа не завершалась с ошибкой.

[Adamos]

YoppaLotus, так вам как раз попытка вызвать геттер у null и возвращает эту ошибку.

[Foggy Finder]

Adamos, нет, вызывать геттер у null даст NRE.

YoppaLotus у вас случайно код не такой?

string ActNumber { 
            get 
            {
                if (this == null) return " ";
                return this.ActNumber;
            }
            set
            {
                this.ActNumber = value;
            }
}

если да, то это неправильно. Вы получите OutOfMemory, в свойствах вы должны обращаться к приватным полям, но никак не к самому свойству.

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

Вы должны обработать null в родительском классе:

return this?.ActNumber ?? " ";

[Foggy Finder]

В моем предыдущем комментарии опечатка, там должно быть не

return this?.ActNumber ?? " ";

а

return field?.ActNumber ?? " ";

[YoppaLotus]

Foggy Finder, точно ведь, недоглядел. Спасибо

[Foggy Finder]

YoppaLotus, напишите ответ или мне написать?

[YoppaLotus]

Foggy Finder, напишите

Answer 1

[Foggy Finder]

Свойства по сути маскируют в себе обычные методы для извлечения/установки полей.
Представим что есть такой метод:

public string GetValue()
{
    return GetValue();
}

Здесь содержится только новый рекурсивный вызов. Какой результат будет получен при вызове такого метода? Естественно, StackOverflowException.

Поэтому каждому свойству должно соответствовать какое-то поле.

private string _actNumber;
public string ActNumber {
    get { return _actNumber; }
    set { _actNumber = value; }
}

Во многих случаях не требуется какая-то определенная проверочная логика, поэтому в языке предусмотрены специальные конструкции (как, например, авто-свойства)

public string ActNumber { get; set; }

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

----

По поводу второй части вопроса. Обрабатывать null - результат нужно на самом верхнем (допустимом) уровне.

В C# для этого есть целый набор null-операторов:

?., ??, ??=

о которых можно прочитать, например, в справке:

?? and ??= operators

Null-conditional operators ?. and ?[]

Comments

[freeExec]

Естественно, OutOfMemory.

Вообще-то StackOverflow

[Foggy Finder]

freeExec, спасибо, сейчас исправлю