views:

707

answers:

6

I have hardly ever noticed a python program that uses else in a for loop.

I recently used it to perform an action based on the loop variable condition while exiting; as it is in the scope.

What is the pythonic way to use an else in a for loop? Are there any notable use cases?

And, yea. I dislike using break statement. I'd rather set the looping condition complex. Would I be able to get any benefit out of it, if I don't like to use break statement anyway.

Worth noting that for loop has an else since the language inception, the first ever version.

+6  A: 

If you have a for loop you don't really have any condition statement. So break is your choice if you like to abort and then else can serve perfectly to handle the case where you were not happy.

for fruit in basket:
   if fruit.kind in ['Orange', 'Apple']:
       fruit.eat()
       break
else:
   print 'The basket contains no desirable fruit'
Stefan Lundström
You misunderstood the concept. The else block is not executed after a break.
Ferdinand Beyer
You are wrong, as else part is executed only when for completes without a break.
Lakshman Prasad
@Ferdinand: Perhaps his explication is not perfect, but from the code you can clearly see, that it's exactly what he meant.
vartec
@vartec: OK, you are right. To me it read like: Using the else block, you can examine the reason why you did the break. My mistake.
Ferdinand Beyer
Well, if you don't have enought total money in your wallet it will not break, hence it will say you dont have enough money. The else statement is also executed if the wallet is completely empty.
Stefan Lundström
You're absolutely right. I changed the greater than sign to a less than sign, so it works now.
Jason Baker
I disagree with that change, Jason.
Stefan Lundström
@Jason: No, now you just broke the code, please revert.
nikow
You're right. I'm an idiot.
Jason Baker
Perhaps we should just agree that the else-statement isnt that self-explainatory or easy to use.
Stefan Lundström
That is true :)
nikow
It is surprisingly unpythonic...
Ferdinand Beyer
It can't be pythonic until there's a syntax to use for ... else in a list comprehension.
Thomas L Holaday
The PyPy implementers are using it.
Thomas L Holaday
Yep. I considered to change my example to have a statement that changes something with the coin before the break. The wallet example should be solved with sum or reduce.
Stefan Lundström
The pattern for-if-(break|return)-else is present in every example I observed in the PyPy corpus.
Thomas L Holaday
+3  A: 

Without using break, else blocks have no benefit for for and while statements. The following two examples are equivalent:

for x in range(10):
  pass
else:
  print "else"

for x in range(10):
  pass
print "else"

The only reason for using else with for or while is to do something after the loop if it terminated normally, meaning without an explicit break.

After a lot of thinking, I can finally come up with a case where this might be useful:

def commit_changes(directory):
    for file in directory:
        if file_is_modified(file):
            break
    else:
        # No changes
        return False

    # Something has been changed
    send_directory_to_server()
    return True
Ferdinand Beyer
Still, the laster use case, doesnt seem an elegant solution, even for this case.
Lakshman Prasad
I totally agree and frankly I have never used this construct.
Ferdinand Beyer
But now, I predict, you are at risk of seeing opportunities to use it everywhere.
Thomas L Holaday
I'd probably write: "if file is modified(file): send_directory_to_server(); return True" and drop the else: clause..
John Fouhy
@John: Do you have a better example for the for-else construct? :)
Ferdinand Beyer
A: 

Here you go:

a = ('y','a','y')
for x in a:
  print x,
else:
  print '!'

It's for the caboose.

edit:

# What happens if we add the ! to a list?

def side_effect(your_list):
  your_list.extend('!')
  for x in your_list:
    print x,

claimant = ['A',' ','g','u','r','u']
side_effect(claimant)
print claimant[-1]

# oh no, claimant now ends with a '!'

edit:

a = (("this","is"),("a","contrived","example"),("of","the","caboose","idiom"))
for b in a:
  for c in b:
    print c,
    if "is" == c:
      break
  else:
    print
Thomas L Holaday
Not so happy! What prevents you from putting '!' within the list, or using the `print '!'` without the else?
Lakshman Prasad
If you use the 'print !' without the else, then you get the '!' even if the for loop had a break, and it is impolite to exult in the presence of a break.
Thomas L Holaday
Also, since tuples are immutable, adding a '!' to the end of a tuple would require copying the whole thing.
Thomas L Holaday
Ferdinand's example is better than mine because it is more flow-control-y.
Thomas L Holaday
+11  A: 

What could be more pythonic than PyPy?

Look at what I discovered starting at line 284 in ctypes_configure/configure.py:

    for i in range(0, info['size'] - csize + 1, info['align']):
        if layout[i:i+csize] == [None] * csize:
            layout_addfield(layout, i, ctype, '_alignment')
            break
    else:
        raise AssertionError("unenforceable alignment %d" % (
            info['align'],))

And here, from line 425 in pypy/annotation/annrpython.py (clicky)

if cell.is_constant():
    return Constant(cell.const)
