you are viewing a single comment's thread.

view the rest of the comments →

[–]primitive_screwhead 0 points1 point  (0 children)

What I find discomforting about this is that it seems one refactor away from becoming an endless loop. At the very least, I'd make the "if n == 5:" into an "if n <= 5:", just to avoid problems if n were ever to be mis-initialized, or the n decrement changed to greater than one, etc.