Как правильно: агрегация DTO-объекта, или дублирование свойств в классе?

Question

[topuserman]

Класс Order, в конструкторе принимает DTO-объект с полями $orderId, $orderAmount и т.д.

Как правильно DTO-объект внедрять в класс ?

class Order {
    private int $orderId;
    private float $orderAmount ;
   // .. и тут все свойства из DTO

    public function __construct(OrderDTO $orderDTO) {
           $this->orderId = $orderDTO->orderId;
           // ...
    }
}

или агрегировать объект DTO прямо в класс:

class Order {
    public OrderDTO $orderDTO;

    public function __construct(OrderDTO $orderDTO) {
           $this->orderDTO= $orderDTO->orderId;
    }
}

?

Comments

[dmitriy]

если предположить что class Order
- сущность

то какую роль выполняет OrderDTO ?

[Максим Федоров]

Почему Order принимает в конструкторе не нужные обьекту атрибуты (id , amount и прочее), а неведомые транспортные обьекты? :)

Хотя может быть фабричный метод, но не желательно

[topuserman]

dmitriy, чтобы два десятка параметров в конструктор не передавать

[dmitriy]

Для этого есть value object, но вашем случае это не нужно

Answer 1

[galliard]

DTO вообще не должен в конструктор заходить, а разбираться при вызове конструктора, то есть:

$order = new Order($orderDTO->orderId, $orderDTO->orderAmount);

Answer 2

[Alex Wells]

Ну хранить поля в Order'е, конечно, отдельно. ДТО служит для передачи данных, а не хранения в модели. Какой тогда вообще смысл от Order?

Можешь в конструктор принимать OrderDTO и там его распаковывать, если так удобней.