1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help

Threaded Mode
error 021: symbol already defined: "vehicleid"
JayceonTaylor
Big Clucker
***
Posts: 64
Threads: 18
Joined: Jul 2013
Reputation: 0
#1

18.07.2013, 15:40
Hello how can i fix that?
pawn Код:
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25128) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25145) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25162) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25179) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25196) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25213) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25230) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25247) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25264) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25281) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25298) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25315) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25332) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25349) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25366) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25383) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25400) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25417) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25434) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25451) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25468) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25485) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25502) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25519) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25536) : error 021: symbol already defined: "vehicleid"
C:\Users\2com\Desktop\Roleplay\gamemodes\gangwars4.pwn(25553) : error 021: symbol already defined: "vehicleid"
pawn Код:
new vehicleid = GetPlayerVehicleID(playerid);
    if(newstate == PLAYER_STATE_DRIVER && oldstate == PLAYER_STATE_ONFOOT)
    {
    if(vehicleid ==dcar1)
    {
    if(strcmp(PlayerName(playerid), "Avto_Kushta", true) != 0)
    {
    new Float:shealth;
    new Float:slx, Float:sly, Float:slz;
    GetPlayerHealth(playerid, shealth);
    SetPlayerHealth(playerid, shealth-5);
    GetPlayerPos(playerid, slx, sly, slz);
    SetPlayerPos(playerid, slx, sly, slz+5);
    SendClientMessage(playerid, COLOR_GREY, "Това превозно средство принадлежи на Автокъщата,За да го купите се свържете с админ!");
        }
       }
    }
    new vehicleid = GetPlayerVehicleID(playerid);
    if(newstate == PLAYER_STATE_DRIVER && oldstate == PLAYER_STATE_ONFOOT)
    {
    if(vehicleid ==dcar2)
    {
    if(strcmp(PlayerName(playerid), "Avto_Kushta", true) != 0)
    {
    new Float:shealth;
    new Float:slx, Float:sly, Float:slz;
    GetPlayerHealth(playerid, shealth);
    SetPlayerHealth(playerid, shealth-5);
    GetPlayerPos(playerid, slx, sly, slz);
    SetPlayerPos(playerid, slx, sly, slz+5);
    SendClientMessage(playerid, COLOR_GREY, "Това превозно средство принадлежи на Автокъщата,За да го купите се свържете с админ!");
        }
       }
    }
    new vehicleid = GetPlayerVehicleID(playerid);
    if(newstate == PLAYER_STATE_DRIVER && oldstate == PLAYER_STATE_ONFOOT)
    {
    if(vehicleid ==dcar3)
    {
    if(strcmp(PlayerName(playerid), "Avto_Kushta", true) != 0)
    {
    new Float:shealth;
    new Float:slx, Float:sly, Float:slz;
    GetPlayerHealth(playerid, shealth);
    SetPlayerHealth(playerid, shealth-5);
    GetPlayerPos(playerid, slx, sly, slz);
    SetPlayerPos(playerid, slx, sly, slz+5);
    SendClientMessage(playerid, COLOR_GREY, "Това превозно средство принадлежи на Автокъщата,За да го купите се свържете с админ!");
        }
       }
    }
    new vehicleid = GetPlayerVehicleID(playerid);
    if(newstate == PLAYER_STATE_DRIVER && oldstate == PLAYER_STATE_ONFOOT)
    {
    if(vehicleid ==dcar4)
    {
    if(strcmp(PlayerName(playerid), "Avto_Kushta", true) != 0)
    {
    new Float:shealth;
    new Float:slx, Float:sly, Float:slz;
    GetPlayerHealth(playerid, shealth);
    SetPlayerHealth(playerid, shealth-5);
    GetPlayerPos(playerid, slx, sly, slz);
    SetPlayerPos(playerid, slx, sly, slz+5);
    SendClientMessage(playerid, COLOR_GREY, "Това превозно средство принадлежи на Автокъщата,За да го купите се свържете с админ!");
        }
       }
    }
Find
Reply
iggy1
High-roller
*****
Posts: 2,421
Threads: 52
Joined: Mar 2009
Reputation: 0
#2

18.07.2013, 15:49
( Последний раз редактировалось iggy1; 18.07.2013 в 15:54. Причина: elaborated )
You need to learn about variable scope.

https://sampwiki.blast.hk/wiki/Scripting_Basics#Scope

