Jednoznačně první, protože to druhé je výraz, který by měl být používán pro svou hodnotu a ne pro svůj vedlejší efekt.
První varianta je srozumitelná i pro lidi, kteří nikdy ternární operátor v Pythonu neviděli. "else None" v druhé variantě je jen matoucí. Další otázka je, co to je evt a jaký je důvod, že nabývá hodnoty None nebo nějaké jiné, která se v boolovském kontextu vyhodnotí jako false. Možná by okolí mělo být napsáno nějak jinak. (Tj. jak se evt získává.) Pokud by byl často potřebný "prázdný" objekt evt, pak by možná stálo za to vyrobit pro něj speciální třídu, která implementuje prázdnou metodu .Skip() a počáteční test by se vynechával. Petr ______________________________________________________________ > Od: Petr Blahos <petrbla...@gmail.com> > Komu: Konference PyCZ <python@py.cz> > Datum: 26.11.2015 08:26 > Předmět: [python] Coding style > >Ahoj, co považujete za lepší coding style? > >if evt: > evt.Skip() > >nebo > >evt.Skip() if evt else None > >? > >Dík, >Petr > > >---------- > >_______________________________________________ >Python mailing list >python@py.cz >http://www.py.cz/mailman/listinfo/python > >Visit: http://www.py.cz > _______________________________________________ Python mailing list python@py.cz http://www.py.cz/mailman/listinfo/python Visit: http://www.py.cz
