[Tutorial] [TUT] How to stop command abusing

[Torran]

Welcome to my very short and simple tutorial!

Credits to Joe_Staff for the code!

For SAMP Default

public OnPlayerCommandText(playerid, cmdtext[])
{
if(strcmp...
{
if(GetPVarInt(playerid,"CMDABUSE")>GetTickCount())return SendClientMessage(playerid,0xFF0000FF,"You must wait before using a command");
SetPVarInt(playerid,"CMDABUSE",GetTickCount()+5000);
// REST OF CODE
return 1;
}

For DCMD

dcmd_....
{
if(GetPVarInt(playerid,"CMDABUSE")>GetTickCount())return SendClientMessage(playerid,0xFF0000FF,"You must wait before using a command");
SetPVarInt(playerid,"CMDABUSE",GetTickCount()+5000);
// REST OF CODE
return 1;
}

For ZCMD

CMD:.....
{
if(GetPVarInt(playerid,"CMDABUSE")>GetTickCount())return SendClientMessage(playerid,0xFF0000FF,"You must wait before using a command");
SetPVarInt(playerid,"CMDABUSE",GetTickCount()+5000);
// REST OF CODE
return 1;
}

[BlackAgent007]

nice tut thanks mate.. ill try it as soon as possible

[biltong]

I thought public functions have to return something.

[Torran]

Originally Posted by biltong

I thought public functions have to return something.

I never return on publics for my timers

[Torran]

Edited first post,
With a much simpler code,
And simpler way of doing it,
Thanks to Joe_Staff,

[[MWR]Blood]

Nice Tut!Very useful

[Rac3r]

Originally Posted by biltong

I thought public functions have to return something.

No.

Bit of help for you:

if( IsPlayerSomething(playerid) )SetPlayerHealth(playerid, 100.0);

This, needs a return. 1 for true, 0 for false.

IsPlayerSomething(playerid)
{
    if(GetPlayerTeam(playerid) == 2) return 1;
    else return 0;  
}

Another thing to be aware of is if you return something from a function, at the end it must also return something.

public MyTimer()
{
    new x;
    x = random(2);
    if(x == 1)return 1;
    //code
    //code
    //code
    //code
    return 1;//you have to return   
}

Most functions though don't need a return for it to finish.

SetTimer("MyTimer",1000,1);

public MyTimer()
{
}

MakePlayerDoSomething(playerid);

MakePlayerDoSomething(playerid)
{
}

[biltong]

I get warnings telling me that public function x should return a value, that's why I thought public functions always return something.

[Rac3r]

There is probably a return higher up in the function. (Like below)

public MyTimer()
{
    new x;
    x = random(2);
    if(x == 1)return 1;// you have a return here, but you don't have one at the end
    //code
    //code
    //code
    //code
}

[johnathon956]

i edited the carmenu by Garsino. so people couldnt abuse it by over spawning cars
lol i even made it so they cant spawn huge planes XD