[Jaua10]

Hey guys i made a simple cmd about no reload my guns, but i cant disabled it when i make /noreload this is what i have:

PHP Code:

CMD:noreload(playerid, params[])
    {
        if(IsPlayerConnected(playerid))
        {
            if (PlayerInfo[playerid][pAdmin] >= 3)
            {
                {
                    {
                        SetTimer("NoReload", 250, 1);
                        SendClientMessage(playerid, COLOR_SUCCESS, "No reload mode, has been enabled !");
                    }
                }
            }
            else
            {
                SendClientMessage(playerid, COLOR_CORRECTION, "You are not authorized to use this command !");
            }
        }
        return 1;
    }
public NoReload(playerid)
{
    GivePlayerWeapon(playerid, GetPlayerWeapon(playerid), 10);
    return 1;
} 


what i made bad? now i want if i type again /noreload says No reload mode, has been disabled !

[TadePoleMG]

Hi

Use SetTimerEx for playerid and also use variable for timer like PlayerNoReloadTime[MAX_PLAYERS] and it works.

[Jaua10]

I TRY TO USE IT BUT PAWN CLOSE WHEN I TRY TO COMPILE any suggestion?

[beckzy]

I'm not sure why you're giving the player 10 ammo every 0.25 seconds, but the correct way to do it is: https://pastebin.com/RbZMa3Ly

[GangstaSunny.]

Quote:

Originally Posted by BeckzyBoi I'm not sure why you're giving the player 10 ammo every 0.25 seconds, but the correct way to do it is: https://pastebin.com/RbZMa3Ly

Which gives the player 10 ammo every 0.25 seconds even if he's not shooting. Also he would lose his weapon if he got like a minigun and a little lag.

This would be the correct way.

PHP Code:

CMD:noreload(playerid, params[])
{
    if(IsPlayerConnected(playerid))
    {
        if(PlayerInfo[playerid][pAdmin] < 3)return SendClientMessage(playerid, COLOR_CORRECTION, "You are not authorized to use this command !");
        if(GetPVarInt(playerid,"NoReloadActive") > 0)//turn off
        {
            DeletePVar(playerid,"NoReloadActive");
            SendClientMessage(playerid, COLOR_SUCCESS, "No-Reload deactivated!");
        }
        else //turn on
        {
            SetPVarInt(playerid,"NoReloadActive",1);
            SendClientMessage(playerid, COLOR_SUCCESS, "No-Reload activated!");
        }
        return 1;
    }
    return 1;
}
public OnPlayerWeaponShot(playerid, weaponid, hittype, hitid, Float:fX, Float:fY, Float:fZ) //called if the player shot
{
    if(GetPVarInt(playerid,"NoReloadActive") > 0){GivePlayerWeapon(playerid, GetPlayerWeapon(playerid), GetPlayerAmmo(playerid)+10);} // give the player +10 ammo
    return 1;
} 


[Jaua10]

oh i got it, thanks alot bro!