Как решить проблему возвращаемых объектов?

Question

[Михаил]

Нужно вывести имя клиента платежа. Возьмем такой пример:

class Payment
{
    /**
     * @var Customer|null
     */
    protected $customer;
    
    public function getCustomer()
    {
        return $this->customer;
    }
    
    public function setCustomer(Customer $customer)
    {
        $this->customer = $customer;
    }
}

class Customer
{
    public function getFirstname()
    {
        return 'Name';
    }
}

/** @var Payment $payment */
echo $payment->getCustomer()->getFirstname();

Как вы могли догадаться, бывает так что клиента в платеже нет. Был удален, например. В этом случае возникнет ошибка потому что обращение к null.
Можно создать метод с проверкой (или просто условие):

public function printCustomerFirstname(Payment $payment)
{
    if ($payment->getCustomer()) {
        echo $payment->getCustomer()->getFirstname();
    }
}

/** @var Payment $payment */
printCustomerFirstname($payment);

Но как бы это решил "синьер"?
Возможно что в 2018 уже выработался какой-то best practice.

Answer 1

[Arman]

1.

class Payment
{
    //...
    public function getCustomerFirstname()
    {
        return $this->customer ? $this->customer->firstname : null;
    }
}

2.

echo ($c = $payment->getCustomer()) ? $c->getFirstname() : '[удален]';

Comments

[Михаил]

Да, согласен что вполне вариант. Собственно, я про него написал.
Но он все же: (1 вариант) мусорит в payment, (2 вариант) замусоривает сам код шаблона.

Answer 2

[Антон Шаманов]

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

Comments

[Михаил]

Вы о закона Деметры. Слишком много магии было бы $payment->getCustomer()->getAddress()->getStreet().

[Антон Шаманов]

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

Answer 3

[grinat]

Ошибки нельзя глушить. Ставь try catch если это не критичная ошибка и лови исключение. Либо проверяй что не нул.

Comments

[Михаил]

Код вывода как правило находится в шаблоне. Какие там try cache в шаблоне-то.

[grinat]

Михаил, а причем тут шаблон вообще? В шаблон ты dto вернуть должен.

[Михаил]

grinat, вроде ddd тег не стоит. По канонам ddd в шаблон идет dto, да. Но это не всегда так.
Да и кидать исключение на отсутствие имени... это не исключительная ситуация.

[grinat]

Михаил, если не всегда так, то это вызовет потом кучу проблем, а раз вызовет проблемы, то это говно. А если это говно, то какая разница что и как там проверяется.

Answer 4

[Евгений Бухарев]

ObjectHelper::getValue($payment,'customer.firstname',$defaultValue);

А внутри getValue реализовать проверку существования свойства, если его нет, то выдаем $defaultValue

Comments

[Михаил]

И так можно. А как нужно - мой вопрос.

[Евгений Бухарев]

Однозначно вам никто на данный вопрос не ответит, многое зависит от контекста задачи.

Answer 5

[dmitriy]

1.

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

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

public function printCustomerFirstname(Payment $payment)
{
    if ($payment->getCustomer()) {
        echo $payment->getCustomer()->getFirstname();
    }
}

echo в методе в 2018 - неприлично

3. Проверка решается выбросом исключения

public function getCustomerFirstname(): string
{
    if(!$this->getCustomer()){
       throw new \Exceptions\CustomerNotFound;
    }
    return $this->getCustomer()->getFirstname();
}

try{
$customerName = $payment->getCustomerFirstname();
}
catch( \Exceptions\CustomerNotFound $ex){
$customerName = null;
}