SA-MP Forums Archive
C:\Users\zero\AppData\Roaming\Sublime Text 2\Packages\User\main.pwn(343) : warning 219: local variable "targetid" shadow - 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: C:\Users\zero\AppData\Roaming\Sublime Text 2\Packages\User\main.pwn(343) : warning 219: local variable "targetid" shadow (/showthread.php?tid=522117)


C:\Users\zero\AppData\Roaming\Sublime Text 2\Packages\User\main.pwn(343) : warning 219: local variable "targetid" shadow - Le3aT - 26.06.2014

I get that error

PHP Code:
C:\Users\zero\AppData\Roaming\Sublime Text 2\Packages\User\main.pwn(343) : warning 219: local variable "targetid" shadows a variable at a preceding level 
The code
PHP Code:
CMD:duel(playerid, params[])
{
    if(InDuel[playerid] == 2) return SendClientMessage(playerid, COL_SERVER, "* You have already sent a duel request. Wait for a reply");
    new string[128], targetname, name[MAX_PLAYER_NAME], playername, str[128];
    targetname = GetPlayerName(targetid, name, sizeof(name));
    playername = GetPlayerName(playerid, name, sizeof(name));
    if(sscanf(params, "u", targetid))
    {
        SendClientMessage(playerid, COL_CMD, "USAGE: /duel [PlayerID]");
        return 1;
    }
    format(string, sizeof(string), "You have offered %s (ID:%i) to duel you", targetname, targetid);
    SendClientMessage(playerid, COL_CMD, string);
    format(str, sizeof(str), "You have been offered to duel %s (ID:%i)", playername, playerid);
    SendClientMessage(targetid, COL_CMD, str);
    InDuel[targetid] = 2;
    InDuel[playerid] = 2;
    SetTimerEx("DuelTimer", 10000, 0, "ii", targetid, playerid);
    return 1;
}
CMD:accept(playerid, params[])
{
    if(strcmp(params, " duel", false)== 0)
    {
        if(InDuel[targetid] == 0)
        {
            return SendClientMessage(playerid, COL_SERVER, "No one offered you a duel");
        }
        if(InDuel[playerid] == 2 && InDuel[targetid] == 2)
        {
            SetPlayerPos(playerid, 2, 2, 2);
            SetPlayerPos(targetid, 2, 2, 2);
        }
        return 1;
    }
    return 1;
}
forward DuelTimer(targetid, playerid);
public DuelTimer(targetid, playerid)
{
    InDuel[targetid] = 0;
    InDuel[playerid] = 0;
    SendClientMessage(playerid, COL_SERVER, "* Your duel request has expired. You can offer once again");
    return 1;
} 
Help please

Re: C:\Users\zero\AppData\Roaming\Sublime Text 2\Packages\User\main.pwn(343) : warning 219: local variable "targetid" sh - Jack_Leslie - 26.06.2014

You don't have 'new targetid;' ?

Re: C:\Users\zero\AppData\Roaming\Sublime Text 2\Packages\User\main.pwn(343) : warning 219: local variable "targetid" sh - Le3aT - 26.06.2014

I got it

Re: C:\Users\zero\AppData\Roaming\Sublime Text 2\Packages\User\main.pwn(343) : warning 219: local variable "targetid" sh - Le3aT - 26.06.2014

I mean, I got it added

Respuesta: C:\Users\zero\AppData\Roaming\Sublime Text 2\Packages\User\main.pwn(343) : warning 219: local variable "targe - SickAttack - 26.06.2014

pawn Code:
CMD:duel(playerid, params[])
{
    if(InDuel[playerid] == 2) return SendClientMessage(playerid, COL_SERVER, "* You have already sent a duel request. Wait for a reply");
    new string[128], targetname[MAX_PLAYER_NAME + 1], name[MAX_PLAYER_NAME + 1], targetid;
    if(sscanf(params, "u", targetid))
    {
        SendClientMessage(playerid, COL_CMD, "USAGE: /duel [PlayerID]");
        return 1;
    }
    GetPlayerName(playerid, name, sizeof(name));
    GetPlayerName(targetid, targetname, sizeof(targetname));
    format(string, sizeof(string), "You have offered %s (ID:%i) to duel you", targetname, targetid);
    SendClientMessage(playerid, COL_CMD, string);
    format(string, sizeof(string), "You have been offered to duel %s (ID:%i)", playername, playerid);
    SendClientMessage(targetid, COL_CMD, string);
    InDuel[targetid] = 2;
    InDuel[playerid] = 2;
    SetTimerEx("DuelTimer", 10000, false, "ii", targetid, playerid);
    return 1;
}
CMD:accept(playerid, params[])
{
    if(strcmp(params, "duel", false) == 0)
    {
        if(InDuel[targetid] == 0)
        {
            return SendClientMessage(playerid, COL_SERVER, "No one offered you a duel");
        }
        if(InDuel[playerid] == 2 && InDuel[targetid] == 2)
        {
            SetPlayerPos(playerid, 2, 2, 2);
            SetPlayerPos(targetid, 2, 2, 2);
        }
        return 1;
    }
    return 1;
}
forward DuelTimer(targetid, playerid);
public DuelTimer(targetid, playerid)
{
    InDuel[targetid] = 0;
    InDuel[playerid] = 0;
    SendClientMessage(playerid, COL_SERVER, "* Your duel request has expired. You can offer once again");
    return 1;
}