Stilisierung von mehrzeiligen Bedingungen in 'if'-Anweisungen?

Question

Manchmal breche ich lange Bedingungen in if s auf mehrere Zeilen. Der naheliegendste Weg, dies zu tun, ist:

  if (cond1 == 'val1' and cond2 == 'val2' and
      cond3 == 'val3' and cond4 == 'val4'):
      do_something

Das ist optisch nicht sehr ansprechend, weil die Aktion mit den Bedingungen verschmilzt. Es ist jedoch die natürliche Art und Weise, die korrekte Python-Einrückung von 4 Leerzeichen zu verwenden.

Im Moment benutze ich:

  if (    cond1 == 'val1' and cond2 == 'val2' and
          cond3 == 'val3' and cond4 == 'val4'):
      do_something

Aber das ist nicht sehr hübsch :-)

Können Sie einen alternativen Weg empfehlen?

Answer 1

Sie brauchen keine 4 Leerzeichen in der zweiten Bedingungszeile zu verwenden. Verwenden Sie vielleicht:

if (cond1 == 'val1' and cond2 == 'val2' and 
       cond3 == 'val3' and cond4 == 'val4'):
    do_something

Vergessen Sie auch nicht, dass der Leerraum flexibler ist, als Sie vielleicht denken:

if (   
       cond1 == 'val1' and cond2 == 'val2' and 
       cond3 == 'val3' and cond4 == 'val4'
   ):
    do_something
if    (cond1 == 'val1' and cond2 == 'val2' and 
       cond3 == 'val3' and cond4 == 'val4'):
    do_something

Beide sind allerdings ziemlich hässlich.

Vielleicht sollten Sie die Klammern (die Stil-Leitfaden entmutigt dies jedoch)?

if cond1 == 'val1' and cond2 == 'val2' and \
   cond3 == 'val3' and cond4 == 'val4':
    do_something

So können Sie zumindest eine gewisse Differenzierung vornehmen.

Oder sogar:

if cond1 == 'val1' and cond2 == 'val2' and \
                       cond3 == 'val3' and \
                       cond4 == 'val4':
    do_something

Ich glaube, ich bevorzuge:

if cond1 == 'val1' and \
   cond2 == 'val2' and \
   cond3 == 'val3' and \
   cond4 == 'val4':
    do_something

Hier ist die Stil-Leitfaden , die (seit 2010) die Verwendung von Klammern empfiehlt.

Answer 2

Für den degenerierten Fall, dass es sich um einfache UND- oder ODER-Verknüpfungen handelt, habe ich auf Folgendes zurückgegriffen.

if all( [cond1 == 'val1', cond2 == 'val2', cond3 == 'val3', cond4 == 'val4'] ):

if any( [cond1 == 'val1', cond2 == 'val2', cond3 == 'val3', cond4 == 'val4'] ):

Sie rasiert ein paar Charaktere und macht deutlich, dass es keine Subtilität in Bezug auf den Zustand gibt.

Answer 3

Ich bevorzuge diesen Stil, wenn ich eine schrecklich große Bedingung habe:

if (
    expr1
    and (expr2 or expr3)
    and hasattr(thingy1, '__eq__')
    or status=="HappyTimes"
):
    do_stuff()
else:
    do_other_stuff()

Answer 4

Jemand muss sich hier für die Verwendung des vertikalen Leerzeichens einsetzen! :)

if (     cond1 == val1
     and cond2 == val2
     and cond3 == val3
   ):
    do_stuff()

Dadurch wird jeder Zustand deutlich sichtbar. Außerdem können so komplexere Bedingungen sauberer ausgedrückt werden:

if (    cond1 == val1
     or 
        (     cond2_1 == val2_1
          and cond2_2 >= val2_2
          and cond2_3 != bad2_3
        )
   ):
    do_more_stuff()

Ja, wir tauschen ein wenig vertikalen Platz gegen Übersichtlichkeit. Das ist es aber wert, IMO.

Answer 5

Das macht nicht so viel besser, aber...

allCondsAreOK = (cond1 == 'val1' and cond2 == 'val2' and
                 cond3 == 'val3' and cond4 == 'val4')

if allCondsAreOK:
   do_something