[Warmachine]

pawn Код:

public OnPlayerUpdate(playerid)
{
    if(GetPlayerAnimationIndex(playerid))
    {
        new animlib[32],animname[32],acstring[128], acxstring[128], acname[MAX_PLAYER_NAME];
        GetPlayerName(playerid, acname, sizeof(acname));
        GetAnimationName(GetPlayerAnimationIndex(playerid), animlib, 32, animname, 32);
        if(strcmp(animlib, "PED", true) == 0)
        {
            if(strcmp(animname, "RUN_PLAYER", true) == 0 && UserStats[playerid][Admin] <= 0)
            {
                format(acxstring, 128, "Player %s[%d] has been banned for use of ******", acname, playerid);
                SendClientMessageToAll(RED, acstring);
                BanEx(playerid, "CJ Run");
            }
        }
    }
    return 1;
}

Does this idea have any cons?

[Ash.]

Hmm... Don't use UsePlayerPedAnims in OnGameModeInit, otherwise you'll ban everyone!

[Warmachine]

It's in

pawn Код:

public OnPlayerUpdate(playerid)

[Ash.]

Quote:

Originally Posted by funky1234 Hmm... Don't use UsePlayerPedAnims in OnGameModeInit, otherwise you'll ban everyone!

Quote:

Originally Posted by Warmachine It's in pawn Код: public OnPlayerUpdate(playerid)

I know, I can see that. My point was that if you were to use the UsePlayerPedAnims() native (thus making everyone run like CJ) your code would ban everyone.

[Sascha]

lol that would be an awesome code then btw (talking about what funky said)

[robintjeh]

Wouldn't it make the server lag when you have loads of players?

[Sascha]

Quote:

Originally Posted by robintjeh Wouldn't it make the server lag when you have loads of players?

good point actually...
if you used a timer it would reduce the lag

[Warmachine]

Quote:

Originally Posted by Sascha good point actually... if you used a timer it would reduce the lag

Thanks for help!

[Hiddos]

Quote:

Originally Posted by robintjeh Wouldn't it make the server lag when you have loads of players?

And here we go again. Having code in OnPlayerUpdate isn't bad, it's just that you need to decide wisely what you can and cannot do in it. The idea of OnPlayerUpdate being bad only comes from some weirdos wondering why their server is lagging while they're saving player data every time OnPlayerUpdate gets called, using dini.

[Mauzen]

You can optimize it by finding out the animation ids and checking for these. What you are doing now is getting the animation index, convert it to strings, and compare these strings two times. Just comparing some numbers will be much faster.