Почему не меняется значение в sqlite?

Question

[mollya]

Сам код:

@bot.callback_query_handler(func=lambda callback: callback.data)
def check_callback_data(callback):
    db = sqlite3.connect("mistify.db")
    c = db.cursor()

    if callback.data == "btn1":
        c.execute(f"UPDATE checker SET vape = 'lol' WHERE user_id = {str(callback.message.from_user.id)}")

почему то значение в таблице не меняется, а если вместо "{str(callback.message.from_user.id)}" подставить свой id, то все работает

Comments

[Everything_is_bad]

да когда же тебе подобные начнут читать доки, очередной с f-string для sql запросов, опять говновидосиков на ютубе посмотрел?

[mollya]

Everything_is_bad, давно пишу с использованием sql, но такое впервые

[mollya]

mollya, не помогло
код:

@bot.callback_query_handler(func=lambda callback: callback.data)
def check_callback_data(callback):
    db = sqlite3.connect("mistify.db")
    c = db.cursor()

    if callback.data == "btn1":
        c.execute("UPDATE checker SET vape = 'lol' WHERE user_id = ?", (str(callback.message.from_user.id),))
    if callback.data == "btn2":

[Даниил Гладыш]

mollya, а колонка user_id какой тип имеет в бд?

[mollya]

Даниил Гладыш, text

[Даниил Гладыш]

mollya, а db.commit() ниже где то прописано?

[mollya]

Даниил Гладыш, да

[Даниил Гладыш]

mollya, ну где именно? Скиньте весь код check_callback_data()

[mollya]

Даниил Гладыш,
код:

def check_callback_data(callback):
    db = sqlite3.connect("mistify.db")
    c = db.cursor()

    if callback.data == "btn1":
        c.execute(f"UPDATE checker SET vape = 'lol' WHERE user_id = {callback.message.from_user.id}")
    if callback.data == "btn2":
        pass
    if callback.data == "btn3":
        pass
    if callback.data == "btn4":
        pass
    if callback.data == "btn5":
        pass
    if callback.data == "btn6":
        pass
    if callback.data == "btn7":
        pass
    if callback.data == "btn8":
        pass
    if callback.data == "btn9":
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        one = types.KeyboardButton(text="1 ")
        two = types.KeyboardButton(text="2 ")
        three = types.KeyboardButton(text="3 ")
        markup.add(one, two, three)
        bot.edit_message_text(chat_id=callback.message.chat.id, message_id=callback.message.id, text="Выберите опцию ")
        bot.send_message(chat_id=callback.message.chat.id, text="1. Смотреть объявления .\n2. Продать товар .\n3. Изменить город .", reply_markup=markup)

    db.commit()
    db.close()

[Даниил Гладыш]

mollya, {callback.message.from_user.id} попробуй поменять на '{callback.message.from_user.id}'

[mollya]

Даниил Гладыш, не помогло

[Сергей Горностаев]

mollya,

давно пишу с использованием sql, но такое впервые

Но prepared statement так и не освоил.

[mollya]

Сергей Горностаев, мне он значит был не нужен

[mollya]

Сергей Горностаев, что это?

[mollya]

Даниил Гладыш, просто дело в callback, но как исправить, не знаю

[Сергей Горностаев]

mollya, правильный способ делать параметризированные sql-запросы. Формирование запросов конкатенацией или интерполяцией строк - это очень плохая практика, за которую в порядочном обществе бьют.

[mollya]

Сергей Горностаев, ничего не понял, что вы написали, но я уже сам разобрался

Answer 1

[Даниил Гладыш]

vape = 'lol'
Здесь ты сам строку обернул в кавычки, а здесь кавычки откуда должны появиться?
user_id = {str(callback.message.from_user.id)}

Во время работы программы вот это всё
{str(callback.message.from_user.id)}
"Поменяется" на строку с id, но так как ты используешь f форматирование, содержимое этой подстроки вставится в строку без кавычек, а они должны быть, по крайней мере если user_id - строковый тип данных.

spoiler

Comments

[mollya]

не помогло

Answer 2

[harrybrown219]

If you're after a clean and modern way to enjoy nicotine without the mess or maintenance, there’s a lot to appreciate in today’s options. The Tornado 9000 offers an impressive puff count that lasts much longer than many similar products, making it a great value. It also features smart draw activation, so there’s no need to press buttons or deal with settings. With its bold flavors and smooth finish, the Tornado 9000 has become a go-to for people who want a no-fuss experience that still delivers strong satisfaction.