Spamming Message - Printable Version

Spamming Message - Printable Version

+- SA-MP Forums Archive (https://sampforum.blast.hk)
+-- Forum: SA-MP Scripting and Plugins (https://sampforum.blast.hk/forumdisplay.php?fid=8)
+--- Forum: Scripting Help (https://sampforum.blast.hk/forumdisplay.php?fid=12)
+--- Thread: Spamming Message (/showthread.php?tid=359204)

Spamming Message - WinterAce - 13.07.2012

pawn Код:

forward Check();
public Check()
{
    for(new i = 0; i < MAX_PLAYERS; i++)
    {
        if(IsPlayerConnected(i))
        {
            if(PlayerInfo[i][pPackageLoading] >= 1)
            {
                PlayerInfo[i][pPackageLoading] --;
            }
            if(PlayerInfo[i][pPackageLoading] == 0)
            {
                SendClientMessage(i, COLOR_LIGHTBLUE, "You are now able to load more products.");
                return 1;
            }
            if(PlayerInfo[i][pPackageLoading] < 0)
            {
                PlayerInfo[i][pPackageLoading] = 0;
            }
        }
    }
    return 1;
}

The message "You are now able to load more products." spams all the way down the page, how do I make it so it only says it once?

Re: Spamming Message - clarencecuzz - 13.07.2012

Change

pawn Код:

if(PlayerInfo[i][pPackageLoading] == 0)

pawn Код:

if(PlayerInfo[i][pPackageLoading] == 1)

The timer stops at 0, so then once PackageLoading reaches 0, it stays like that. So it's going to send the message everytime it detects PackageLoading at 0.

Re: Spamming Message - WinterAce - 13.07.2012

Yeah, is there any way it just sends the message once when the timer hits 0?

Re: Spamming Message - Vince - 13.07.2012

This ought to work;

pawn Код:

forward Check();
public Check()
{
    for(new i = 0; i < MAX_PLAYERS; i++)
    {
        if(!IsPlayerConnected(i)) continue;

        /*
            Very important to decrement before comparing.
            If the package loading var is already 0,
            the function will set it to -1 and skip the following block.
            It will then execute the next block to set the variable back to 0.

            If the package loading var is not 0, the first statement will
            decrement the var, but it will not send the message until the
            var reaches 0.
        */
        
        if(--PlayerInfo[i][pPackageLoading] == 0)
        {
            SendClientMessage(i, COLOR_LIGHTBLUE, "You are now able to load more products.");
            continue;
        }
        if(PlayerInfo[i][pPackageLoading] < 0)
        {
            PlayerInfo[i][pPackageLoading] = 0;
        }
    }
    return 1;
}

Re: Spamming Message - WinterAce - 13.07.2012

Quote:

Originally Posted by Vince

This ought to work;

pawn Код:

forward Check();
public Check()
{
    for(new i = 0; i < MAX_PLAYERS; i++)
    {
        if(!IsPlayerConnected(i)) continue;

        /*
            Very important to decrement before comparing.
            If the package loading var is already 0,
            the function will set it to -1 and skip the following block.
            It will then execute the next block to set the variable back to 0.

            If the package loading var is not 0, the first statement will
            decrement the var, but it will not send the message until the
            var reaches 0.
        */
        
        if(--PlayerInfo[i][pPackageLoading] == 0)
        {
            SendClientMessage(i, COLOR_LIGHTBLUE, "You are now able to load more products.");
            continue;
        }
        if(PlayerInfo[i][pPackageLoading] < 0)
        {
            PlayerInfo[i][pPackageLoading] = 0;
        }
    }
    return 1;
}

Still the same Vince, just spams the message down the page. I can't think of a way possible.. unless I use 'stock'?

Re: Spamming Message - Elysian` - 13.07.2012

I had this happened before to me, try this;

pawn Код:

forward Check();
public Check()
{
        if(!IsPlayerConnected(i))

            for(new i = 0; i < MAX_PLAYERS; i++)
            {
                if(--PlayerInfo[i][pPackageLoading] == 0)
                {
                    SendClientMessage(i, COLOR_LIGHTBLUE, "You are now able to load more products.");
                    continue;
                }
                if(PlayerInfo[i][pPackageLoading] < 0)
                {
                    PlayerInfo[i][pPackageLoading] = 0;
                }
            }
        }
    return 1;
}

Re: Spamming Message - Roko_foko - 13.07.2012

pawn Код:

forward Check();
public Check()
{
    for(new i = 0; i < MAX_PLAYERS; i++)
    {
        if(IsPlayerConnected(i))
        {
            if(PlayerInfo[i][pPackageLoading] == 1)
            {
                SendClientMessage(i, COLOR_LIGHTBLUE, "You are now able to load more products.");
            }
            if(PlayerInfo[i][pPackageLoading] >= 1)
            {
                PlayerInfo[i][pPackageLoading] --;
            }
            if(PlayerInfo[i][pPackageLoading] < 0)// AND WHY DO YOU NEED THIS? I WOULD DELETE THIS.
            {
                PlayerInfo[i][pPackageLoading] = 0;
            }
        }
    }
    return 1;
}

this should work

Re: Spamming Message - WinterAce - 13.07.2012

Thanks guys, I added Vince's one wrong, managed to get it to work, thanks Vince.

[pawn]
forward Check();
public Check()
{
for(new i = 0; i < MAX_PLAYERS; i++)
{
if(!IsPlayerConnected(i)) continue;

if(--PlayerInfo[i][pPackageLoading] == 0 && PlayerInfo[i][pJob] == JOB_TRUCKER )
{
SendClientMessage(i, COLOR_LIGHTBLUE, "You are now able to load more products.");
continue;
}
}
return 1;
}[pawn]

I have this now, would it work so only the players with a trucker job would get the message? I know quite a bit about scripting, but not too sure if this would work in PlayerUpdate

Re: Spamming Message - Vince - 13.07.2012

You could replace the IsPlayerConnected with the following, which will just skip over anyone that is not a trucker.

pawn Код:

if(PlayerInfo[i][pJob] != JOB_TRUCKER) continue;