Почему у меня программа вообще не реагирует на свойства в python?

Question

[ruslashru]

Я создал героя(villain) и злодея(heroe) и хотел, чтобы пользователь сам вводил количество хп и урон героя и злодея, и если хп и урон превышал 10000 или был меньше или равно нулю, то программа этого не давала сделать. Но ей почему-то всё равно,
она даёт ввести число меньше нуля, как-будто просто не видит свойство. Помогите пожалуйста я устал бороться и искать ошибку в своей программе.

class Heroes(object):
    def __init__(self):
        self.__hp = int(input('Введите количество здоровья героя: '))
        self.__damage = int(input('Введите количество урона героя: '))
    @property
    def hp(self):
        return self.__hp
    @hp.setter
    def hp(self, new_hp):
        if new_hp > 10000:
            print('Здоровье не может быть больше 10000')
        elif new_hp <= 0:
            print('Здоровье не может быть меньше или равно нулю')
        else:
            self.__hp = new_hp
            print('s')
    @property
    def damage(self):
        return self.__damage
    @damage.setter
    def damage(self, new_damage):
        if new_damage > 10000:
            print('Урон не может быть больше 10000')
        elif new_damage <= 0:
            print('Урон не может быть меньше или равно нулю')
        else:
            self.__damage = new_damage
            print('t')
class Villains(Heroes):
    def __init__(self):
        self.__hp = int(input('Введите количество здоровья злодея: '))
        self.__damage = int(input('Введите количество урона злодея: '))
heroe = Heroes()
villain = Villains()

Comments

[Everything_is_bad]

Но ей почему-то всё равно, она даёт ввести число меньше нуля, как-будто просто не видит свойство.

а где тут проверка на ввод меньше нуля?

Answer 1

[Aragorn]

Вы напрямую присваиваете значения приватным атрибутам __hp и __damage, и поэтому сеттеры не срабатывают. Уберите подчеркивания

self.hp = int(input('Введите количество здоровья героя: '))
self.damage = int(input('Введите количество урона героя: '))

А вообще вам надо просто логику пересмотреть, как-то странно у вас все сделано

Answer 2

[Daemon23RUS]

class Heroes(object):
    def __init__(self):
        self.__hp = int(input('Введите количество здоровья героя: '))
        self.__damage = int(input('Введите количество урона героя: '))

она даёт ввести число меньше нуля, как-будто просто не видит свойство.

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

Answer 3

[dim5x]

Разве IDE не подчёркивает вам проблемные места?

Вы не вызываете конструктор родительского класса:

class Villains(Heroes):
    def __init__(self):
        super().__init__()

И не очень понятно когда проблема проявляется, при инициализации переменных? Но вы не проверяете никак __hp и __damage. А там, где есть логика проверки, надо понимать что __damage != new_damage и __hp != new_hp.

После инициализации всё корректно, имхо, работает:

Comments

[Aragorn]

имхо, работает

А, имхо, не работает. Что дает эта инициализация? Вы хоть пробовали запускать?

[dim5x]

Aragorn, я-то пробовал (как и всегда). Если вы внимательно прочтёте мой ответ, то увидете:

Но вы не проверяете никак __hp и __damage.

А вот откуда у вас 6 вводов мне не понятно.

Я считаю что всё работает ровно так, как написано. Инициализация не проверяется, после неё код срабатывает корректно.

heroe = Heroes()
villain = Villains()

villain.hp=-120
villain.damage=11000

Что с этим делать и нужно ли - воля автора.