Как можно провести сравнение в цикле?

Question

[ekzotika]

Есть такой код. Нужно провести сравнение, чтобы все строки были равны соответствующим. Можно сделать это как-то красивее, в цикле или что-то типа того? Заранее спасибо за помощь.

qs_cart = ECart.objects.filter(user_id=obj.user_id)
                qs_delivery = EOrderDelivery.objects.filter(order_id__in=qs_cart.values('order_id'))
                for idx, obj in enumerate(qs_delivery):
                    if idx == 0:
                        fields = ['city', 'country', 'flat', 'house', 'region', 'street', 'postal_code']
                        obj_ok = obj
                        id_ok = qs_cart[0].order_id
                    else:
                        if (getattr(obj, fields[0]) == getattr(obj_ok, fields[0])) and (
                                getattr(obj, fields[1]) == getattr(obj_ok, fields[1])) and (
                                getattr(obj, fields[2]) == getattr(obj_ok, fields[2])) and (
                                getattr(obj, fields[3]) == getattr(obj_ok, fields[3])) and (
                                getattr(obj, fields[4]) == getattr(obj_ok, fields[4])) and (
                                getattr(obj, fields[5]) == getattr(obj_ok, fields[5])) and (
                                getattr(obj, fields[6]) == getattr(obj_ok, fields[6])):
                            qs_cart.update(order_id=id_ok)

Answer 1

[Владимир Куц]

Эту всю простыню:

(getattr(obj, fields[0]) == getattr(obj_ok, fields[0])) and (
getattr(obj, fields[1]) == getattr(obj_ok, fields[1])) and (
getattr(obj, fields[2]) == getattr(obj_ok, fields[2])) and (
getattr(obj, fields[3]) == getattr(obj_ok, fields[3])) and (
getattr(obj, fields[4]) == getattr(obj_ok, fields[4])) and (
getattr(obj, fields[5]) == getattr(obj_ok, fields[5])) and (
getattr(obj, fields[6]) == getattr(obj_ok, fields[6])):

Можно на это заменить:

all(getattr(obj, x) == getattr(obj_ok, x) for x in fields)

Comments

[ekzotika]

Большое Вам спасибо!

Answer 2

[Roman K]

Передавай кортеж полей на сравнение https://docs.python.org/3/library/operator.html#op...

Comments

[ekzotika]

А можете, пожалуйста, подсказать, как мне передать кортеж?

[Roman K]

ekzotika, как и всегда

getter = attrgetter('city', 'country', 'flat', 'house', 'region', 'street', 'postal_code')

...

getter(obj) == getter(obj_ok)

[o5a]

ekzotika,

if attrgetter(*fields)(obj) == attrgetter(*fields)(obj_ok)

[Roman K]

o5a, Зачем создавать геттер каждый раз и зачем в целом хранить fields в виде списка для одного лишь действия — чтобы распаковать его и собрать в кортеж?)

[o5a]

Roman Kitaev, да, не оптимально, я просто пример использования показал