Как сделать проверку на нулевое значение Sqlite3?

Question

[CODEFER]

Здравствуйте. Сделал проверку на нулевое значение, но она не работает.
Код:

cursor.execute("UPDATE promo SET activations = activations - 1 WHERE promo_code=?", (promo_code,))
    conn.commit()
    
    if activations <= 0:
        bot.send_message(user_id, "Промокод уже активирован максимальное количество раз.")
    else:
        bot.send_message(user_id, "Промокод активирован!")

Суть работы кода:
в столбце activations (в базе данных sqlite3) есть значение. Любое.

С каждой активацией промокода, строчка:

cursor.execute("UPDATE promo SET activations = activations - 1 WHERE promo_code=?", (promo_code,))
    conn.commit()

отнимает - 1 от значения которое есть в столбце activations. И когда значение достигает нулевого значения, строчка:

if activations <= 0:
        bot.send_message(user_id, "Промокод уже активирован максимальное количество раз.")
    else:
        bot.send_message(user_id, "Промокод активирован!")

Должна проверить на нулевое значение в этом столбце и бот должен сообщить, что промокод уже активирован максимальное количество раз. Или если значение не нулевое, то бот пишет: промокод активирован!
Но эта строчка работает некорректно. После нуля пошли отрицательные числа соответственно , а код продолжает писать "промокод активирован!"

Подскажите пожалуйста.

Answer 1

[Руслан .]

А как переменная activations связана с полем activations в таблице БД?

Comments

[CODEFER]

У меня ещё есть код на генерацию промокодов, там есть переменная activations , чтобы в бд записывалось то значение, которое я ввожу.

А вот с таблицей в бд походу не контачит. Как сделать эту проверку?

[Руслан .]

CODEFER, нужно начитать отдельным запросом (через select) после update значение поля
activations из таблицы promo в переменную и затем в проверках использовать значение этой переменной.

[Руслан .]

CODEFER, как то так:

res = cursor.execute("SELECT activations FROM promo WHERE promo_code=?", (promo_code,))

 activations = res.fetchone()[0]

[IvanU7n]

CODEFER, или воспользоваться RETURNING activations в UPDATE вместо отдельного SELECT

[CODEFER]

Руслан ., спасибо

[o5a]

CODEFER, Лучше делать именно по логике:
1. опросили текущее состояние (количество оставшихся активаций, как выше указали через 'select activations ...') для данного промокода
2. если оно больше вычитаемого (как в данном случае > 1), тогда уже делаем update, commit и сообщение об успешном окончании, если нет, то даже не пытаемся update, и сразу ругаемся.

Если же решите воспользоваться вариантом update ... returning, то учитывайте, что commit нужно делать не сразу после update, а в блоке else, когда промокод успешно активирован, иначе вы так сохраните в базе уход в минус.

[Руслан .]

o5a, тут можно в update добавить условие в where что activations > 0.
И тогда update не будет загонять в минус поле activations. А последующая проверка срабртает, как и сейчас.

[o5a]

Руслан ., можно, но только если человек знает, что делает. Подобное условие ничего не вернет из запроса, поэтому придется дополнительно проверять на отсутствие строк.