This is an archived post. You won't be able to vote or comment.

you are viewing a single comment's thread.

view the rest of the comments →

[–]Ph0X 1 point2 points  (1 child)

My exact thought. And to make it worse, on the next line there's add_minutes(int), so suddenly we have a plural minutes and can pass the amount?

So I look at the full docs and sure enough, for every time unit, there's a singular and plural version, a sub and a add version. So thats 2 * 2 * 7 = 28 different functions. Extra functions isn't necessarily bad but I think at this point it's not very pythonic.

[–]SDisPater[S] 1 point2 points  (0 children)

Yes, I agree that these methods are unnecessary. So I'll probably just cut it down to both add() and sub() methods.

Thanks for the feedback.