Python returning none instead of string

Question

This is my code for reference

def tally(tournament_results):

    results = tournament_results.split("\n")

    if results != ['']:
        for i in results:
            entry = i.split(";")
            if entry[0] not in teams:
                create_team(entry[0])

            if entry[1] not in teams:
                create_team(entry[1])

            if entry[2].strip() == "win":
                result(entry[0], entry[1], 'decision')
            elif entry[2].strip() == "loss":
                result(entry[1], entry[0], 'decision')
            elif entry[2].strip() == "draw":
                result(entry[0], entry[1], 'draw')

    output_file()


def output_file():
    global teams
    sorted_x = sorted(teams.items(), key=lambda teams: teams[1][4], reverse=True)
    result = template.format("Team", "MP", "W", "D", "L", "P")
    for entry in sorted_x:
        result = result + "\n" + (template.format(entry[0], entry[1][0], entry[1][1], entry[1][2], entry[1][3], entry[1][4]))

    teams = {}

    print(result)
    return result

if __name__ == '__main__':
    print(tally('Allegoric Alaskans;Blithering Badgers;win\n'
                   'Devastating Donkeys;Courageous Californians;draw\n'
                   'Devastating Donkeys;Allegoric Alaskans;win\n'
                   'Courageous Californians;Blithering Badgers;loss\n'
                   'Blithering Badgers;Devastating Donkeys;loss\n'
                   'Allegoric Alaskans;Courageous Californians;win'))

The print statement accurately prints the expected output which is:

Team                           | MP |  W |  D |  L |  P 
Devastating Donkeys            |  3 |  2 |  1 |  0 |  7 
Allegoric Alaskans             |  3 |  2 |  0 |  1 |  6 
Blithering Badgers             |  3 |  1 |  0 |  2 |  3 
Courageous Californians        |  3 |  0 |  1 |  2 |  1

However, my return value for the same variable is None. I don't quite understand why.

Any help would be appreciated

@Barmar oof I'm dumb. Thanks! I can put your as correct solution if you post as answer — Afsheen Taheri
– Afsheen Taheri, Commented Mar 17, 2019 at 16:38

busybear · Accepted Answer · 2019-03-17 16:50:04Z

3

Add a return statement to tally: change output_file() to return output_file()

edited Mar 17, 2019 at 16:50

busybear

10.7k1 gold badge29 silver badges44 bronze badges

answered Mar 17, 2019 at 16:39

1LeveL1

535 bronze badges

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

Comments

Barmar · Accepted Answer · 2019-03-17 16:37:19Z

2

You're never printing the result of output_file(). You're printing the result of tally(), but it doesn't return anything. If you want that to print the result of output_file(), change:

output_file()

to:

return output_file()

answered Mar 17, 2019 at 16:37

Barmar

789k57 gold badges554 silver badges669 bronze badges

Collectives™ on Stack Overflow

Python returning none instead of string

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related