Стоит ли в геттере использовать проверку на null?

Question

[Noortvel]

Стоит ли в геттере использовать проверку на null?
Или лучше инициализировать в Awake()?Просто тогда придется для каждого класса получать переменную по отдельность, где нужна инициализация в Awake().
Вот пример:

ExampleClass ec;

public ExampleClass GetExampleClass(){
    if(ec == null){
        ec = GetComponent<ExampleClass>();
    }
    return ec;

}

Answer 1

[GavriKos]

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

Comments

[Валерий Абакумов]

GavriKos согласен, выглядит как-то сомнительно. И очень уж похоже на реализацию паттерна Singleton, только через свойство. Возможно, конечно, что в Unity3D такая реализация принята за некий шаблон?

[GavriKos]

Валерий Абакумов Нет, юнити тут ни при чем. Вместо GetComponent можно подставить инстанциацию объекта, например - и будет вообще чистый c#.

[Валерий Абакумов]

GavriKos ну, я имею в виду, что вот такая запись, которую привел в своем вопросе Noortvel, принята за некий шаблон, best practices для Unity...

[GavriKos]

Валерий Абакумов Я понял вас сразу. Нет, не принято. GetComponent вообще не является Best Practies.

Answer 2

[MrDywar Pichugin]

Лучше так:
И это вопрос только касательно времени инициализации, похоже на Lazy.
Если объект точно нужен - то в Awake(), если таких объектов много и запрашиваемое свойство нужно не всегда, то можно и так, разница в производительности будет незначительной.

ExampleClass ec;
public ExampleClass GetExampleClass
    {
        get
        {
            return ec ?? GetComponent<ExampleClass>();
        }
    }

Comments

[Даниил Басманов]

Два вопросика в юнити не работают для компонентов, их лучше не использовать.