else:
    for v in known_variables:
        if self.bindings[v] is cell:
            return v
    else:
        raise CannotSimplify

In pypy/annotation/binaryop.py, starting at line 751:

def is_((pbc1, pbc2)):
    thistype = pairtype(SomePBC, SomePBC)
    s = super(thistype, pair(pbc1, pbc2)).is_()
    if not s.is_constant():
        if not pbc1.can_be_None or not pbc2.can_be_None:
            for desc in pbc1.descriptions:
                if desc in pbc2.descriptions:
                    break
            else:
                s.const = False    # no common desc in the two sets
    return s

A non-one-liner in pypy/annotation/classdef.py, starting at line 176:

def add_source_for_attribute(self, attr, source):
    """Adds information about a constant source for an attribute.
    """
    for cdef in self.getmro():
        if attr in cdef.attrs:
            # the Attribute() exists already for this class (or a parent)
            attrdef = cdef.attrs[attr]
            s_prev_value = attrdef.s_value
            attrdef.add_constant_source(self, source)
            # we should reflow from all the reader's position,
            # but as an optimization we try to see if the attribute
            # has really been generalized
            if attrdef.s_value != s_prev_value:
                attrdef.mutated(cdef) # reflow from all read positions
            return
    else:
        # remember the source in self.attr_sources
        sources = self.attr_sources.setdefault(attr, [])
        sources.append(source)
        # register the source in any Attribute found in subclasses,
        # to restore invariant (III)
        # NB. add_constant_source() may discover new subdefs but the
        #     right thing will happen to them because self.attr_sources
        #     was already updated
        if not source.instance_level:
            for subdef in self.getallsubdefs():
                if attr in subdef.attrs:
                    attrdef = subdef.attrs[attr]
                    s_prev_value = attrdef.s_value
                    attrdef.add_constant_source(self, source)
                    if attrdef.s_value != s_prev_value:
                        attrdef.mutated(subdef) # reflow from all read positions

Later in the same file, starting at line 307, an example with an illuminating comment:

def generalize_attr(self, attr, s_value=None):
    # if the attribute exists in a superclass, generalize there,
    # as imposed by invariant (I)
    for clsdef in self.getmro():
        if attr in clsdef.attrs:
            clsdef._generalize_attr(attr, s_value)
            break
    else:
        self._generalize_attr(attr, s_value)
Thomas L Holaday
The else: clause in classdef.py:176 is unnecessary since they exit the loop via 'return'.. Incidentally, how did you locate these? Just eyeballing the source, or something smarter?
John Fouhy
+1: Nice research!
Ferdinand Beyer
grep for (for|else) at the same level of indentation within ten lines of each other. If I had the skills of a sea cucumber, I'd figure out where in the PyPy compiler the for ... else flow control happens, add logging, and then build -all.
Thomas L Holaday
+2  A: 

Perhaps the best answer comes from the official Python tutorial:

break and continue Statements, and else Clauses on Loops:

Loop statements may have an else clause; it is executed when the loop terminates through exhaustion of the list (with for) or when the condition becomes false (with while), but not when the loop is terminated by a break statement

Oli
+3  A: 

Basically, it simplifies any loop that uses a boolean flag like this:

found = False                # <-- initialize boolean
for divisor in range(2, n):
    if n % divisor == 0:
        found = True         # <-- update boolean
        break  # optional, but continuing would be a waste of time

if found:                    # <-- check boolean
    print n, "is composite"
else:
    print n, "is prime"

and allows you to skip the management of the flag:

for divisor in range(2, n):
    if n % divisor == 0:
        print n, "is composite"
        break
else:
    print n, "is prime"

Note that there is already a natural place for code to execute when you do find a divisor - right before the break. The only new feature here is a place for code to execute when you tried all divisor and did not find any.

This helps only in conjuction with break. You still need booleans if you can't break (e.g. because you looking for the last match, or have to track several conditions in parallel).

Oh, and BTW, this works for while loops just as well.

any/all

Nowdays, if the only purpose of the loop is a yes-or-no answer, you might be able to write it much shorter with the any()/all() functions with a generator or generator expression that yields booleans:

if any(n % divisor == 0 
       for divisor in range(2, n)):
    print n, "is composite"
else:
    print n, "is prime"

Note the elegancy! The code is 1:1 what you want to say!

[This is as effecient as a loop with a break, because the any() function is short-circuiting, only running the generator expression until it yeilds True. In fact it's usually even faster than a loop. Simpler Python code tends to have less overhear.]

This is less workable if you have other side effects - for example if you want to find the divisor. You can still do it (ab)using the fact that non-0 value are true in Python:

divisor = any(d for d in range(2, n) if n % d == 0)
if divisor:
    print n, "is divisible by", divisor
else:
    print n, "is prime"

but as you see this is getting shaky - wouldn't work if 0 was a possible divisor value...

Beni Cherniavsky-Paskin