Блог

Мне нужно проверить, существует ли ключ в словаре Python, и значение этого не равно null/пусто для нескольких ключей, прежде чем продолжить обработку.

 exception = False  if 'key1' in d and d['key1']:  pass else:  exception = True  if 'key2' in d and d['key2']:  pass else:  exception = True  if 'key3' in d and d['key3']:  pass else:  exception = True  if not exception:  #Process it  

Я чувствую, что этот код очень уродлив и не является питоническим.

Можем ли мы написать логику лучше?

Вы можете использовать all и генератор:

 if all(k in d and d[k] for k in ['key1', 'key2', 'key3']):  pass else:  exception = True  

Вы действительно можете пропустить использование флага:

 if not all(k in d and d[k] for k in ['key1', 'key2', 'key3']):  # process  

или

 if any(k not in d or not d[k] for k in ['key1', 'key2', 'key3']):  # process  

Вы можете использовать d.get(k) :

 if all( d.get(k) for k in ["key1","key2","key3"]):  exception = False else:  exception = True   

или

 exception = all( d.get(k) for k in ["key1","key2","key3"])  

Вы можете использовать try/except с operator.itemgetter или использовать dict.get :

 from operator import itemgetter try:  exception = not all(itemgetter('key1', 'key2', 'key3')(d)) except KeyError:  exception = True  

Или,

 exception = not all(map(d.get, ('key1', 'key2', 'key3')))  

Это должно дать вам правильный Bool.

 exception = False for k in ['key1', 'key2', 'key3']:  if k in d and d[k]:  exception = True  break  

Ответ @mozway более питонический и лучший, но в любом случае здесь используется другой подход с использованием функции.

 def check_dict(dictionary):  for key, value in dictionary.items():  if key not in ("key1", "key2", "key3"):  return False  return True