[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

more pythonic way

My non-expert vote is for

if month is None:
    month = datetime.date.today().month

Because you're checking for your default value, not whether the boolean version of what they did give you is True or False. It's explicit, it's not reliant on any __bool__() function implementations or overrides, etc.

-----Original Message-----
From: Python-list [mailto:python-list-bounces+david.raymond=tomtom.com at python.org] On Behalf Of Felix Lazaro Carbonell
Sent: Monday, February 11, 2019 2:30 PM
To: python-list at python.org
Subject: more pythonic way


Hello to everyone:

Could you please tell me wich way of writing this method is more pythonic:



    def find_monthly_expenses(month=None, year=None):

        month = month or datetime.date.today()



Or it should better be:


        if not month:

            month = datetime.date.today()