Whitespace: you example doesn’t go far enough. how about some extra line breaks, and spaces around braces and ‘while’, etc. I refuse to believe anyone writes code like you “Bad” example.

Never Delete: no, no, no. Never leave commented code in your files, it just clutters up otherwise readable functions. It’s fine to comment stuff while testing, it obviously saves time when switching between different logic, but once everything’s working the code should be neat!

Use braces: well this is a controversial one. Firstly your modified ‘if’ statement is as fugly, if not moreso. If there is only one statement for the ‘if’ or ‘while’ there’s no reason to use braces, just put the statement n the next line. It’s blindingly obvious that if you add another statement to the ‘if’ you’ll need to use braces.