0

Возникает ошибка при использовании абстрактного метода, моя задача состояла в том, чтобы сделать абстрактный метод, который дальше будет переопределен, но возникает ошибка: 'Flat' object has no attribute 'every_month'

Пожалуйста, прошу исправьте мой код, чтобы при вызове абстрактного метода в дальнейшем выводило число по задумке

from abc import ABC, abstractmethod


class City:
    def __init__(self):
        self.name_city = ' '

    def get_name(self):
        return self.name_city

    def set_name(self, name_city):
        self.name = name_city


class Building:
    def __init__(self):
        self.name_street = ' '
        self.house_number = 0
        self.square = 0
        self.payment = 0

    def get_name_street(self):
        return self.name_city

    def get_house_number(self):
        return self.house_number

    def get_square(self):
        return self.square

    def get_payment(self):
        return self.payment

    def set_name_street(self, name_city):
        self.name_city = name_city

    def set_house_number(self, house_number):
        self.house_number = house_number

    def set_payment(self,payment):
        self.payment = payment

class Room(ABC,Building):
    def __init__(self):
        super().__init__()
        self.number=0
        self.square=0
    def get_number(self):
        return self.number
    def get_square(self):
        return self.square
    def set_number(self,number):
        self.number=number
    def set_square(self,square):
        self.set_square=square
    @abstractmethod
    def every_month_payment(self):
        self.every_month=self.payment*self.square
class Flat(Room):
    def add_people(self,name_people):
        self.name_people=name_people
        self.all_people=[]
        self.all_people.append(self.name_people)
    def get_people(self):
        return self.all_people
    def every_month_payment(self):
        return self.every_month * (1 + len(self.all_people) * 0.1)
class Office(Room):
    def set_name_firm(self,name):
        self.name_firm=name
    def get_name_firm(self):
        return self.name_firm
    def set_type(self,type):
        self.type=type
    def get_type(self):
        return self.type
    def every_month_payment(self):
        return super().every_month * 2
names1=Flat()
names1.set_payment(5)
names1.set_square(6)
names1.add_people('IVANOV')
names1.every_month_payment()```
3
  • не совсем понятно зачем Вам тут абстрактный метод, тем более который имеет реализацию, согласно ооп - это уже не абстрактный метод (Абстра́ктный ме́тод — в объектно-ориентированном программировании, метод класса, реализация для которого отсутствует) переопределять можно и не абстрактные методы 5 окт 2022 в 21:20
  • вот Вы обращаетесь к self.every_month но его нет ни в одном из классов, то что Вы его пытаетесь задать в вызове абстрактного метода, весьма странно, ведь в наследнике вы этот метод перекрываете, тогда там следует обращаться через super() 5 окт 2022 в 22:01
  • при добавлении нового постояльца вы зачем-то затираете старых self.all_people=[] не понятно зачем в вопросе класс City он нигде не используется 5 окт 2022 в 22:06

1 ответ 1

0

Что касается ответа, то его можно было оформить немого по разному, я предпочел такой вариант как наиболее краткий. Если Вы добавляете геттеры и сеттеры для свойств класса то вторым логичным шагом было бы делать и сами свойства или properties, но даже без этого если они есть то думаю их стоит как-то использовать (обычно они предназначены для того, чтобы оградить непосредственное использование переменных класса, добавляя например необходимые проверки в сеттерах и преобразования в геттерах, хотя последнее не обязательно):

class A:
  def __init__(self):
    self._test=0
  def set_test(self, test):
    self._test = test  
  def get_test(self):
    return self._test
  test = property(get_test, set_test)  # добавление свойства

a = A()
print(a.test) # нет необходимости писать a.get_test()
a.test = 5    # нет необходимости писать a.set_test(5)
print(a.test)

В ответе на вопрос я убрал все что не касалось самой сути вопроса чтобы легче было заметить внесенные изменения.

Так же подправил расчет платы, исходя из того, что полагаю чистая плата взимается за одного и +10 % за дополнительного жильца (Вы можете использовать свою формулу, но она считает +10% и для первого жильца, т.е. один жилец будет платить 110% чистой платы):

class Building:
    def __init__(self):
        self._square = 0
        self._payment = 0

    def get_square(self):
        return self._square

    def set_square(self, square):
        # тут может быть проверка
        self._square = square

    def get_payment(self):
        return self._payment

    def set_payment(self, payment):
        # тут может быть проверка
        self._payment = payment

    square = property(get_square, set_square)
    payment = property(get_payment, set_payment) 
        
class Room(Building):
    def __init__(self):
        super().__init__()

    def every_month_payment(self):
        # тут используется доступ уже через геттеры
        return self.payment * self.square

class Flat(Room):
    def __init__(self):
        super().__init__()
        self.all_people=[]

    def add_people(self,name_people):
        self.all_people.append(name_people)

    def get_people(self):
        return self.all_people

    def every_month_payment(self):
        neighbors = len(self.get_people()) - 1 
        return super().every_month_payment() * (
          1 + ((neighbors) * 0.1 if neighbors > 0 else 0)
        ) 

names1=Flat()
names1.set_payment(5)
names1.set_square(6)
print(names1.every_month_payment()) # 30
names1.add_people('IVANOV')
print(names1.every_month_payment()) # 30
names1.add_people('PETROV')
print(names1.every_month_payment()) # 33.0
print(names1.get_people())

В классе Flat методе every_month_payment мы сначала обращаемся к методу every_month_payment в базовом классе а позже используем результат в наших расчетах, но решить эту задачу можно было бы и другими способами если бы у нас был другой не перекрытый классом наследником метод, так или иначе абстрактный метод тут не нужен, тем более как я говорил он не должен иметь реализации.

Абстра́ктный ме́тод — в объектно-ориентированном программировании, метод класса, реализация для которого отсутствует

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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