Just a question but why does the code have to look fancy or have to look a curtain way? does it take longer if it has unused stuff or longer commands in? I'm not being a richard about this I'm just asking.
It's not about it being fancy - I'm pointing out some mistakes I noticed so he can avoid them in his next add-ons and write better ones.
If it works isn't that all that matters?
Not quite. All add-ons are loaded in the same environment, and people like to install hundreds of them.
So you need to make sure you're not accidently conflicting with something else - like using a global $path variable when you meant to use a local one, or having a generic "reloadMe" function.
I'd guess about 50% of all help topics are caused by incompatible add-ons.