3. Finally, I think it's bad practice to put parentheses "()" after your end keywords. You don't need them.
Parenthesis make it clear that you're referencing a function/method rather than a global/field. Things looking like what they are = good practise, surely?