Possibly unintended assignement

Possibly unintended assignement - 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: Possibly unintended assignement (/showthread.php?tid=345634)

Possibly unintended assignement - Why - 25.05.2012

Hello.

I am currently fixing on my trucking system I made for my server, however I get a warning on the load command.

Код:

vx-rp.pwn(12344) : warning 211: possibly unintended assignment

My loadtruck command is like this:

pawn Код:

CMD:loadtruck(playerid, params[]) {
    if(jobVariables[playerVariables[playerid][pJob]][jJobType] = 5) { // Line 12344
    if(IsATruck(GetPlayerVehicleID(playerid))){
    if(playerVariables[playerid][pCheckpoint] == 0) {
    SetPlayerCheckpoint(playerid, 2197.9854,-2662.6870,13.5469, 5.0);
    SendClientMessage(playerid, COLOR_COOLBLUE, "A red checkpoint has been marked. Enter it to load the truck.");
    playerVariables[playerid][pCheckpoint] = 9;
    } else return SendClientMessage(playerid, COLOR_GREY, "You already have an existing checkpoint or trucking route.");
    } else return SendClientMessage(playerid, COLOR_GREY, "You are not in an Ocean Dock truck.");
    } else return SendClientMessage(playerid, COLOR_GREY, "You are not a trucker."); 
    return 1;
}

Re: Possibly unintended assignement - Moh_ - 25.05.2012

Quote:

Originally Posted by Why

Hello.

I am currently fixing on my trucking system I made for my server, however I get a warning on the load command.

Код:

vx-rp.pwn(12344) : warning 211: possibly unintended assignment

My loadtruck command is like this:

pawn Код:

CMD:loadtruck(playerid, params[]) {
    if(jobVariables[playerVariables[playerid][pJob]][jJobType] = 5) { // Line 12344
    if(IsATruck(GetPlayerVehicleID(playerid))){
    if(playerVariables[playerid][pCheckpoint] == 0) {
    SetPlayerCheckpoint(playerid, 2197.9854,-2662.6870,13.5469, 5.0);
    SendClientMessage(playerid, COLOR_COOLBLUE, "A red checkpoint has been marked. Enter it to load the truck.");
    playerVariables[playerid][pCheckpoint] = 9;
    } else return SendClientMessage(playerid, COLOR_GREY, "You already have an existing checkpoint or trucking route.");
    } else return SendClientMessage(playerid, COLOR_GREY, "You are not in an Ocean Dock truck.");
    } else return SendClientMessage(playerid, COLOR_GREY, "You are not a trucker."); 
    return 1;
}

== instead of =

Re: Possibly unintended assignement - Catalyst- - 25.05.2012

Replace line 12344 with this...

pawn Код:

if(jobVariables[playerVariables[playerid][pJob]][jJobType] == 5)

Re: Possibly unintended assignement - Why - 25.05.2012

Ah, I didn't see that one. Thanks. Rep'd.