Path: csiph.com!goblin1!goblin3!goblin.stu.neva.ru!news.netfront.net!not-for-mail
From: =?utf-8?q?Petr_P=C5=99ikryl?= <prikryl@atlas.cz>
Newsgroups: cz.comp.lang.python
Subject: Re: [python] =?utf-8?q?Coding_style?=
Date: Thu, 26 Nov 2015 12:09:16 +0100
Organization: Netfront http://www.netfront.net/
Lines: 48
Message-ID: <mailman.15.1448536158.3505.python@py.cz>
References: <CAE1QKmTSX_A3RcsPpkG=+Ra0aNqkw2Psz5r2ct6gMwLe5VL8wQ@mail.gmail.com>
NNTP-Posting-Host: 194.228.3.217
Mime-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
X-Trace: adenine.netfront.net 1448536169 55448 194.228.3.217 (26 Nov 2015 11:09:29 GMT)
X-Complaints-To: news@netfront.net
NNTP-Posting-Date: Thu, 26 Nov 2015 11:09:29 +0000 (UTC)
To: =?utf-8?q?Konference_PyCZ?= <python@py.cz>
Received-SPF: pass (Address passes the Sender Policy Framework) SPF=MAILFROM; sender=prikryl@atlas.cz; remoteip=::ffff:46.255.225.252; remotehost=; helo=gmmr1.centrum.cz; receiver=hugo.zagamma.cz;
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=atlas.cz; s=mail; t=1448536156; bh=WpEsDZ70DPXshEw8UkR9y8tiWS2VBRmMJa/dnk6YnGQ=; h=To:Subject:Date:From:References:In-Reply-To:From; b=HPLH9O3xeEwWBG64Vc/0ko1mvlM7VvU772vv6/9elGgH+VlGtgZVJZKXaiMYIye/Q lMyeDeWp4DluwhHbFDVccaIBPL7B50l55Y2ZFgh4qil79DPydPmR7ISrmnP936YupA VkmpkwG1n/oqrbhObY7KY4LoCAM/azY0afNSpuBw=
In-Reply-To: <CAE1QKmTSX_A3RcsPpkG=+Ra0aNqkw2Psz5r2ct6gMwLe5VL8wQ@mail.gmail.com>
X-Mailer: Centrum Email 5.3
X-Priority: 3
X-Original-From: prikryl@atlas.cz
X-Maser: Georgo
X-BeenThere: python@py.cz
X-Mailman-Version: 2.1.20
Precedence: list
List-Id: Konference PyCZ <python.py.cz>
List-Unsubscribe: <http://www.py.cz/mailman/options/python>, <mailto:python-request@py.cz?subject=unsubscribe>
List-Archive: <http://www.py.cz/pipermail/python/>
List-Post: <mailto:python@py.cz>
List-Help: <mailto:python-request@py.cz?subject=help>
List-Subscribe: <http://www.py.cz/mailman/listinfo/python>, <mailto:python-request@py.cz?subject=subscribe>
Xref: csiph.com cz.comp.lang.python:3053

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 <petrblahos@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
>

--- news://freenews.netfront.net/ - complaints: news@netfront.net ---