Почему не работает unique в модели Yii2?

Question

[lolka02]

Вобщем есть консольное команда Hello там есть action Test. при запуске одновременно несколько таких процессов получается дубли и не работает валидация unique в модели

public function actionTest()
    {
        $email = '[e-mail]';
        $doubleModels = Test::findOne(['email' => $email]);
        if (!$doubleModels) {
            $model = new Test();
            $model->email = $email;
            $model->save();
        }
    }

А в модели Test.php есть валидация

/**
     * {@inheritdoc}
     */
    public function rules()
    {
        return [
            ['email', 'unique'],
            [['email'], 'string', 'max' => 255],
        ];
    }

для проверки сделал несколько одинаковых крон задач и одновременно запускаю
23 */1 * * * cd /var/www/domain/data/www; /usr/bin/php yii hello/test
23 */1 * * * cd /var/www/domain/data/www; /usr/bin/php yii hello/test
23 */1 * * * cd /var/www/domain/data/www; /usr/bin/php yii hello/test
23 */1 * * * cd /var/www/domain/data/www; /usr/bin/php yii hello/test

Получается 4 записи , хотя в модели есть unique у поле email.

Как исправить такие косяки? чтобы в рабочих скриптах такого не было

Answer 1

[Максим]

Попробуйте так

public function actionTest()
    {
        $email = '[e-mail]';
        $emailExists = Test::find()->where(['email' => $email])->exists();
        if (!$emailExists) {
            $model = new Test();
            $model->email = $email;
            $model->save(false);
        }
    }

Comments

[lolka02]

зачем $model->save(false);

[Максим]

lolka02, затем, что у вас проверка существует ли email находится ниже. У вас помимо email могут быть другие проверки. В нашем случае их учитывать не нужно. Поэтому false. Да и вообще, из модели валидации лучше выносить в отдельные формы.

Unique делает точно такой же запрос. Зачем вам два запроса проверки?

[lolka02]

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

[Максим]

lolka02, я вам показал хороший вариант для проверки уникальности. Не хватает только привести email к нижнему регистру. Что с вашей валидацией модели не понятно. Пришлите всю модель. Тогда смогу подсказать что не так с вашей проверкой.

[lolka02]

Максим, да там все ок , была проблема при одновременном запуске, ваше условие может и подойдет а вот $model->save(false); зачем false ставить и игнорировать другие проверки вопрос возник.

[Максим]

lolka02, долгая тема для обсуждения. Предположим, что есть админ и пользователь. Пользователь обязательно должен заполнить email, а админу разрешается редактировать профиль без обязательного заполнения.

Чтобы не было таких конфликтов, то и валидацию мы отключаем в модели. Тогда все данные проходят. Вы скажите: какая глупость, есть сценарии. Да, такое есть. Но если вам приходилось сталкиваться с множеством условий в правилах валидации, то вы меня поймёте, как проект становится хламом.

У вас возник вопрос. Тогда как валидировать данные, если не в сущности? В форме! На каждое действие мы просто создаём новую форму со своими правилами валидации. А из модели их убираем. После валидации данные переносим в модель и сохраняемся.

При таком подходе у вас никогда не будет проблем с валидацией. А код ваш станет чище.

Answer 2

[Arman]

unique делает сам запрос и проверяет наличие, выходит несколько процессов успевают одновременно и проверить и записать, поэтому на стороне БД ставим уникальный ключ на поле эл. почты

Comments

[lolka02]

тогда придется try catch еще добавить. иначе скрипт прервется при добавлении записи

[Arman]

lolka02, смотря какая задача стоит

[Максим]

Уникальное после в БД имеет место быть, но это не то, что нужно. А как проверить валидный email или нет? [e-mail] и [e-mail] будут разными для базы данных. Да и вообще приложение не должно надеяться на базу данных. Лучше это перенести в приложение.

[lolka02]

Arman, ну циклом добавляются данные. следовательно если в базе сделать уникальным это поле, то скрипт прервется mysql ошибкой. Так? чтобы не было такого придется сделать try catch

[Arman]

lolka02, если немного забыть про Yii и идет массовая вставка, то еще есть INSERT VALUES($arrayValues) - одним запросом несколько записей, и "insert on duplicate key update", последнее вроде тоже поддерживается Yii. Есть есть "INSERT IGNORE" - не будет кидать ошибку, а просто будет считать что все хорошо, но ничего не запишет в БД

[Arman]

Максим, одно другому не мешает, я ведь не говорю убирать валидатор unique или другие валидаторы, только вот все эти валидаторы лишь страхуют, что БД будет хранить и как только ей решать. Например, валидатор говорит что это целое число, а в БД поле строка, то приложение назад получит целое число или число в виде строки? также bool и т.д.

[Максим]

Arman, именно. Одно другому не мешает. Но вопрос все же про валидацию на стороне клиента) а если это nosql база?)