Python: return statement still returns none from function

Question

I've looked at all of the other "Returns none" questions on here and none of them seem to solve my problem.

rates = []
for date in unformatted_returns: # Please ignore undefined variables, it is redundant in this context
    if date[0] >= cutoff_date:
        date_i = unformatted_returns.index(date)
        r = date_initialize(date[0], date_i)
        print "r is returned as:", r
        rates.append(r)
        print date[0]
    else:
        continue

def date_initialize(date, date_i):
        print " initializing date configuration"
        # Does a bunch of junk
        rate_of_return_calc(date_new_i, date_i)

def rate_of_return_calc(date_new_i, date_i):
        r_new = unformatted_returns[int(date_i)] # Reverse naming, I know
        r_old = unformatted_returns[int(date_new_i)] # Reverse naming, I know
        if not r_new or not r_old:
            raise ValueError('r_new or r_old are not defined!!')
            # This should never be true and I don't want anything returned from here anyhow
        else:
            ror = (float(r_new[1])-float(r_old[1]))/float(r_old[1])
            print "ror is calculated as", ror
            return ror

The functions them selves work fine, the output is like so:

initializing date configuration
('2014-2-28', u'93.52')
ror is calculated as -0.142643284859
r is returned as: None
2015-2-2
>>>

ror is the correct value, but why does it not get returned when I have it written right there return ror?? Doesn't make any sense to me

There is no return in your date_initialize function. Thus it implicitly returns None. — krethika
– krethika, Commented Jun 11, 2015 at 19:57
Not to mention the syntax error this code would produce due to line 5. — TigerhawkT3
– TigerhawkT3, Commented Jun 11, 2015 at 19:58
Still @McLean : If you are posting codes it should be be syntax error free. As saying goes : When you are in Rome ,Do it like Romans. — coder3521
– coder3521, Commented Jun 11, 2015 at 20:09
@csharpcoder I must have accidentally deleted the bracket when I was indenting all of the content since it never copies over the way it should. My bad. — Alex McLean
– Alex McLean, Commented Jun 11, 2015 at 20:11

Gecko · Accepted Answer · 2015-06-11 19:56:55Z

3

You need to return it here too

def date_initialize(date, date_i):
        print " initializing date configuration"
        # Does a bunch of junk
        return rate_of_return_calc(date_new_i, date_i)

answered Jun 11, 2015 at 19:56

Gecko

1,40812 silver badges14 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Blair · Accepted Answer · 2015-06-11 19:56:47Z

0

In date_initialize, you need to return the function that is returning the value that you want. Explicitly, change your call from

rate_of_return_calc(date_new_i, date_i)

to

return rate_of_return_calc(date_new_i, date_i)

Your first call, to date_initialize, does not return anything. Therefore, when you call rate_of_return_calc, you receive the value and then throw it away. You need to return it to pass the value along to your main function.

answered Jun 11, 2015 at 19:56

Blair

6,7031 gold badge38 silver badges42 bronze badges

Comments

jukart · Accepted Answer · 2015-06-11 19:58:38Z

0

You need to return the value in date_initialize too:

def date_initialize(date, date_i):
    print " initializing date configuration"
    # Does a bunch of junk
    return rate_of_return_calc(date_new_i, date_i)

answered Jun 11, 2015 at 19:58

jukart

611 silver badge3 bronze badges

Collectives™ on Stack Overflow

Python: return statement still returns none from function

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related