// End if

Andrew Forward on
if {

} // End if

It is comments like this that are absolutely a complete waste of time. Comments should be written to help other developers run through your code, not to show your boss that this curly brace closes off the the if above, whereas that curly braces completes the while loop.

Maybe those comments are needed because the if/while { ... } statement spans more than a few lines – which is a strong indication that the method should be broken into smaller, more digestible pieces.

Comments are closed.