В чём ошибка в коде?

Question

[Kekatoz]

У меня есть пуста таблица в MySQL:

CREATE TABLE IF NOT EXISTS games(
    id INTEGER AUTO_INCREMENT PRIMARY KEY,
    game_id INT UNIQUE,
    name VARCHAR(255),
    date DATE
);

Через pymysql в python я прописываю команду:

cursor.execute("INSERT games(game_id, name, date) VALUES (2, 'Ships in the ocean', 20180212)")

и получаю следующую ошибку:

pymysql.err.IntegrityError: (1062, "Duplicate entry '2' for key 'games.game_id'")

таблица при этом пустая.

как решить проблему?

Comments

[Георгий Еремеев]

таблица при этом пустая.

Вообще нет записей? А если есть, то наверно есть записи с game_id = 2 и вам не нужен UNIQUE

[Kekatoz]

Георгий Еремеев, таблица полностью пустая. никаких значений нет.

[Георгий Еремеев]

Kekatoz, вы где то еще делали game_id INT UNIQUE ? Возможно они использую один индекс по умолчанию.

[Kekatoz]

Георгий Еремеев, делал но в другой таблице. они никак не пересекаются.

[Slava Rozhnev]

Ошибка не воспроизводится MySQL fiddle
Возможно триггер на вставку данных?

[Георгий Еремеев]

Kekatoz, все же попробуйте поставить unique через отдельное создание индекса дав ему имя

create unique index games_game_id_uindex
	on games (game_id);

[Kekatoz]

Георгий Еремеев, всё так же, ничего не изменилось ;(

[Георгий Еремеев]

Kekatoz, а репликация есть? )))

[Kekatoz]

Slava Rozhnev, в этом и проблема, что ввод непосредственно через workbench проходит нормально. через python жалуется на дубликат.

[Kekatoz]

Георгий Еремеев, не знаю что это) но подключение настроено. то есть извлечь данные из таблиц я могу. а занести нет

[Георгий Еремеев]

Kekatoz, хм... python и workbench конектятся точно к одному и тому же месту?...

[Kekatoz]

Георгий Еремеев, в mysql host: DESKTOP-HHK5M8I
В python : localhost

база данных хранится у меня на компьютере

в select user() root@localhost

[Георгий Еремеев]

Kekatoz, у меня есть еще вариант который подходит :D Все это обернуто в TRANSACTION а инсерт каким то чудом два раза выполняет и по этому ты видишь ошибку но не видишь данных :)

[o5a]

Kekatoz, судя по ошибке, Вы ошибаетесь насчет заполненности таблицы, и на момент вставки данные с game_id=2 там уже есть.

Не видя кода нельзя сказать, где именно ошибка. Возможно, занесение записей делается повторно в каком-нибудь цикле.

То, что не видите записей в таблице извне после ошибки, не значит, что записей там на момент вставки. Просто при ошибке всё откатывается.

Можете перед этой вставкой для себя сделать проверку

cursor.execute('select * from games where game_id = 2')
print(cursor.fetchall())

И в целом нет смысла делать какой-то дополнительный уникальный game_id если в таблице уже есть первичный ключ id, он и служит уникальным идентификатором. Если же действительно нужен дополнительный код игры (типа внешнего UUID), то и генерироваться он должен автоматически.

[Kekatoz]

блок кода на вставку:

try:

cursor = connection.cursor()
for element in range(0, len(list_with_values), 3):
insert_request = "INSERT games(game_id, name, date) VALUES (%d, '%s', %d)" % (
77, list_with_values[element + 1], list_with_values[element + 2])
cursor.execute(insert_request)
connection.commit()
cursor.close()

finally:
connection.close()

[o5a]

Kekatoz, ну и в результате game_id фиксирован, и если список list_with_values больше 3 элементов, то он снова попытается вставить в таблицу запись с тем же самым game_id, вызывая ошибку.
И кстати почему здесь код 77, а в изначальном ошибка по коду 2 ?

[Vitsliputsli]

Раз MySQL пишет, что запись есть, значит она есть. При поиске ошибок, нужно локализовывать проблему. В данном случае, не помешало бы убрать циклы и оставить только вставку и все бы стало на свои места.

[Kekatoz]

Vitsliputsli, но мне нужен цикл, который будет проходить по каждому третьему элементу массивая начиная с нулевого индекса, и записывать значения в бд в виде "элемент, элемент +1, элемент + 2"

[Kekatoz]

исходя из вышенаписанного я подкорректировал код:

try:
cursor = connection.cursor()
for element in range(0, int(len(list_with_values) / 3), 3):
insert_request = "INSERT games(game_id, name, date) VALUES (%d, '%s', %d)" % (
list_with_values[element], list_with_values[element + 1], list_with_values[element + 2])
cursor.execute(insert_request)
connection.commit()
cursor.close()

но записывает только первое значение, остальные игнорируются

[o5a]

Kekatoz, ну хорошо, хотите заносить в game_id свое значение из списка, заносите. Но зачем было добавлять деление списка на 3, если цель - пройтись по всему списку. Нужно было и оставлять как до этого
for element in range(0, len(list_with_values), 3):
И для такого конечно же каждое третье значение в этом списке (начиная с нулевого) не должны повторяться, иначе опять словите ошибку уникальности.

Answer 1

[ThunderCat]

Во первых нахрена было давать полю автоинкремент если вы его руками будете вставлять? Это не к тому чтобы убрать автоинкремент, это к тому что это поле в запросе указывать вообще не надо.

Во вторых - у таблицы которую вы уже использовали, наверняка уже существовал id=2, и она это "помнит", то что вы не можете создать такой ключ который уже использовался гарантирует консистентность данных.

UPD: Уники емнип так же попадают в индекс использования, то есть второй уник вы не создадите, особенно если это поле как-то связано с другими через внешний ключ.

Comments

[Георгий Еремеев]

Он в id не инсертит

[Kekatoz]

автоинкремент на поле id а не game_id. и ошибку о дубликате выдаёт независимо от того, какой game_id я ввожу. пробовал даже числа которые я не юзал

[ThunderCat]

Kekatoz, связь по внешнему ключу есть к этому полю?

[Kekatoz]

ThunderCat, есть. в другой таблице

[ThunderCat]

Kekatoz, Значит вставить/удалить/изменить это поле можно только в связке с внешним ключом.

[Kekatoz]

ThunderCat, я неправильно сформулировал. связь есть от этого поля. не к нему

[Vitsliputsli]

ThunderCat,

у таблицы которую вы уже использовали, наверняка уже существовал id=2, и она это "помнит"

Записи нет, но СУБД ее "помнит" и не дает использовать такой же id? Это как?