Что лучше Null или пустая строка в ValueObject?

Question

[d99999]

Что лучше использовать строку которая будет в ValueObject или null?

public function __construct(Request $request)
    {
        $this->fullname = $request->request->get('fullname', null);

Весь конструктор:

public function __construct(Request $request)
    {
        $contactInfo = $request->request->get('contactInfo');

        $this->fullname = $request->request->get('fullname', null);
        $this->email = $contactInfo['email'] ?? null;
        $this->phone = $contactInfo['phone'] ?? null;
        $this->linkedInUrl = $contactInfo['linkedInUrl'] ?? null;
        $this->currentCompany = $request->request->get('current_company', null);
        $this->jobId = $request->request->get('job_id', null);
    }

Comments

[Василий Банников]

Смотря какой смысл в вашем случае несёт null и пустая строка.
В идеале - значение всегда должно быть валидное

[Максим Грищенко]

У вас не VO
А хреново знает что, зачем реквестор передаёте

Что за собственно value представляет ваш value object?

Answer 1

[MikUrrey]

Вы получаете и обрабатываете GET-параметры, обычный GET-параметр - это строка.
"По-правильному" -

$this->fullname = $request->request->get('fullname', '');

Создавать NullObject в данном случае - это излишество, PHP - один из немногих ЯП, где строка не является объектом. Сгодился бы и null, но это "не православно".

Comments

[Максим Грищенко]

Православные пустой строки

Пустая строка — это псевдотип

Либо есть значение либо null
Раз без него нельзя

[MikUrrey]

В маленьком офисе у подножия Фудзиямы
Мы с коллегой нашли идеальный код.
Жаль заказчик умер от старости, не увидит такой красоты.