[FunnyBear]

Hey guys,

I added a system, if any civilian tries to steal a Law Enforcement vehicle, their wanted level goes up.

pawn Code:

public OnPlayerEnterVehicle(playerid, vehicleid, ispassenger)
{
    if(vehicleid == Ambulance1 || vehicleid == Ambulance2 || vehicleid == Ambulance3
    || vehicleid == Ambulance4 && gTeam[playerid] == TEAM_CIV || gTeam[playerid] == TEAM_CIV_ROBBER ||  gTeam[playerid] == TEAM_CIV_RAPIST ||  gTeam[playerid] == TEAM_CIV_TERRORIST ||  gTeam[playerid] == TEAM_CIV_DRUGGIE ||  gTeam[playerid] == TEAM_CIV_WEAPONSELLER ||  gTeam[playerid] == TEAM_CIV_SPY ||  gTeam[playerid] == TEAM_CIV_KIDNAPPER )
    {
        SendClientMessage(playerid, INFO, "[WANTED LEVEL] Wanted level 2 - Stealing an Enforcement vehicle!");
        SetPlayerWantedLevel(playerid, 2);
        return 1;
    }
    return 1;
}

But if I enter any vehicle, my wanted level still goes up, no matter what vehicle I enter!!!

Please help!!

[Rock]

Ohh, nevermind.
I understood something else.

[FunnyBear]

I don't get what you mean

[Tigerkiller]

pawn Code:

public OnPlayerEnterVehicle(playerid, vehicleid, ispassenger)
{
    if(vehicleid == Ambulance1 || vehicleid == Ambulance2 || vehicleid == Ambulance3
    || vehicleid == Ambulance4 && gTeam[playerid] != TEAM_COP) // Define your Cop Team if you haven't yet
    {
        SendClientMessage(playerid, INFO, "[WANTED LEVEL] Wanted level 2 - Stealing an Enforcement vehicle!");
        SetPlayerWantedLevel(playerid, GetPlayerWantedLevel(playerid) +2);
    }
    return 1;
}

[RajatPawar]

pawn Code:

SetPlayerWantedLevel(playerid, GetPlayerWantedLevel(playerid) + 1);

Also, try switching cases with vehicle model id's, rather than checking so many things and if's and ||s.
*Damn, I thought Vince was asking this.*

[Misiur]

Group your conditions.

pawn Code:

if((vehicleid == Ambulance1 || vehicleid == Ambulance2 || vehicleid == Ambulance3
    || vehicleid == Ambulance4) && (gTeam[playerid] == TEAM_CIV || gTeam[playerid] == TEAM_CIV_ROBBER ||  gTeam[playerid] == TEAM_CIV_RAPIST ||  gTeam[playerid] == TEAM_CIV_TERRORIST ||  gTeam[playerid] == TEAM_CIV_DRUGGIE ||  gTeam[playerid] == TEAM_CIV_WEAPONSELLER ||  gTeam[playerid] == TEAM_CIV_SPY ||  gTeam[playerid] == TEAM_CIV_KIDNAPPER))

#e: Group AND check simply if is not a cop/medic instead checking every other option

[Tigerkiller]

Just check if he isn't a Cop, which would be easier at all! ( gTeam[player] != TEAM_COP )

[RajatPawar]

Quote:

Originally Posted by Tigerkiller Just check if he isn't a Cop, which would be easier at all! ( gTeam[player] != TEAM_COP )

He wants a level only if the vehicle is a public service vehicle, not any vehicle.

[Tigerkiller]

Quote:

Originally Posted by Tigerkiller pawn Code: public OnPlayerEnterVehicle(playerid, vehicleid, ispassenger) {     if(vehicleid == Ambulance1 || vehicleid == Ambulance2 || vehicleid == Ambulance3     || vehicleid == Ambulance4 && gTeam[playerid] != TEAM_COP) // Define your Cop Team if you haven't yet     {         SendClientMessage(playerid, INFO, "[WANTED LEVEL] Wanted level 2 - Stealing an Enforcement vehicle!");         SetPlayerWantedLevel(playerid, GetPlayerWantedLevel(playerid) +2);     }     return 1; }

....

[FunnyBear]

It works. BUT, theres is a problem, if I go next to the ambulance and press enter, I get the wanted level/message whereas, it should only appear when I'm actually sitting inside the ambulance.

Help?