else should be on the same line as the closing brace of the if, for consistency, I think