Почему с отрицательным кол-во хп персонаж жив?

Question

[dada_arewnd]

Есть персонаж и у него изначально 15 HP, также есть монстр который сносит 10 HP.

if cursor.execute(f"SELECT HP FROM users WHERE id = {id}").fetchone()[0] <= 0:
    print("Вы погибли")
if cursor.execute(f"SELECT HP FROM users WHERE id = {id}").fetchone()[0] >= 1:
    cursor.execute(f"UPDATE users SET HP = HP - {attack_1} WHERE id = {id}")
    print(f'Вам снесли 10 HP! \nВаше здоровье {hp}')

Должно быть примерно так:
-Первый удар: Осталось 5 HP
-Второй удар:Вы погибли
Как это происходит:
-Первый удар: Осталось 5 HP
-Второй удар: Осталось -5 HP
-Третий удар: Вы погибли

Comments

[o5a]

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

# текущее здоровье
def get_hp(user_id):
    return cursor.execute("SELECT hp FROM users WHERE id = ?", (user_id, )).fetchone()[0]

# обновить здоровье
def update_hp(user_id, hp):
    cursor.execute("UPDATE users SET hp = hp - ? WHERE id = ?", (hp, user_id, ))
    conn.commit()

# и дальше уже их использовать
hp = get_hp(user_id)
if hp <= 0:
 ...
if hp ...
    update_hp(user_id, attack_1)

Answer 1

[Евгений Гатин]

Все правильно. Сначала идет проверка, что хп больше-равно 1. В случае с 5 это истина, потом удар и минус 10 хп, затем сообщение f'Вам снесли 10 HP! \nВаше здоровье {hp}. Нужно сделать (вкратце) так.

if hp >= 1:
    if hp - attack >= 1:
        cursor.execute(f"UPDATE users SET HP = HP - {attack_1} WHERE id = {id}")
        print(f'Вам снесли 10 HP! \nВаше здоровье {hp}')
    else:
        hp = 0
        print("Вы погибли")
else:
    cursor.execute(f"SELECT HP FROM users WHERE id = {id}").fetchone()[0] <= 0:
    print("Вы погибли")

Answer 2

[Vindicar]

1. Включи логику. Ты сначала проверяешь, умер ли персонаж, а потом наносишь урон. Отсюда и наблюдаемое поведение.
2. Никогда не используй f-строки для SQL запросов, освой parametrized queries.