Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > comp.lang.python > #21694

Re: Style question (Poll)

Show all headers | 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.

Back to comp.lang.python | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

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