You can't have 2 (or more) variables in the same scope, that have the same name.
pawn Код:
new vehicleid = GetPlayerVehicleID(playerid);//first definition
    if(newstate == PLAYER_STATE_DRIVER && oldstate == PLAYER_STATE_ONFOOT)
    {
    if(vehicleid ==dcar1)
    {
    if(strcmp(PlayerName(playerid), "Avto_Kushta", true) != 0)
    {
   //...
        }
       }
    }
    new vehicleid = GetPlayerVehicleID(playerid);//<<Symbol already defined above
You should also learn to indent your code properly.
Find
Reply
JayceonTaylor
Big Clucker
***
Posts: 64
Threads: 18
Joined: Jul 2013
Reputation: 0
#3

18.07.2013, 15:59
How can i ? show me example
Find
Reply
iggy1
High-roller
*****
Posts: 2,421
Threads: 52
Joined: Mar 2009
Reputation: 0
#4

18.07.2013, 16:16
pawn Код:
if(newstate == PLAYER_STATE_DRIVER && oldstate == PLAYER_STATE_ONFOOT)
{
    new vehicleid = GetPlayerVehicleID(playerid);
    
    if(vehicleid ==dcar1)
    {
        if(strcmp(PlayerName(playerid), "Avto_Kushta", true) != 0)
        {
            new Float:shealth;
            new Float:slx, Float:sly, Float:slz;
            GetPlayerHealth(playerid, shealth);
            SetPlayerHealth(playerid, shealth-5);
            GetPlayerPos(playerid, slx, sly, slz);
            SetPlayerPos(playerid, slx, sly, slz+5);
            SendClientMessage(playerid, COLOR_GREY, "Това превозно средство принадлежи на Автокъщата,За да го купите се свържете с админ!");
        }
    }
    else if(vehicleid ==dcar2)
    {
        if(strcmp(PlayerName(playerid), "Avto_Kushta", true) != 0)
        {
            new Float:shealth;
            new Float:slx, Float:sly, Float:slz;
            GetPlayerHealth(playerid, shealth);
            SetPlayerHealth(playerid, shealth-5);
            GetPlayerPos(playerid, slx, sly, slz);
            SetPlayerPos(playerid, slx, sly, slz+5);
            SendClientMessage(playerid, COLOR_GREY, "Това превозно средство принадлежи на Автокъщата,За да го купите се свържете с админ!");
        }
    }
    else if(vehicleid ==dcar3)
    {
        if(strcmp(PlayerName(playerid), "Avto_Kushta", true) != 0)
        {
            new Float:shealth;
            new Float:slx, Float:sly, Float:slz;
            GetPlayerHealth(playerid, shealth);
            SetPlayerHealth(playerid, shealth-5);
            GetPlayerPos(playerid, slx, sly, slz);
            SetPlayerPos(playerid, slx, sly, slz+5);
            SendClientMessage(playerid, COLOR_GREY, "Това превозно средство принадлежи на Автокъщата,За да го купите се свържете с админ!");
        }
    }
    else if(vehicleid ==dcar4)
    {
        if(strcmp(PlayerName(playerid), "Avto_Kushta", true) != 0)
        {
            new Float:shealth;
            new Float:slx, Float:sly, Float:slz;
            GetPlayerHealth(playerid, shealth);
            SetPlayerHealth(playerid, shealth-5);
            GetPlayerPos(playerid, slx, sly, slz);
            SetPlayerPos(playerid, slx, sly, slz+5);
            SendClientMessage(playerid, COLOR_GREY, "Това превозно средство принадлежи на Автокъщата,За да го купите се свържете с админ!");
        }
    }
}
Done my good deed for the day.
Find
Reply
JayceonTaylor
Big Clucker
***
Posts: 64
Threads: 18
Joined: Jul 2013
Reputation: 0
#5

18.07.2013, 16:38
Works, thanks bro
Find
Reply
Elie1996
Big Clucker
***
Posts: 66
Threads: 5
Joined: Jul 2013
Reputation: 0
#6

18.07.2013, 22:11
Alright, I know what he did works, because all he's done is actualy remove the line "new vehicleid"
but you've got to be careful: it might affect vehicleid that was defined earlier, if you want to get the new vehicle ID and not ruin what vehicleid was before, then just define a new variable like this:
pawn Код:
new vehicleid2;
Find
Reply
« Next Oldest | Next Newest »


Forum Jump:


Users browsing this thread: 1 Guest(s)
  1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help