0

подскажите пожалуйста, как вернуть переменной объекта значение по умолчанию / начальное значение?

В примере ниже - объект пистолет, после выстрела кол-во патронов -1, после применения метода reloading кол-во патронов должно стать начальным - 10. Если прописываем 10 напрямую в методе reloading, то для любого другого объекта, с начальным значением bullet, отличным от 10 "перезарядка" будет некорректной...

    def __init__(self, name, damage, bullet):        
        self.name = name
        self.bullet = bullet
        self.damage = damage
                
    
    def shot(self):
        self.bullet -= 1
        return (f'Произведен выстрел из "{self.name}"!\n'
                f'Нанесен урон: {self.damage}\n'
                f'Осталось патронов: {self.bullet}\n')
    
    "Метод 'Перезарядить оружие'."
    def reloading(self):
        self.bullet = 10
        return (f'"{self.name}" перезаряжен!\n'
                f'Патронов: {self.bullet}\n''')
    

    def __str__(self):
        return (
            f'Текущее состояние оружия: «{self.name}»\n'
            f'Урон: {self.damage}\n'
            f'Патронов: {self.bullet}\n' 
            )

first_gun = Gun('Пистолет Макарова', damage=5, bullet=10)

print(first_gun)
print(first_gun.shot())
print(first_gun.reloading())
print(first_gun)```
2
  • Почему некорректной? Если считать 10 как максимальное количество патронов в любом пистолете (размер обоймы, размер барабана), то reloading() должно давать полное заполнение независимо от того, сколько было патронов ранее. А если вы в ` __init__()` прописываете именно максимальную вместимость, то тогда у вас должно быть 2 переменных - максимальная вместимость (размер обоймы) и текущее количество патронов. И в reloading() делать текущее количество патронов = размер обоймы
    – DmitryK
    Commented 9 февр. 2023 в 13:28
  • если задаем объект класса second_gun = Gun('Пистолет Second', damage=5, bullet=5), перезарядка даст 10 патронов, а надо 5... Commented 9 февр. 2023 в 13:30

1 ответ 1

3

Для этого нужно это начальное значение запомнить.
Если вы в __init__() прописываете именно максимальную вместимость, то тогда у вас должно быть 2 переменных - максимальная вместимость (размер обоймы) и текущее количество патронов. И в reloading() делать текущее количество патронов = размер обоймы

def __init__(self, name, damage, clip):        
    self.name = name
    self.clip = clip  # размер обоймы
    self.bullet = self.clip  # текущее количество патронов
    self.damage = damage

def shot(self):
    self.bullet -= 1  # бесконечное количество патронов

"Метод 'Перезарядить оружие'."
def reloading(self):
    self.bullet = self.clip

И ещё момент - формально у вас сейчас бесконечная обойма, т.к. вы при выстреле не проверяете закончились ли патроны. У вас будет количество патронов -1 а вы будете продолжать стрелять.

4
  • Ещё лучше прямо в инит вместо self.bullet = self.clip сделать вызов self.reloading() ) Чисто из принпипа DRY
    – CrazyElf
    Commented 9 февр. 2023 в 13:41
  • 1
    Согласен - тогда не будет дублирования кода. Или в __init__() прописать self.bullet = 0 - тогда все пистолеты при создании будут незаряженными до явного вызова метода reloading()
    – DmitryK
    Commented 9 февр. 2023 в 13:44
  • Спасибо! Попробовал оба варианта, оба работают как надо) но второй, там где вызов self.reloading(), пока сложнее для понимания... то есть в атрибутах класса, вызываем метод класса? Commented 9 февр. 2023 в 17:25
  • @Сергей Не совсем так. Просто в одном методе класса (конструкторе) вызываем другой метод класса. Такое довольно часто практикуется - один метод вызывать из другого.
    – CrazyElf
    Commented 9 февр. 2023 в 17:37

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.