Нормально ли такое правило валидации?

Question

[Михаил]

Обычно в моделях одноуровневые данные. Сложности возникают с несколькими уровнями.

Нормален ли такой метод валидации?

<?php

class Items
{
    public function rules()
    {
        return [
            [['items'], 'checkItem']
        ];
    }

    protected function checkItem()
    {
        $status = (is_array($this->items) || is_object($this->items)) && $this->items;

        if ($status) {
            foreach ($this->items as $item) {
                $itemModel = new Item;
                
                $status = ($itemModel->load($item, '') && $itemModel->validate());

                if (!$status) {
                    break;
                }
            }
        }

        return $status;
    }
}

Comments

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

Есть для этого специальный валидатор each.

[Михаил]

Александр Макаров: да, недавно узнал, цикл можно убрать. Я сразу решил предложить сообществу идею валидации, по мне, так этого сейчас не хватает https://github.com/yiisoft/yii2/issues/11968

Answer 1

[Максим Тимофеев]

1. Вместо

public function rules()
    {
        return [
            ['items'], 'checkItem'
        ];
    }

наверное

public function rules()
    {
        return [
            [['items'], 'checkItem']
        ];
    }

2. function checkItem() у Вас всегда возвращает одинаковый результат в чем смысл?
3.

foreach ($this->items as $item) {
                $item = new Item;

смысл перебирать $item если сразу же подменять ее значение новым пустым объектом. Да и наверное так new Item()

Так что как по мне - ахинея полная

Comments

[Михаил]

Я больше писал как пример.. Ок, я пофиксил спорные места. checkItem не всегда возвращает одинаковый результат.

[Максим Тимофеев]

Михаил: должно работать, но лучше использовать это www.yiiframework.com/doc-2.0/guide-input-multiple-...

[Михаил]

Максим Тимофеев: это скорее работа с несколькими моделями чем элементами в модели. Я пишу модели для стороннего API ( Что почитать по архитектуре приложения? ) и хочу чтобы была нормальная валидация.