43

Consider this code:

def foo(foo_input):
    if 0 <= foo_input <= 100:
        return f_input

This returns None in the case where foo_input > 100. But could it actually not return anything? Or does a function always have to return something?


See also: return, return None, and no return at all?

9
  • 1
    Why does it bother you the function returns None? Apr 8, 2012 at 22:20
  • chaining functions, and if I was going to have to write try : except clauses for each of them (because of the none that is passed)
    – beoliver
    Apr 8, 2012 at 22:22
  • use969617: What would happen if no output was returned? If it was the execution being skipped, then you can get the same functionality using an if statement to check for None - although if it's more likely you will get a result than None, catching exceptions is a better idea, as Python follows the ask for forgiveness, not permission principle. Apr 8, 2012 at 22:23
  • I was wondering if I could get away without even having to use if statements :)
    – beoliver
    Apr 8, 2012 at 22:25
  • 1
    Well, remember that exceptions fall through, so if you want to catch an error due to None anywhere in your process, then you can simply wrap your entire block in a catch for TypeErrors. Obviously only wrap the segment for which this is necessary, but there is no need for a ton of the same try ... except ... blocks Apr 8, 2012 at 22:30

6 Answers 6

47

Functions always return something (at least None, when no return-statement was reached during execution and the end of the function is reached).

Another case is when they are interrupted by exceptions. In this case exception handling will "dominate over the stack" and you will return to the appropriate except or get some nasty error :)

Regarding your problem I must say there are two possibilities: Either you have something to return or you do not have.

  • If you have something to return then do so, if not then don't.
  • If you rely on something being returned that has a certain type but you cannot return anything meaningful of this type then None will tell the caller that this was the case ( There is no better way to tell the caller that "nothing" is returned then by None, so check for it and you will be fine)
31

No. If a return statement is not reached before the end of the function then an implicit None is returned.

0
12

If a return statement is not reached, the function returns None.

def set_x():
    x = 2
5

I'm not sure what you really are trying to do. Here are a few things you might like:

def foo(foo_input, foo_default):
    if 0 <= foo_input <= 100:
        return f_input
    else:
        return foo_default


def foo(foo_input):
    if 0 <= foo_input <= 100:
        return f_input
    raise ValueError, "foo_input was not in range [0, 100]"

Wait, you said "filter". Are you filtering a series of values and you just want to extract the ones that meet a criteria? That's easy in Python:

def foo_check(x):
    return 0 <= x <= 100

filtered_list = [x for x in unfiltered_sequence if foo_check(x)]

And you said "chaining functions". Again that's easy if we are talking about filtering a sequence:

def foo_filter(seq):
    for x in seq:
        if 0 <= x <= 100:
            yield x

def other_filter(seq):
    for x in seq:
        if meets_criterion(x):
            yield x


def do_the_task(seq):
    for x in other_filter(foo_filter(seq)):
        do_something(x)

EDIT: Here is a nice introduction to iterators and generators in Python. http://www.learningpython.com/2009/02/23/iterators-iterables-and-generators-oh-my/

20
  • and this would be fine if I was filtering 100 or so a second?
    – beoliver
    Apr 8, 2012 at 22:35
  • There is no reason to use a function there, it's clearer just to do it as [x for x in sequence if 0 <= x <= 100]. Apr 8, 2012 at 22:36
  • @steveha: why use a list comprehension when there is a built-in filter function? filtered_list = filter(foo_filter, unfiltered_sequence) Apr 8, 2012 at 22:36
  • @Lattyware: I would say otherwise. Also it depends on how used the reader is to the one or the other construction. And as a lazy programmer I would say: it is shorter :) Apr 8, 2012 at 22:38
  • 1
    @Nobody, fire up Python 2.x, and from the >>> prompt type help(filter) and read what it says. It says, quite plainly, that it will return a list, a tuple, or a string. Now fire up Python 3.x and do the same thing; it says, quite plainly, that it will return an iterator. So maybe you don't see why it should do it, but I am talking about what it actually does. I think the reason is historical: I think filter() was introduced before iterators were added. Of course in Python 2.x you can use itertools.ifilter() or write your own function that loops and calls yield.
    – steveha
    Apr 8, 2012 at 22:52
1

I sort of like the implicit return None but pylint flags it as bad style, warning:

Either all return statements in a function should return an expression, or none of them should.pylint(inconsistent-return-statements)

Hence,

def foo(foo_input):
    if 0 <= foo_input <= 100:
        return f_input
    return None

might be better style, even if they are functionally the same.


More info available here, where the Pylint change-log states:

A new Python checker was added to warn about inconsistent-return-statements. A function or a method has inconsistent return statements if it returns both explicit and implicit values ...

According to PEP8, if any return statement returns an expression, any return statements where no value is returned should explicitly state this as return None, and an explicit return statement should be present at the end of the function (if reachable).

1

Functions always return something.

All 3 functions below return None:

def test1():
    print("test1")
    return None

def test2():
    print("test2")
    return

def test3():
    print("test3")

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.