Множественные ifs в понимании списка
Я использовал понимание списка, которое должно хранить keys и values в переменной itemDict, учитывая тот факт, что некоторые элементы, если они присутствуют в item['keys'], не будут включены в itemDict. Я использовал несколько условий if, чтобы сделать это возможным.
Это то, что я пробовал:
itemDict = {item['keys']:item['value'] for item in soup.select('input[keys]') if '$cmdPrint' not in item['name'] and 'btnView' not in item['name'] and 'btnMyDoc' not in item['key']}
Как я могу переписать эти условия, чтобы сделать их краткими?
Не использовать понимание списка, вероятно, лучшая идея. Но вы также можете извлечь условия if в один вызов функции.
Быть кратким — непостоянная цель. Лучше быть читабельным, занимать несколько дополнительных строк и ценить свое время и время тех, кто должен читать ваш код после вас.
Отличный пример непонятного (и не поддающегося отладке) понимания списка. Отличная работа. Предлагаем вам использовать циклы for - гораздо удобнее в обслуживании
Ответы 1
Вы можете использовать функцию check и встроенную функцию all. Также переформатируйте понимание для удобочитаемости.
def check(item):
return all(('$cmdPrint' not in item['name'],
'btnView' not in item['name'],
'btnMyDoc' not in item['key']))
itemDict = {item['keys']:item['value']
for item in soup.select('input[keys]')
if check(item)}
Или использовать его в понимании, но это одно длинное понимание.
itemDict = {item['keys']:item['value']
for item in soup.select('input[keys]')
if all(('$cmdPrint' not in item['name'],
'btnView' not in item['name'],
'btnMyDoc' not in item['key']))}
Или просто используйте обычный цикл for:
itemDict = {}
for item in soup.select('input[keys]')
if check(item):
itemDict[item['keys']] = item['value']
Я предлагаю использовать петлю
for.