Что я сделал не правильно или что я не продумал?

Question

[Vadim kyklaed]

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

Пример работы программы: по моему мнению должен был победить герой
12 11
5 5
-4 -5
Тролль победил

import random

health_hero = 20
health_troll = 20


while health_hero >0 or health_troll >0:
    damage_hero = random.randint(1,10)
    damage_troll = random.randint(1,10)
    health_hero-=damage_troll
    health_troll-=damage_hero
    print(health_hero, health_troll)
    if health_hero<0:
        print("Тролль победил")
    elif health_troll<0:
        print("Победил Герой")

Answer 1

[aRegius]

Привет, Вадим.

while health_hero > 0 and health_troll > 0:
    damage_hero = random.randint(1, 5)
    damage_troll = random.randint(1, 5)
    health_hero -= damage_troll
    health_troll -= damage_hero
    print(health_hero, health_troll)

if health_hero > health_troll:
    print("Победил герой!")
else:
    print("Победил тролль!")

Comments

[Артем Кисленко]

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

[aRegius]

Артем Кисленко:
1. Это не реализация. Это ответ на один вопрос, вырванный из контекста.
2. Корректность программы (игры) может определить только автор. Что значит, "мертвый не может ответить"? Я, например, хочу реализовать возможность "воскрешения" некоторых своих героев... Или, почему вы отрицательное здоровье считаете "мертвый"? В моей реализации это, возможно, "тяжело раненый"... И т.д. и т.п.
Для корректной реализации подобных программ, когда автором являетесь не вы, нужно очень много времени провести в беседе с потенциальным заказчиком, чтобы четко, до мелочей, понять, что ему нужно.

[Артем Кисленко]

aRegius: я прошу прощения, я не в коем случая не критиковал, я просто хотел подсказать Vadim kyklaed, вот и все. Извините, еще раз.

[aRegius]

Артем Кисленко: Артем, да бросьте, все нормально :) Ваш комментарий уместен, точка зрения мне понятна :)

Answer 2

[Иван]

Оба имели отрицательное здоровье, но первым вы проверяете здоровье героя => победил тролль.

Comments

[Vadim kyklaed]

так нужно их сравнивать на то у кого меньше и тогда печать ответ ?

[NaName]

сравнение health<0 надо делать после удара. т.к. мертвые наверное не могут в обратку ударить