Почему не меняется значение в 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]

не помогло