Проверка в sqlite3?

Question

[Workguy]

Вопрос состоит в том, чтобы выполнить проверку на существование данной записи в БД. Пишу как в документации, проводил небольшой дебаг, но всё равно, при попытке добавить уже существующую запись в БД она успешно добавляется... Вот сам код:

import sqlite3

db = sqlite3.connect('test.db')
sql = db.cursor()

sql.execute("""create table if not exists players(
   nick text,
   score integer
)""")
db.commit()

nick = input()
score = int(input())

sql.execute("select nick from players where nick = '{nick}'")
select = sql.fetchone()

if select is None:
   sql.execute(f'insert into players(nick, score) values (?, ?)', (nick, score))
   db.commit()

   print('Successfully done!')
else:
   print('Sorry...')

for value in sql.execute('select * from players'):
   print(value)

Очень прошу помочь!

Comments

[o5a]

sql.execute("select nick from players where nick = '{nick}'")

Отсутствует указание f"" так что он проверяет строку '{nick}' а не значение переменной.
А вообще, как уже ниже упомянули, лучше использовать передачу параметров.

И для проверки уникальности в SQL есть свои методы - первичный ключ, уникальные индексы. Почитайте. Тогда не придется делать проверку наличия записи в базе перед добавлением.

[devdb]

o5a, Если индекс уникальный, то проверка всё же будет делаться - автоматически самой БД, но при этом ещё выдаваться ошибка. В принципе, так проще, но и если самому проверять наличие записи перед добавлением, то разницы практически не будет, imho (если речь не о больших и нагруженных БД).
А если нужно обновление некоторых ячеек в существующей записи, то лучше наверное всё же самому её пытаться предварительно найти. Хотя можно воспользоваться "INSERT ... ON CONFLICT ... DO UPDATE SET".

[o5a]

devdb, если есть уникальность, то не нужно самому делать проверок, можно использовать "ON CONFLICT IGNORE" или "INSERT OR IGNORE" безо всяких ошибок.

Answer 1

[soremix]

Не знаю какую документацию вы читали, но в оффициальной запрос select выглядит явно по другому:

t = ('RHAT',)
c.execute('SELECT * FROM stocks WHERE symbol=?', t)

Put ? as a placeholder wherever you want to use a value, and then provide a tuple of values as the second argument to the cursor’s execute() method. (Other database modules may use a different placeholder, such as %s or :1.) For example:

Если все сделать как в документации:

sql.execute("select nick from players where nick = ?", (nick,))

то все прекрасно работает

Comments

[devdb]

"официальной", в отличие от английского "official" ;)

[Workguy]

Это все хорошо, но эта часть программы работает исправно, а не работает проверка на существование данной записи в БД:
if select is None:
sql.execute(f'insert into players(nick, score) values (?, ?)', (nick, score))
db.commit()
print('Successfully done!')
else:
print('Sorry...')

[devdb]

Workguy, зачем Вам f"" строка?