Как пофиксить SQL запрос?

Question

[Risent Veber]

Данный код после подстановки параметров :player_id, :mob_id, :count

do $$
begin
  IF EXISTS (SELECT * FROM player_mob_kills WHERE player_id= :player_id and mob_id= :mob_id) THEN
      UPDATE player_mob_kills SET count = count + :count WHERE player_id= :player_id and mob_id= :mob_id;
  ELSE
      INSERT INTO player_mob_kills (player_id, mob_id, count) VALUES  (:player_id, :mob_id, :count);
  END IF;
end
$$;

иногда падает с ошибкой

SQLSTATE[23505]: Unique violation: 7 ERROR:  duplicate key value violates unique constraint \"player_mob_kills_player_id_mob_id_unique\"\nDETAIL: 
 Key (player_id, mob_id)=(54298, 23) already exists.

Answer 1

[Максим Федоров]

ошибка возникает в следствии попытки добавить данные с одинаковыми Primary Key. Используйте UPSERT если это позволяет Ваша версия постгреса, если не позволяет - в интернете полной информации как реализовать самому его аналог в более старых версиях.

Comments

[Risent Veber]

Собственно база не позволяет т.k. вышеуказанная фича только для версий >= 9.5. Поэтому данный код и написан с целью ее заменить - может ты знаешь еще конретные варианты как это сделать?

[Максим Федоров]

либо с помощью RULE либо отлавливать EXCEPTION
https://job-blog.bullgare.ru/2010/08/postgresql-%D... - вот тут вроде оба способа описаны

Answer 2

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

Может так?
INSERT IGNORE INTO player_mob_kills
Судя по всему ошибка возникает когда игрок уже убивал этого моба. Тогда правильнеебудет добавить столбец с количеством и проверка есть ли такая пара игрок-моб, если есть то счётчик ++
INSERT IGNORE для вашего случая не думаю что подходит судя по контексту запроса...

Comments

[Risent Veber]

Внимательно посмотрите в код - там такая проверка уже есть

Answer 3

[Евгений Быков]

Можно сделать условное добавление строки с нулевым счетчиком и потом выполнить обновление строки для изменения счетчика:

INSERT INTO player_mob_kills (player_id, mob_id, count)
    SELECT :player_id, :mob_id, 0
        WHERE NOT EXISTS(SELECT *
                             FROM player_mob_kills
                             WHERE player_id = :player_id
                               AND mob_id = :mob_id
                        )
UPDATE player_mob_kills
    SET count=count + :count
    WHERE player_id = :player_id
      AND mob_id = :mob_id;