Re: Style question (Poll)

Path	csiph.com!newsfeed.hal-mli.net!feeder3.hal-mli.net!newsfeed.hal-mli.net!feeder1.hal-mli.net!npeer01.iad.highwinds-media.com!news.highwinds-media.com!feed-me.highwinds-media.com!postnews.google.com!glegroupsg2000goo.googlegroups.com!not-for-mail
From	Jon Clements <joncle@googlemail.com>
Newsgroups	comp.lang.python
Subject	Re: Style question (Poll)
Date	Thu, 15 Mar 2012 10:13:41 -0700 (PDT)
Organization	http://groups.google.com
Lines	73
Message-ID	<24625945.5378.1331831621256.JavaMail.geo-discussion-forums@vblb5> (permalink)
References	<CANm61jcTZNVJAbkj2ZGJ+X1-mUZOoXsR90VstC4rb8UOo1x8Ug@mail.gmail.com> <CAJ6cK1b4Kop1uvEKLXUV3qBZkAajf+g-+i3pTksUVHxazy5EFg@mail.gmail.com> <mailman.653.1331759783.3037.python-list@python.org>
NNTP-Posting-Host	86.156.91.130
Mime-Version	1.0
Content-Type	text/plain; charset=ISO-8859-1
X-Trace	posting.google.com 1331831621 26227 127.0.0.1 (15 Mar 2012 17:13:41 GMT)
X-Complaints-To	groups-abuse@google.com
NNTP-Posting-Date	Thu, 15 Mar 2012 17:13:41 +0000 (UTC)
Cc	python-list@python.org
In-Reply-To	<mailman.653.1331759783.3037.python-list@python.org>
Complaints-To	groups-abuse@google.com
Injection-Info	glegroupsg2000goo.googlegroups.com; posting-host=86.156.91.130; posting-account=HLD_OAoAAAD-0RilNRZUjdKEwXt97Q9q
User-Agent	G2/1.0
X-Received-Bytes	2931
Xref	csiph.com comp.lang.python:21694

Show key headers only | View raw

On Wednesday, 14 March 2012 21:16:05 UTC, Terry Reedy  wrote:
> On 3/14/2012 4:49 PM, Arnaud Delobelle wrote:
> > On 14 March 2012 20:37, Croepha<croepha@gmail.com>  wrote:
> >> Which is preferred:
> >>
> >> for value in list:
> >>   if not value is another_value:
> >>     value.do_something()
> >>     break
> 
> Do you really mean 'is' or '=='?
> 
> If you mean x is not y, write it that way.
> 'not x is y' can be misread and misunderstood, depending on whether
> the 'is' is true or not.
> 
>  >>> not 1 is 1
> False
>  >>> not (1 is 1)
> False
>  >>> (not 1) is 1
> False
> 
> Does not matter how read.
> 
>  >>> not (1 is 0)
> True
>  >>> (not 1) is 0
> False
>  >>> not 1 is 0
> True
> 
> Does matter how read.
> 
> >> if list and not list[0] is another_value:
> >>   list[0].do_something()
> 
> Or
> try:
>    value = mylist[0]
>    if value is not another_value: value.dosomething
> except IndexError:
>    pass
> 
> I would not do this in this case of index 0, but if the index were a 
> complicated expression or expensive function call, making 'if list' an 
> inadequate test, I might.
> 
> > Hard to say, since they don't do the same thing :)
> >
> > I suspect you meant:
> >
> > for value in list:
> >     if not value is another_value:
> >         value.do_something()
> >     break
> >
> > I always feel uncomfortable with this because it's misleading: a loop
> > that never loops.
> 
> I agree. Please do not do this in public ;-).
> 
> -- 
> Terry Jan Reedy

I'm not sure it's efficient or even if I like it, but it avoids try/except and the use of a for loop.

if next( iter(mylist), object() ) is not another_value:
    # ...

Just my 2p,

Jon.

Thread

Re: Style question (Poll) Terry Reedy <tjreedy@udel.edu> - 2012-03-14 17:16 -0400
  Re: Style question (Poll) Jon Clements <joncle@googlemail.com> - 2012-03-15 10:13 -0700
  Re: Style question (Poll) Jon Clements <joncle@googlemail.com> - 2012-03-15 10:13 -0700

csiph-web