Как избежать race condition при вставки новой записи в бд SQL, PHP?

Question

[Артем]

Привет!

Как можно безопасно вставить новую запись в базу данных избежав повторной вставки (при race condition).
Есть такой код:

$dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
  
    $dbh->beginTransaction();

$result = $dbh->exec("INSERT INTO `rating` (`ID`, `count`, `post_id`, `user_id`, `sign_ts`, `date`) SELECT NULL, '4', '1', '1', '".$id."', CURRENT_TIMESTAMP FROM DUAL WHERE NOT EXISTS (SELECT * FROM `rating` WHERE `sign_ts`='".$id."' FOR UPDATE)");

        if($result){
            echo "Запись вставлена";
            echo $dbh->commit();
            } else {
        $dbh->rollBack();
            }

Если есть какие то предложения, буду рад вашему ответу, спасибо.

Answer 1

[Виктор Кожухарь]

Создайте в таблице какой-то уникальный индекс, основанный на ваших данных, и второй раз одни и те же данные просто не получится вставить.

Comments

[ctoburningmvp]

ALTER TABLE rating ADD UNIQUE (`sign_ts`, `post_id`, `user_id`);

$dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

try {
    $stmt = $dbh->prepare("INSERT IGNORE INTO `rating` (`count`, `post_id`, `user_id`, `sign_ts`, `date`) VALUES ('4', '1', '1', :sign_ts, CURRENT_TIMESTAMP)");
    $stmt->execute([':sign_ts' => $id]);

    if ($stmt->rowCount() > 0) {
        echo "Запись вставлена";
    } else {
        echo "Запись уже существует";
    }
} catch (Exception $e) {
    echo "Ошибка: " . $e->getMessage();
}

[Ипатьев]

ctoburningmvp, только вот это всё - try, catch, echo getMessage - надо убрать. Этот код не имеет смысла при разработке и представляет угрозу в продакшене.

Answer 2

[Руслан Федосеев]

используйте оператор replace вместо insert.