Path: csiph.com!newsfeed.hal-mli.net!feeder3.hal-mli.net!newsfeed.hal-mli.net!feeder1.hal-mli.net!newsfeed.xs4all.nl!newsfeed4.news.xs4all.nl!xs4all!post.news.xs4all.nl!not-for-mail
MIME-Version: 1.0
In-Reply-To: <0f01ed1d-bc24-4286-8d1c-bfee7baedbc2@googlegroups.com>
References: <mailman.8744.1396276875.18130.python-list@python.org> <533a3fd8$0$2909$c3e8da3$76491128@news.astraweb.com> <0f01ed1d-bc24-4286-8d1c-bfee7baedbc2@googlegroups.com>
From: Ian Kelly <ian.g.kelly@gmail.com>
Date: Tue, 1 Apr 2014 00:28:53 -0600
Subject: Re: Code style query: multiple assignments in if/elif tree
To: Python <python-list@python.org>
Content-Type: text/plain; charset=ISO-8859-1
Precedence: list
Newsgroups: comp.lang.python
Message-ID: <mailman.8772.1396333782.18130.python-list@python.org>
Lines: 51
NNTP-Posting-Host: 2001:888:2000:d::a6
Xref: csiph.com comp.lang.python:69475

On Mon, Mar 31, 2014 at 11:45 PM, Rustom Mody <rustompmody@gmail.com> wrote:
> On Tue, Apr 1, 2014 at 3:26 PM, Steven D'Aprano  wrote:
>
>> Haskell has nifty pattern-matching syntax for this that looks quite close
>> to the mathematical hybrid function syntax, but in Python, we're limited
>> to explicitly using an if. If I were coding this, and I'm not, I'd wrap
>> it in a function. One advantage of a state variable rather than a
>> continuous time function is that we can do this:
>> def accel(state):
>>     return {NO_BRAKING: 0.0,
>>             LOW_BRAKING: 0.2,
>>             MID_BRAKING: 0.425,
>>             HIGH_BRAKING: 0.85}[state]
>
> Neat
> I would put the dict in a variable. And those _BRAKINGs are GALLing me!
>
> breaking = {NO:0.0, LOW:0.2, MID:0.425:, HIGH:0.85}
> def accel(state): return breaking[state]

If I were doing this in Python 3.4 I would be sorely tempted to use an
Enum for the state.  Then the acceleration could just be an attribute
(or method) of the state itself!

class BrakingState(Enum):
    no_braking = 0.0
    low_braking = 0.2
    mid_braking = 0.425
    high_braking = 0.85

    @property
    def acceleration(self):
        return self.value

>>> print(BrakingState.low_braking.acceleration)
0.2


Alternatively the __init__ method could set the acceleration directly
from the value(s) passed in, as in the Planet example:

https://docs.python.org/3/library/enum.html#planet

The problem I have with either of these approaches though is that if
two distinct states happen to have the same acceleration, they will be
considered aliases for the same state.  This is because the
acceleration value is doing double duty as a parameter of the state
and also as its identity.  Likewise in the Planet example, if a new
planet happened to have the same mass and radius as Jupiter, it would
be considered the same planet as Jupiter. I haven't managed to work
out a wholly satisfying way to avoid this.