Как правильно парсить словарь?

Question

[Андрей Гриневич]

Доброго времени суток, возникает ошибка, в случае, если в декоратор передать несуществующий ключ

def get_params(*args):
    def my_decorator(method):
        def wrapped(**kwargs):
            items={"params":{"power":{"basevalue":5}, "damage":{"basevalue":5}}}
            test = {item: items["params"].get(item)["basevalue"] for item in args}
            kwargs.update(test)
            return method(**kwargs)
        return wrapped
    return my_decorator



@get_params("power","damagea")
def params_from(power=None, damage=None,**kwargs):
    return power, damage


print params_from()

можно ли как-то правильнее расписать декоратор, чтобы программа не падала в таком случае?

Answer 1

[Максим Васильев]

А зачем у вас константный items внутри wrapped?
Если вынести его в get_params и проверить наличие args, то можно выкинуть исключение в момент декорирования, а не в момент вызова.

Comments

[Андрей Гриневич]

Я сильно упростил пример, items это поле у модели, так или иначе из него нужно извлекать только нужные параметры и на уровне декоратора.

[Максим Васильев]

И набор поей модели заранее не известен?

[Андрей Гриневич]

Нет, почему же :) почему я вообще решил поступить так.
У объекта имеется куча методов, в малую часть из них нужно передавать определённые аргументы например как "power" и "damage". Но значения этих аргументов находятся в глубине словарей.

[Максим Васильев]

Чтото я вобще не пойму - а чего вы получить то хотите?
Чтобы программа игнорирвала несуществующий ключ или что?

[Андрей Гриневич]

Идеально - да, игнорирование несуществующего ключа без нелепых проверок.

[Максим Васильев]

test = {item: items["params"].get(item)["basevalue"] for item in args if item in items["params"]}

[Андрей Гриневич]

Во!, оно) Спасибо)

Answer 2

[Mintormo]

Вот так пойдет?

def get_params(*args):
    def my_decorator(method):
        def wrapped(**kwargs):
            items = {"params":{"power":{"basevalue":5}, "damage":{"basevalue":5}}}
            test  = {item: items["params"].get(item)["basevalue"]
                       for item in args if item in items["params"]}
            kwargs.update(test)
            return method(**kwargs)
        return wrapped
    return my_decorator

@get_params("power","damagea")
def params_from(power=None, damage=None,**kwargs):
    return power, damage

print params_from()

Comments

[Андрей Гриневич]

Вполне неплохо, спасибо!

Answer 3

[s1dney]

Скорее всего я что-то не так понял

try:
	test = dictionary['not_exists_keys']
except KeyError:
	return False

Comments

[Андрей Гриневич]

Да, такой вариант правильный, но я копаю в сторону получения второго ключа без try/except, нужно нечто сродни гету, но для NoneType гет не срабатывает =/