Как сократить код?

Question

[lcd1232]

Есть код ввида

item = {}
if 'firstname' not in item or 'lastname' not in item:
                temp = item['name'].split()
                item['firstname'] = temp[0]
                item['lastname'] = ' '.join(temp[1:])

Нужно как-то сократить условие или сделать, чтобы оно было более читабельнее. Как это можно сделать?

Comments

[AtomKrieg]

Вроде норм код

[sim3x]

Зачем вообще тут проверка?

[lcd1232]

sim3x: Просто в коде не факт, что будут эти поля. Я просто хотел показать этим, как выглядит item.

[sim3x]

lcd1232:
Пересохранение в том же дикте выглядит подозрительно, как-будто у тебя там помойка

Придумай схему в которой твои входящий итем не будет изменяться, а ты будешь из него извлекать все в свою структуру дикт или named tuple и уже с гарантированно нормальными даннми работать внутри

[lcd1232]

sim3x: это scrapy

[sim3x]

lcd1232:

def process_item(self, item, spider):
    result = {}
    if item.get('firstname') and item.get('lastname'):
        result['firstname'], result['lastname'] = item['firstname'], item['lastname']
    # .....
return result

Answer 1

[lega]

item.get('firstname') and item.get('lastname') or some_code()

есть отличие, но для данного случая работать будет

Comments

[lcd1232]

Думал, что есть элегантей способ. Просто увидел тут(https://pythonworld.ru/osnovy/instrukciya-if-elif-... и думал в моем случае что-нибудь такое есть.

[sim3x]

lcd1232: лучше как лега предложил - так проверишь не пустоту values

Answer 2

[abcd0x00]

>>> d = {'aaa': 1, 'bbb': 2}
>>> 
>>> if set(d) & {'aaa', 'ccc'}:
...     print('yes')
... else:
...     print('no')
... 
yes
>>> 
>>> if set(d) & {'ddd', 'ccc'}:
...     print('yes')
... else:
...     print('no')
... 
no
>>>