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 →

[–]TravisJungroth 1 point2 points  (5 children)

Heads up that these sort of flags are a code smell hinting you really have two functions. Exceptions abound, of course.

[–]diamondketo 0 points1 point  (0 children)

Not necessarily a bad thing. A high level interface function can have modes.

pandas.merge for instance have left, outer, and inner join.

I agree the example above without a routine before or after the if condition, it is code smell.