Как проверить добавление в базу в PDO?

Question

[Оптимус Пьян]

Сабж в теме...
Раньше я писал так:

if ( -1 !== mysql_affected_rows() ) {
            echo 'Вы успешно зарегистрированы!';
        } else {
            echo 'Ошибка при регистрации';
        }

В PDO читал есть такая штука как номер последней добавленной записи:
$insertId=$db->lastInsertId();
Но если вставка не удалась она же мне вернёт просто предыдущую и всё...

Answer 1

[Алексей]

Неужели так сложно почитать мануал?

Comments

[Оптимус Пьян]

Читал, но там чё-то так дофига что вот именно этого места просто не нашёл (или не обратил на него внимания), спасибо вам!

[Алексей]

Гулится же в два слова lmgtfy.com/?q=mysql_affected_rows+pdo

[Оптимус Пьян]

@rdifb0 Да да, я заметил, что ссылка у вас не на гугл )) Да и мне бы в голову не пришло так гуглить

Answer 2

[Сергей Протько]

lastInsertId надо вызывать когда вы удостоверились что данные были записаны в базу. А удостовериться можно банально проверив что при выполнении запроса не возникло ошибок. mysql_affected_rows тут не нужен, достаточно делать проверки на наличие ошибок после exec, а еще лучше - сказать PDO в случае чего выкидывать исключения и обрабатывать их.

Answer 3

[Дмитрий Энтелис]

Использовать lastInsertId для проверки удалась вставка или нет - плохая идея.
Небольшой пример с php.net

Пример

Example on a new Row: 
<?php 
$sql = "INSERT INTO city (`city`) VALUES ('Paris') ON DUPLICATE KEY UPDATE `city` = 'Paris"; 
$dbh->query($sql); 
echo $dbh->lastInsertId(); 
?> 

Above displays: 1 
Expected display: 1 

Example on an existing row that gets updated: 
<?php 
$sql = "INSERT INTO city (`city`) VALUES ('Paris') ON DUPLICATE KEY UPDATE `city` = 'Paris"; 
$dbh->query($sql); 
echo $dbh->lastInsertId(); 
?> 
Above displays: 2 
Expected display: 1 (since no new records were inserted)

Поведение вызвано тем что innodb всегда увеличивает автоинкрементный счетчик вне зависимости от того удалась вставка или нет.

На мой взгляд если у Вас есть необходимость проверять успешность вставки, лучше использовать транзакции, но и там есть нюансы (пример оттуда же)

<?php 
try { 
    $dbh = new PDO('mysql:host=localhost;dbname=test', 'username', 'password'); 

    $stmt = $dbh->prepare("INSERT INTO test (name, email) VALUES(?,?)"); 

    try { 
        $dbh->beginTransaction(); 
        $tmt->execute( array('user', '[e-mail]')); 
        $dbh->commit(); 
        print $dbh->lastInsertId(); 
    } catch(PDOExecption $e) { 
        $dbh->rollback(); 
        print "Error!: " . $e->getMessage() . "</br>"; 
    } 
} catch( PDOExecption $e ) { 
    print "Error!: " . $e->getMessage() . "</br>"; 
} 
?>

Comments

[Оптимус Пьян]

Спасибо, да про транзакции читал хорошо что их откатывать можно, сложноваты и громоздки мне кажется они, сейчас попробую по простому PDOStatement::rowCount() если не получится тогда с вашим примером разбираться буду.

[FanatPHP]

НЕ НАДО ПИСАТЬ ЕРУНДЫ. 1. Если какой-то неграмотный дятел 6 лет назад написал на пхп нет какой-то бред, то это не значит, что его надо обязательно повторять. 2. Транзакции для проверки успешности вставки не нужны абсолютно. Рекомендую сначала узнать, что это такое.

[FanatPHP]

@marrk2 не надо разбираться с этим примером, он бессмысленный.

Answer 4

[FanatPHP]

Как проверить добавление в базу в PDO?

По какой-то причне новички этот вопрос задают постоянно.
Но при этом никто не может ответить, ПОЧЕМУ надо проверять успешность вставки.
Только чтобы написать сакраментальную фразу 'Ошибка при регистрации'?

Ну то есть, понятно, что в мануале из прошлого века был код -1 !== mysql_affected_rows(), и его надо обязательно перетащить в новый проект. Но надо иногда задумываься - какой смысл в тех действиях, которые ты совершаешь. Какой юзкейс должен привести к необходимости такой ошибки. И как надо по-человечески её обрабатывать.

Дя тех, кто хочет жить в XXI веке, а не застрять навечно в XX, несколько пояснений:

1. Проверять результат работы функций PDO не нужно никогда.
2. Если хочется вывести сообщение типа "что-то сломалось", надо это не писать после каждой функции, а делать ЦЕНТРАЛИЗОВАННО, в error handler-e
3. Если хочется проверить одну-единственную ошибку, про которую заранее известно, и есть понимание того, как её надо обрабатывать - можно использовать try-catch.

Всё. Кроме этих трёх пунктов никаких проверок на успешность вставки быть не должно

Comments

[Оптимус Пьян]

В целом да, скорее всего действительно нет смысла. Ну бывали у меня случаи когда база на сервере падала, а остальное работало. Сейчас конечно не особо надо, а если оплату на сайте поводить, то вполне может когда ни будь пригодится, а тут хоп в коде уже заготовка есть ))

[FanatPHP]

@marrk2 Опять 25. ЧЕМ тебе поможет эта заготовка при оплате?

[Оптимус Пьян]

@FanatPHP не думал об этом )