Как удалять число, которое ввел пользователь, из бд?

Question

[helldess]

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

Код запроса

def set_item_vpn(self, tovarid, colvo):
        return self.cursor.execute("UPDATE tovars_vpn SET colvo = ? WHERE tovarid = ?", (colvo, tovarid,))

Код хендлера

@dp.message_handler(state=purchase.input_amount)
async def check_number(message: types.Message, state: FSMContext):
    async with state.proxy() as data:
        products = data['productsvpn']
    chat_id = message.from_user.id
    balance = BotDB.user_balance(chat_id)
    tovarid = products[0]
    cheque = random.randint(1000000000, 9999999999)
    if number(message.text):
        mes = int(message.text)
        if mes <= products[5]:
            await bot.delete_message(message.from_user.id, message.message_id - 1)
            await bot.send_message(message.from_user.id, f"""✅ <b>Вы успешно купили товар(ы).</b>
➖➖➖➖➖➖➖➖➖➖➖➖➖
 Чек: #{cheque}
 Название товара: {products[1]}
 Куплено товаров: {mes}
 Сумма покупки: {products[4] * mes}₽
 Покупатель: @{message.from_user.full_name} ({message.from_user.id})""", parse_mode='html')
            BotDB.set_item_vpn(tovarid, products[5]-mes)
            await state.finish()
        else:
            await bot.send_message(message.from_user.id, f"""❌ <b>Неверное количество товаров.</b>
➖➖➖➖➖➖➖➖➖➖➖➖➖
 Введите количество товаров для покупки
▶️ От 1 до {products[5]}
➖➖➖➖➖➖➖➖➖➖➖➖➖
 Товар: {products[1]}
 Ваш баланс: {balance}₽""", parse_mode='html')
    else:
        await bot.send_message(message.from_user.id, f"""❌ <b>Данные были введены неверно.</b>
➖➖➖➖➖➖➖➖➖➖➖➖➖
 Введите количество товаров для покупки
▶️ От 1 до {products[5]}
➖➖➖➖➖➖➖➖➖➖➖➖➖
 Товар: {products[1]}
 Ваш баланс: {balance}₽""", parse_mode='html')

Comments

[helldess]

Dr. Bacon, не удаление, а апдейт, т.е число должно выситаться из того, что есть в бд, немного не правильно сформулировал, извиняюсь.

Answer 1

[Александр Нестеров]

1.Где вы видите удаление? У вас в запросе идет UPDATE.
2.А commit после запроса где?

Comments

[helldess]

не удаление, а апдейт, т.е число должно выситаться из того, что есть в бд, немного не правильно сформулировал, извиняюсь.

[helldess]

А commit в самом конце кода бд, после всех запросов

[Александр Нестеров]

helldess, ну значит и результат будет "после".
В приведенном коде обращение к классу BotDB, который отвечает за работу с базой, происходит дважды:

• Для получения баланса;
  
• Для обновления значений;
  

Метод получения баланса выполняется первым и коммитить не может (и не должен).
Метод обновления выполняется вторым и коммит в нем явно не прописан.
И чисто логически - на момент завершения выполнения кода коммит для данного запроса все еще не отработал.

[helldess]

Александр Нестеров, т.е мне нужно просто добавить коммит во 2 запрос? А если мне нужно будет создать ещё несколько таких запросов, то может стоит в каждый тогда коммит добавить или всегда в последний запрос с апдейтом?

[Александр Нестеров]

helldess,
В случае, когда данные обновляются в произвольное время (как в текущем случае, когда запрос выполняется при взаимодействии с пользователем), то да, стоит коммитить каждый раз.

[Александр Нестеров]

helldess,
В случае, когда данные обновляются в произвольное время (как в текущем случае, когда запрос выполняется при взаимодействии с пользователем), то да, стоит коммитить каждый раз.

[helldess]

Александр Нестеров, хорошо, спасибо, попробую - отмечу ваш ответ