Как правильно получать значения поля объекта класса?

Question

[Boris19]

Как наиболее правильно с точки зрения архитектуры приложения получать значения полей какого-либо объекта?
Просто напрямую или лучше создавать соответствующую функцию, которая будет возвращать значение внутренней переменной? И надо ли в таком случае имя переменной начинать с нижнего подчеркивания?

Answer 1

[Andrey Dugin]

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

1) Если поле статическое - напрямую.
2) Если поле вычисляемое в момент запроса - через @property.
3) Можно через функцию, если зависит от параметров.

Надо ли начинать с подчёркивания - определяется программистом. Должно ли это поле быть"видно" вовне?

Answer 2

[Roman K]

1. Геттеры и сеттеры в питоне для обычных атрибутов не принято использовать.
2. Если атрибут "необычный", то геттеры и сеттеры реализуются через @property, @attr.setter, @attr.deleter, настоящее же поле скрывается из интерфейса нижним подчёркиванием.
Примером необычного атрибута может быть реализация пароля:

class User:
    def __init__(self, username, password):
        self.username = username
        self._password = make_password(password)

    @property
    def password(self):
        return self._password

    @password.setter
    def password(self, value):
        self._password = make_password(value)

Comments

[javedimka]

Допустимо ли писать логику прямо в геттер или надо обязательно делать приколы с underscore аттрибутами?

[Roman K]

javedimka, логику в геттер? что? вопрос неясен.

[javedimka]

Roman Kitaev,
У меня в одном скрипте есть несколько таких штук:

@property
def busy_time(self):
    """timedelta : time laser was in work"""
    values = [self.summary[key] for key in self.summary if key != 'idle']
    return sum(values, timedelta())

И вопрос в том, допускается ли такое использование или «по науке» надо обязательно написать метод, в моем случае длинною в одну строку. Уходить в функциональщину с чем-то типо self.busy_time = self.get_busy_time() не хочется, вызывать кучу методов в init, чтобы навесить аттрибуты - тоже.

[Roman K]

javedimka, ну, вроде, по задумке создателей, проперти созданы именно для того, чтобы для них указывались сеттеры. Однако, я часто сам юзаю проперти там, где что-то вычисляется из связанных атрибутов или атрибутов объекта, для удобства. Например:

@property
def cover(self):
    x = self.turboimage_set.first()
    if not x:
        x = self.product.cover()
    return x and x.file

или

@property
def is_expired(self):
    return timezone.now() > self.expires_at