Как в Yii2 не падать с ошибкой при повторе в БД?

Question

[Padre]

При записи данных в БД

$phone = new Users();
$phone ->name= '89404445566';
$phone ->save();
if($phone->save()) {
   return 'Ок';
} else {
   return 'Ошибка';
}

Происходит падение с ошибкой Duplicate entry for key но это понятно почему, вопрос как это обработать что бы не вызывать падения приложения?

1) Первый вариант INSERT IGNORE его при способе ActiveRecord можно задействовать, если да, то как указать добавление IGNORE в запрос? Или тут надо обязательно переходить на Data Access Objects что бы ручками запрос писать

2) Второй вариант я так понимаю можно поймать исключение, но исключение - это когда дальнейшая работа приложения невозможна, а тут как раз хочется продолжить и показать нормальную ошибку?

Answer 1

[Дмитрий]

Добрый вечер.
Уберите первое сохранение в базу $phone ->save();

P.S.

Хочу добится что бы если пользователь по ошибке или специально ввёл одинаковые данные приложение не упало, а корректно сказало ему "извините такая запись уже есть" и всё.

Для этого, в yii2, существует валидация.
В данном случае Вам подойдёт вот этот встроенный валидатор, который будет проверять передаваемые пользователем данные на уникальность. При попытке передать одинаковые данные он выдаст предупреждение.
Вот так, например, можно проверять адрес e-mail на уникальность

// в модели, в правилах валидации пишите
['email', 'unique', 'targetClass' => self::className(),
                                'message' => 'Извините, такой e-mail уже существует в базе данных',

Так можно проверять и логины и телефоны и всё что пожелаете...

Comments

[Padre]

Добрый
Что-то не прокатило...

[Дмитрий]

Padre: Сначала Вы показываете такой код

$phone ->save();
if($phone->save()) {

Потом
if($add->save()){
Вы определитесь и опишите проблему более понятно, с примерами кода и код не надо выкладывать скриншотами, не ленитесь писать.

[Padre]

Не ленюсь писать, просто в приложении имя переменной поменял, код остался полностью аналогичный примеру. Скриншот кинул что бы вы увидели что приложение указывает именно на ошибку в этой строчке и что у меня нет ->save() в другом месте

[Дмитрий]

Padre: А полный текст ошибки не поленились написать?

[Дмитрий]

Padre: Структура таблицы какая?
Есть уникальный ключ?
Возможно из-за этого.

[Padre]

slo_nik: Integrity constraint violation – yii\db\IntegrityException
SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry 'QI' for key 'PRIMARY'
Caused by: PDOException
Ну и дальше уже стак трейс по строкам файлов

[Дмитрий]

Padre: Насколько я понимаю, Вы пытаетесь записать значение, которое должно быть уникальным, то есть, неповторимым.

[Padre]

slo_nik: конечно из-за этого, идёт дубль ключа в таблице country столбце code вопрос-то и был как сделать что бы приложение ошибку базы проглотило и не ругалось

[Padre]

slo_nik: да всё верно, я пытаюсь записать значение которое уже есть в БД в primary key и делаю это специально что бы была ошибка базы

[Дмитрий]

Padre: Структура таблицы какая?

[Padre]

slo_nik: вопрос вообще не об этом немного, но если вы настаиваете:
code char(2) utf8_general_ci PRIMARY
name char(52) utf8_general_ci
population int(11)

[Дмитрий]

да всё верно, я пытаюсь записать значение которое уже есть в БД в primary key и делаю это специально что бы была ошибка базы

А зачем? Чего Вы хотите добиться?

[Padre]

slo_nik: Хочу добится что бы если пользователь по ошибке или специально ввёл одинаковые данные приложение не упало, а корректно сказало ему "извините такая запись уже есть" и всё.

[Nik Gubin]

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

[Дмитрий]

Padre:

Хочу добится что бы если пользователь по ошибке или специально ввёл одинаковые данные приложение не упало, а корректно сказало ему "извините такая запись уже есть" и всё.

Для этого, в yii2, существует валидация.
В данном случае Вам подойдёт вот этот встроенный валидатор, который будет проверять передаваемые пользователем данные на уникальность.

Answer 2

[Дмитрий Дарт]

Если try/catch юзать то от Yii2

try {
    //...
}
catch (\yii\db\Exception $exception) {
    //...
}

Answer 3

[Nik Gubin]

С ActiveRecord не работал, может там есть способы, как сделать изначально проверку, но можно использовать следующее:

try {
    $phone = new Users();
    $phone->name = '89404445566';
    $phone->save();
    
    // Если на момент "save" возникнет ошибка, то дальше код выполняться не будет и не вернёт true
    return true;
} catch (Exception $exception) {
    // Тут возникла какая-то ошибка при сохранении в БД
    // Можете логировать или вернуть из метода значение false
    // Например:
    return false;
}

Что делать внутри try или catche по итогу, решать вам, но этот способ точно отловит любое исключение и не уронит весь скрипт.

P.S. Ах, вы сами писали про такой вариант. Но при "ловле" исключения никто не мешает нам продолжить работу дальше.

Comments

[Padre]

Блин, это какая-то напасть, он и так падает...

[Nik Gubin]

Padre: а почему бы изначально не проверять наличие записи по уникальному полю, которое и конфликтует у вас?

[Padre]

Nik Gubin: да вот, так и придётся это делать

[Nik Gubin]

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