Path: csiph.com!v102.xanadu-bbs.net!xanadu-bbs.net!news.albasani.net!rt.uk.eu.org!newsfeed.xs4all.nl!newsfeed4a.news.xs4all.nl!xs4all!newsgate.cistron.nl!newsgate.news.xs4all.nl!post.news.xs4all.nl!not-for-mail
MIME-Version: 1.0
In-Reply-To: <8084-1396540962-768613@sneakemail.com>
References: <8084-1396540962-768613@sneakemail.com>
From: Ian Kelly <ian.g.kelly@gmail.com>
Date: Thu, 3 Apr 2014 12:12:21 -0600
Subject: Re: Yet Another Switch-Case Syntax Proposal
To: Python <python-list@python.org>
Content-Type: text/plain; charset=ISO-8859-1
Precedence: list
Newsgroups: comp.lang.python
Message-ID: <mailman.8848.1396548791.18130.python-list@python.org>
Lines: 61
NNTP-Posting-Host: 2001:888:2000:d::a6
Xref: csiph.com comp.lang.python:69599

On Thu, Apr 3, 2014 at 10:02 AM, Lucas Malor <3kywjyds5d@snkmail.com> wrote:
>> __contains__ is not part of the interface for iterables
>
> For what I know there's not an Iterable interface. For example List simply extends Object. I hope that an ABC Iterable class will be introduced in a future.

It already exists:

>>> import collections.abc
>>> isinstance([1,2,3], collections.abc.Iterable)
True

>> Instead of disabling fallthrough by default, why not disable it all together?
>
> I was tempted but there are cases in which it's useful. An example
>
> switch day casein ("Monday", "Thursday", "Wednesday", "Tuesday", "Friday"):
>     gotowork = True
>     continue
> casein ("Monday", "Thursday", "Wednesday", "Tuesday", "Friday"):
>     daytype = "ferial"
> casein ("Saturday", "Sunday")
>     daytype = "festive"

I don't see how it's useful there.  The first two cases are the same,
so just combine the code into one suite.  For overlapping cases where
the intersection needs to be handled by both suites, you can match
against the union and use conditionals to determine in more detail
which code to run.  For example, if you would write:

switch day case in ("Mon", "Wed", "Fri"):
    lunch_time = datetime.time(12)
    meeting_time = datetime.time(14)
    continue
case in ("Tue", "Thu"):
    lunch_time = datetime.time(11, 30)
    meeting_time = datetime.time(12, 30)
    continue
case in ("Mon", "Tue", "Wed", "Thu", "Fri"):
    go_to_work = True
    day_type = "ferial"
case in ("Sat", "Sun"):
    go_to_work = False
    day_type = "festive"

Use this instead:

switch day case in ("Mon", "Tue", "Wed", "Thu", "Fri"):
    go_to_work = True
    day_type = "ferial"
    if day in ("Tue", "Thu"):
        lunch_time = datetime.time(11, 30)
        meeting_time = datetime.time(12, 30)
    else:
        lunch_time = datetime.time(12)
        meeting_time = datetime.time(14)
case in ("Sat", "Sun"):
    go_to_work = False
    day_type = "festive"

You get an extra level of indentation this way, but it reads less like
spaghetti code.