Python 2.7: это хорошая практика - устанавливать все атрибуты args / kwargs с помощью setattr?

Предыдущее обсуждение: Декоратор Python для автоматического определения переменных __init__, Python: стоит ли динамически создавать переменные?

Плюсы:

• Уменьшает дублирование кода

Минусы:

• Менее читаемый
• Сложнее найти ссылки на элементы (что может также нарушить работу статических анализаторов, таких как pylint)
• Количество шаблонного кода, необходимого для обработки всех сценариев (например, позиционный аргумент, переданный как ключевое слово один; проверьте наличие всех необходимых аргументов), делает сокращение кода несуществующим и дублирует стандартный код вызова подпрограммы
• Требуется много аргументов в конструкторе, которые нужно просто установить как члены - это антипаттерн сам по себе

Вопрос: ... менее повторяющийся и более короткий код

В вашем примере кода требуется 9 строк и 28 ключевых слов.

class Employee(object):
    def __init__(self, name, surname, salary, address, phone, mail):
        self.name = name
        self.surname = surename
        self.salary = salary
        self.address = address
        self.phone = phone
        self.mail = mail

По умолчанию требуется 6 строк и 19 ключевых слов. Итак, вашему примеру требуется больше, а не «более короткий код». Я не вижу никакого "повторяющегося ... кода" в коде по умолчанию, все назначения выполняются один раз.

Сравнил эти две строчки, сделал то же самое. Определите, какие аргументы могут передаваться:

self._allowed_attrs = ['name', 'surname', 'salary', 'address', 'phone', 'mail']  

с участием

def __init__(self, name, surname, salary, address, phone, mail):

Второй требует меньше усилий и делает все за один раз.
Не требуется if key in self._allowed_attrs:, так как python сделает это за вас.

В реальном проекте я бы использовал что-то вроде этого

class Employee(object):
    def __init__(self, person, salary=None):
        self.id = unique_id()
        self.person = person
        self.salary = salary

Все данные, относящиеся к person, должны быть обобщены в object person.

Заключение:
Для данного примера class Employee я бы никогда не использовал (*args, **kwargs).
(*args, **kwargs) аргументы полезны только в том случае, если невозможно предсказать, какие аргументы будут переданы.

Из-за этого невозможно понять, каких аргументов ожидает класс.

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

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

Конечно, лучше, если вы позаботитесь о правильной обработке атрибутов и даже о дезинфекции содержимого, поэтому лучше всего иметь либо декоратор для метода __init__, либо его эквивалент в базовом классе __init__, который будет сделайте все, что нужно: проверьте, подходят ли переданные параметры для конкретного класса, а затем используйте setattr для установки их значений в экземпляре.

Я думаю, что менее волшебный способ - это иметь соглашение в иерархии классов для объявления требуемых параметров как атрибутов класса. Таким образом, вы можете использовать эти атрибуты класса для документирования ожидаемых параметров и их типов, сохранить подпись __init__ как *args, **kwargs, а инициализация вашего базового класса обработать их все.

Базовые модели SQLAlchemy делают это - вы указываете атрибуты класса как специальные «инструментированные атрибуты», и они автоматически назначаются при вызове в __init__.

Более простой способ:

_sentinel = object()

class Base(object):
    def __init__(self, *args, **kwargs):
        for attr_name, class_attr in self.__class__.__dict__.items():
            if isinstance(class_attr, type) and kwargs.get(attr_name, _sentinel) != _sentinel:
                attr_value = kwargs[attr_name]
                if not isinstance(attr_value, class_attr):
                    raise TypeError("Parameter {} is expected to be of type {}".format(attr_name, class_attr))
                setattr(self, attr_name, attr_value)


class Person(Base):
    name = str
    age = int
    phonenumber = Phone
    ...

Это потребовало бы, чтобы все параметры в класс были переданы как именованные параметры, но все они будут автоматически назначаться атрибутам экземпляра, это будет работать, быть документированным и безопасным. Если вы хотите стать еще лучше, просто определите какой-нибудь классный дескриптор в качестве значения attr вашего класса.