Ошибка sqlite3.IntegrityError: UNIQUE constraint failed: users.user_id, что делать?

Question

[d3m0l]

conn = sqlite3.connect('db.db', check_same_thread=False)
cursor = conn.cursor()

def db_db_val(user_id: int, user_name: str, username: str):
    cursor.execute('INSERT INTO users (user_id, user_name, username) VALUES (?, ?, ?)',(user_id, user_name, username))
    conn.commit()



@bot.message_handler(commands=['db'])
def test(message):
   # try:
        bot.send_message(message.chat.id, 'test')
        if message.from_user.id in cursor.execute('SELECT user_id FROM users'):
            bot.send_message(message.chat.id, 'upd')

            cursor.execute(f'UPDATE users SET user_id = {message.from_user.id} WHERE user_id = {message.from_user.id}')
            cursor.execute(sql)
            conn.commit()

        else:
            bot.send_message(message.chat.id,'add')

            us_id = message.from_user.id
            us_name = message.from_user.first_name
            username = message.from_user.username

            db_db_val(user_id=us_id, user_name=us_name, username=username)
  #  except:
  #      bot.send_message(message.chat.id,'123')

в бд добавляет а update не работает, не понимаю в чем проблема, помогити

ошибка: sqlite3.IntegrityError: UNIQUE constraint failed: users.user_id

Answer 1

[Vindicar]

message.from_user.id in cursor.execute('SELECT user_id FROM users')
На кой нужно делать выборку всех пользователей, чтобы потом питоном искать в ней данные? Используй ключевое слово WHERE, и используй prepared statement для подстановки значения из переменной, а не форматирование строк.

f'UPDATE users SET user_id = {message.from_user.id} WHERE user_id = {message.from_user.id}'
"Если id = 1, то сделай id равным 1". На кой?!
И да, не используй форматирование строк для формирования строки запроса. Используй prepared statements.

Например так:

ids = cursor.execute('SELECT user_id FROM users WHERE user_id = ?', (message.from_user.id,))

> не понимаю в чем проблема
В недостатке знаний по основам SQL, и в попытке писать ботов, не разобравшись в основах.

Comments

[d3m0l]

да, только учусь чет сделать в sql, кстати спасибо за гайд

[o5a]

d3m0l, помимо уже указанных проблем, изначальная идея (проверить, что пользователь в базе, если нет - обновить) не работает еще потому, что результат SELECT возвращается в виде списка кортежей (по 1 на каждую строку данных), а не списка самих данных user_id.
cursor.execute('SELECT user_id FROM users')
вернет данные в виде [(1,), (2,)...]
а не просто [1,2,...]
Соответственно и проверка

if message.from_user.id in cursor.execute('SELECT user_id FROM users'):

в таком виде работать не будет, т.к. она получается в виде
if 1 in [(1,), (2,)...]: