> My preference is to actually break that logic up and avoid the wrapping in the first place, as in [2]. Which in this particular class has the side benefit of that value being used again in the same function anyways.
> I'm starting to realize that Brandon Rhodes really had a big impact on my ideas of styling as I've been learning Python these past few years, as this was another one style I'm stealing from that same talk [3].
On 2016-04-15 18:03, Victor Stinner wrote:> Hum.>> if (width == 0> and height == 0> and color == 'red'> and emphasis == 'strong'> or highlight > 100):> raise ValueError("sorry, you lose")>> Please remove one space to vertically align "and" operators with the> opening parenthesis:>> if (width == 0> and height == 0> and color == 'red'> and emphasis == 'strong'> or highlight > 100):> raise ValueError("sorry, you lose")>> (I'm not sure that the difference is obvious in a mail client, you> need a fixed width font which is not the case in my Gmail editor.)>> It helps to visually see that the multiline test and the raise> instruction are in two different blocks.>> (Moreover, the pep8 checks of OpenStack simply reject such syntax, but> I cannot use this syntax anymore :-))>I always half-indent continuation lines: if (width == 0 and height == 0 and color == 'red' and emphasis == 'strong' or highlight > 100): raise ValueError("sorry, you lose")_______________________________________________Python-Dev mailing listPython-Dev@python.orghttps://mail.python.org/mailman/listinfo/python-devUnsubscribe: https://mail.python.org/mailman/options/python-dev/ianlee1521%40gmail.com