Тайпкастинг в DTO — можно, или нет?

Question

[driverx18]

Я создал класс DTO, допустим, ProductRaitingMinDTO, в котором я сделал свойства $stars, $productId, $textStatus. Тайпхинтингом в докблоке указал, что свойства соответственно float, int, string. Сразу скажу, что проект, к сожалению на версии 5.6 PHP. Однако из хранилища все данные приходят строками, и вот вопрос, как лучше, в конструкторе DTO сделать

public function __construct($stars, $productId, $textStatus)
    {
        $this->stars = (float) $stars;
        $this->productId = (int) $productId;
        $this->textStatus = $textStatus;
    }

или при создании ДТОшки
делать

new ProductRaitingMinDTO(
                        (float) $stars,
                        (int) $productId,
                        $textStatus
                        );

Какой из вариантов предпочтительнее? Допускается ли тайпкастинг в конструкторе DTO?

Answer 1

[kafkiansky]

Лучше в конструкторе дтошки: так вы никогда не забудете скастить тип.

upd: а еще лучше сделать VO для каждого свойства и валидировать до поступления в дтошку, потому что дтошка - это просто структура данных. В целом, приводить к нужному типу можете и в дтошке, ничего страшного.

Comments

[driverx18]

Просто в ДТО как я знаю нельзя делать никаких валидаций, тайпкастинг не будет ею считаться?

[kafkiansky]

driverx18, обновил ответ. Тайпкастинг не совсем валидация. Вы же просто приводите насильно к нужному типу. Валидацией было бы проверить аргумент на тип и выкинуть ошибку, если тип не подходящий. Так что да, можете приводить к нужному типу в самом конструкторе дто, чтобы не заботиться об этом дальше по коду и случайно не забыть.

[driverx18]

kafkiansky, ок, спасибо!