Kwargs разбор наилучшей практики

Question

Есть ли более компактный / эффективный способ сделать это?

    for key in kwargs:
        if key == 'log':
            self.log = kwargs[key]
        elif key == 'bin':
            self.bin = kwargs[key]
        elif key == 'pid':
            self.pid = kwargs[key]
        elif key == 'conf':
            self.conf = kwargs[key]

Answer 1

Чтобы достичь именно того, о чем вы просили, вы можете использовать

for key in ('log', 'bin', 'pid', 'conf'):
    if key in kwargs:
        setattr(self, key, kwargs[key])

или

self.__dict__.update((key, kwargs[key])
                     for key in ('log', 'bin', 'pid', 'conf')
                     if key in kwargs)

Однако я бы предпочел что-то вроде этого:

def f(log=None, bin=None, pid=None, conf=None):
    self.log = log
    self.bin = bin
    self.pid = pid
    self.conf = conf

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

Answer 2

self.log = kwargs.get('log', default_log)
self.bin = kwargs.get('bin', default_bin)
self.pid = kwargs.get('pid', default_pid)
self.conf = kwargs.get('conf', default_conf)

Это дает дополнительное преимущество, что self.log назначается в любом случае (AttributeError означает, что ваш код сломан до чертиков, ничего более. Всегда проверяйте, что все всегда назначено.) Без лишних self.log = default_log строк. Вы можете опустить значение по умолчанию, чтобы получить None.

Answer 3

Если ключ, указанный в get(), отсутствует в словаре, результат будет None.

self.log = kwargs.get('log')
self.bin = kwargs.get('bin')
self.pid = kwargs.get('pid')
self.conf = kwargs.get('conf')

Answer 4

for k,v in kwarg.iteritems():
   setattr(self, k, v)

, в котором setattr(self, "bin", "val") похоже на вызов self.bin = "val"

Однако более желательно иметь белый список, как у @Sven Marnach.

Answer 5

for k,v in kw.items():
   setattr(self, k, v)

Answer 6

self.__dict__.update(kwargs)