06.12.2013, 20:36
The first thing I saw was this:
Which is really, REALLY bad practice. Instead of ignoring the problem, work on your indentation.
Also try to work on the logic on some parts of your code. Such as in your OnPlayerRequestClass, you use
Better would be to use else:
I don't really get your 'anticheat'. Apperantly anybody wo manages to kill somebody without getting hit is a hacker?
A final tip. As you are releasing this code, you hope others will use your script and maybe even continue working on it. For this to really take off, documentation is a must. You must explain what your functions do, and how they work. The most commenly used way is to add comments to your code.
pawn Код:
#pragma tabsize 0
Also try to work on the logic on some parts of your code. Such as in your OnPlayerRequestClass, you use
pawn Код:
if { ..}
else if { .. }
else if { .. }
if {..}
pawn Код:
if { ..} // If this is true.
else if { .. } // If the above isn't true, but this is.
else if { .. } // If the above isn't true, but this is.
else {..} // If none of the above were true.
A final tip. As you are releasing this code, you hope others will use your script and maybe even continue working on it. For this to really take off, documentation is a must. You must explain what your functions do, and how they work. The most commenly used way is to add comments to your code.