19.06.2013, 14:26
Just a few tips on naming your functions:
- Don't use abbreviations unless absolutely required (SendMessAdmins and GetArm).
- Make sure your function names describe fully what they do: GiveHealth - Give health to what? GivePlayerHealth.
- IsPlayerAim - ing..
- Fuck(playerid, reason) - nice.
You don't actually explain what any of these do. Albeit the majority of them are self-explanatory..
Also, there's no such thing as 'stocks'. People say that because of the 'stock' keyword. They are FUNCTIONS, that use the stock keyword.
IsPlayerInZone(playerid) - what zone? There's only a playerid parameter!
GetPlayerIP - This already exists..?
SendMessAdmins - I assume this is for RCON? Nobody uses RCON really..
But yeah, nice effort.
- Don't use abbreviations unless absolutely required (SendMessAdmins and GetArm).
- Make sure your function names describe fully what they do: GiveHealth - Give health to what? GivePlayerHealth.
- IsPlayerAim - ing..
- Fuck(playerid, reason) - nice.
You don't actually explain what any of these do. Albeit the majority of them are self-explanatory..
Also, there's no such thing as 'stocks'. People say that because of the 'stock' keyword. They are FUNCTIONS, that use the stock keyword.
IsPlayerInZone(playerid) - what zone? There's only a playerid parameter!
GetPlayerIP - This already exists..?
SendMessAdmins - I assume this is for RCON? Nobody uses RCON really..
But yeah, nice effort.

