Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

Looks like solid code. My only gripe is the shadowing of x. I would prefer to see `for _ in range`. You do redefine it immediately so it's not the most confusing, but it could trip people up especially as it's x and not i or something.


Hahaha thanks, I never noticed that. If I ever print it out and frame it I'll be sure to fix it




Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: