Styl kodu - w przypadku, gdy

Question

Który z tych trzech elementów jest lepszym stylem kodowania lub bardziej czytelny? foo powinien być prowadzony na elementach z obu słowników, ale mydict2 może być None

Wariant 1:

for a,b in mydict1.items(): 
    foo(a,b) 
if mydict2: 
    for a,b in mydict2.items(): 
     foo(a,b) 

Opcja 2:

for a,b in mydict1.items(): 
    foo(a,b) 
for a,b in mydict2.items() if mydict2 else dict().items(): 
    foo(a,b) 

Wariant 3:

for a,b in chain(mydict1.items(), mydict2.items() if mydict2 else dict().items()): 
    foo(a,b) 

Answer 1

Podoba mi się trzecia opcja, ponieważ pojedyncza pętla ma zamiar program jest jaśniejszy. Gdybym to ja zrobiłbym generator pomocniczy, w interesie oddzielania obaw.

def mydictitems(*dicts): 
    for d in dicts: 
     if d: 
       yield from d.items() 

for a,b in mydictitems(mydict1, mydict2): 
    foo(a,b) 

Answer 2

Detect sprawę z boku na początku, i zastąpić go z pustym dict - to null object pattern:

if mydict2 is None: 
    mydict2 = {} 

ten jest identyczny do wspólnego wzorca stosowanego w celu uniknięcia zmienny domyślne argumenty. Wtedy zawsze można mieć jeden (bardzo prosty) pętlę:

for a, b in chain(mydict.items(), mydict2.items()): 

Jeśli kontrolować odpowiedni kod, rozważyć zmianę rzeczy, tak że nie może być mydict2None w pierwszej kolejności.