Best practics ООП?

Question

[Олег Колесников]

Меня интересуют небольшие тонкости в написании классов и их свойств. Они не особо вляют на работу, возможно даже их стоит просто отнести к codeStyle.
Упоминаний об этом в интернете я не нашел, просто возник спор с другом. Возможно вы поможете нам его разрешить.

Суть вопроса - у класса есть свойство. По умолчанию оно, например равно 1, и не является константой.
Лучше его объявить во время объявления переменной, или задать значение, например в контроллере, или просто при получении геттером проверять его на существование.
Если в кратце, то как лучше писать?
1.

class one
{
    private $type = 1;
}

2.

class one
{
    private $type;
    public function __construct()
    {
        $this->type = 1;
    }
}

3.

class one
{
    private $type;
    public function getType()
    {
        return $this->type ?? 1;
    }
}

Хотлеось бы услышать поподробнее почему вы именно так считаете, а так же, если этот класс, является сущностью dataMapper или ActiveRecord

Comments

[Adamos]

One of the best practices of code writing is turning on spell-checker and using pure English instead of pidgin :)
Первые два варианта в пыхе идентичны. Первый не пудрит зря мозги читающему - значит, лучше.
Третий - очевидный говнокод. На кой ляд при каждом обращении к свойству проверять его инициализированность, если у него есть значение по умолчанию и язык позволяет назначить его надежно и читаемо?

Answer 1

[Evgeny Svirsky]

2й вариант без сомнений единственно верный.

Comments

[Олег Колесников]

Как видите, в комментариях все считают иначе.
Можете подробнее рассказать почему именно 2?

[Evgeny Svirsky]

Если говорить с точки зрения ООП, то на мой взгляд верным будет задавать дефалтные значения именно при создании объекта (на момент создания), т.е. в конструкторе. Представьте у вас 20 пропертей и вам нужно будет по классу искать где же там есть дефалтные а где нет. А так в конструкторе наглядно все будет видно. К тому же обязательно придется что-то через условия задавать в итоге и вам придется перенести это в конструктор.
На самом деле на любителя, но я сторонник читабельности кода, поэтому такие вещи делал бы в конструкторе. Хотя 1й способ на мой взгляд выиграет по перфомансу 2й.. но прирост будет совсем небольшой.

Answer 2

[Александр Торопов]

Если это свойство влияет на объект, из-за каких то внешних факторов, то конечно его нужно инициализировать в конструкторе.

class house 
{
    private $type;
    public function __construct($housetype)
    {
        $this->type = $housetype;
    }
}

Конечно в двух словах такое будет сложно объяснить, можно ознакомиться с книгами:
Шаблоны корпоративных приложений
PHP. Объекты, шаблоны и методики программирования

Answer 3

[Boris Köln]

Первый вариант чище.

Comments

[Олег Колесников]

Хотлеось бы услышать поподробнее почему вы именно так считаете, а так же, если этот класс, является сущностью dataMapper или ActiveRecord

[Boris Köln]

Почему "первый вариант" - прочитайте мой ответ еще раз.
Почему "чище" - очевидно, что меньше лишнего кода.