No.
Spells aren't written in vJass dude.
I, for one, quite enjoy writing spells in vJass. Makes life a whole lot easier.
You must be blind or you just don't pay enought attention to what you are reading.
Im neither blind nor do i pay inadequate attention to what im reading.
Oh, and you were obviously able to use globals, so dont tell me JassHelper doesnt work for you. Learning about libraries/scopes is fairly easy. You only have to do minimal adjustments to your code.
Abusing hashtables is never a good idea. I hope you realize we only have 256 instances to throw around.
The contest is over, respect the participants, see you next time when you get some reading-skill.
This is not an attack on you, the judging is over and i have no intention of changing the outcome. Im merely complaining about the poor quality of code review.
Triggering Unit is much faster
So what? Lots of stuff is faster than other alternatives. If you want to optimize, dont look at stuff like this. Look for what code gets executed most often and optimize that.
The guy didnt null units in his functions -.-
Then say that. Players are also handle vars, as are groups and timers (both of which you dont have to null if youre recycling them).
Which you still havent mentioned. One doesnt play a sound and the other doesnt play a sound. Is there a difference in the silence they cause?
GetLastCreatedUnit is a function.
Of course. Now take a look at what that function actually does. Good day.
EDIT:
I'm jassing since 1,5 month, so the script was done when it wasn't even a month.
All of this doesnt change the fact that your code suffered from the things i mentioned. However much you learned in that month.
The contest is over, respect the participants and judges, end of story.
I dont think automatic respect is appropriate.