18.12.2012, 09:24
I like to user OnPlayerUpdate.
You should explain abit more or redirect(maybe) to wiki links, cause I don't think newbies will know what is a pubic function and a stock, and what's the difference. They won't know what does SetTimerEx do.
A newbie won't know what is a timer.
You have to ident your code, so newbies know the proper way of writing codes, and cause if someone copies this and pastes it in his script, he will get loose identation errors.
Should tell the players what "playerid" means and is it a string or an integer.
Tell them why do you have to forward a public function, not just put this here and put that there.
A newbie won't know that this is an Array.
You Could've said "You can use it like this too: new Float:Xpos, Float:Ypos, Float:Zpos;".
Should say why did you add "Float" before its name.
You should explain abit more or redirect(maybe) to wiki links, cause I don't think newbies will know what is a pubic function and a stock, and what's the difference. They won't know what does SetTimerEx do.
Quote:
This creates a timer which checks for the player's vehicle speed every 100ms. |
You have to ident your code, so newbies know the proper way of writing codes, and cause if someone copies this and pastes it in his script, he will get loose identation errors.
Should tell the players what "playerid" means and is it a string or an integer.
Quote:
Forward the function | Put this code under the OnPlayerConnect function: |
Quote:
new Float: xPos[3]; |
You Could've said "You can use it like this too: new Float:Xpos, Float:Ypos, Float:Zpos;".
Should say why did you add "Float" before its name.