[Muhamed.pwn]

Hey.

I have a small issue with OnPlayerStateChange.


I check if the player is the Driver, then under this code I add another code for checking if the player is in a Job car to (not)remove them from the vehicle.

The first block works but all the code under it won't work!
Here it is:

Код:

	if(GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
	{
		if(BusCar(GetPlayerVehicleID(playerid))) // This works
		{
	 		if(PlayerInfo[playerid][pJob] == 1)
			{
	  			SendClientMessage(playerid,-1,"{1589FF} Welcome Bus Driver.");
			}
			else SendClientMessage(playerid,COLOR_GREY,"This vehicle is LOCKED!"); RemovePlayerFromVehicle(playerid);
		}
		if(MechanicCar(GetPlayerVehicleID(playerid))) // All the code under this don't work whatever I put under here
		{
	 		if(PlayerInfo[playerid][pJob] == 2)
			{
	  			SendClientMessage(playerid,-1,"{1589FF} Welcome Mechanic.");
			}
			else SendClientMessage(playerid,COLOR_GREY,"This vehicle is LOCKED!"); RemovePlayerFromVehicle(playerid);
		}
	}

Thanks.

[Kebab-]

Try this

Код:

	if(GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
	{
		if(BusCar(GetPlayerVehicleID(playerid))) // This works
		{
	 		if(PlayerInfo[playerid][pJob] == 1)
			{
	  			SendClientMessage(playerid,-1,"{1589FF} Welcome Bus Driver.");
			}
			else return SendClientMessage(playerid,COLOR_GREY,"This vehicle is LOCKED!"); RemovePlayerFromVehicle(playerid);
		}
		if(MechanicCar(GetPlayerVehicleID(playerid))) // All the code under this don't work whatever I put under here
		{
	 		if(PlayerInfo[playerid][pJob] == 2)
			{
	  			SendClientMessage(playerid,-1,"{1589FF} Welcome Mechanic.");
			}
			else return SendClientMessage(playerid,COLOR_GREY,"This vehicle is LOCKED!"); RemovePlayerFromVehicle(playerid);
		}
	}

You've forgot to add the return's to a few sections and tell me if it works!

[Muhamed.pwn]

Still the same.

[NotIntegrated]

Use "else if",

Код:

else if(MechanicCar(GetPlayerVehicleID(playerid)))

[Muhamed.pwn]

Quote:

Originally Posted by NotIntegrated Use "else if", Код: else if(MechanicCar(GetPlayerVehicleID(playerid)))

Doesn't work, either.

[NotIntegrated]

At the end of OnPlayerStateChange are you returning 0 or 1? And can we see the code for MechanicCar?

[Muhamed.pwn]

Quote:

Originally Posted by NotIntegrated At the end of OnPlayerStateChange are you returning 0 or 1? And can we see the code for MechanicCar?

OnPlayerStateChange returns 1, here's the MechanicCar stock (it's the same as BusCar):

Код:

stock MechanicCar(vehicleid)
{
    for(new i;i <= sizeof(TowTrucks);i++)
    {
        if(vehicleid == TowTrucks[i]) return 1;
    }
    return 0;
}

[Threshold]

pawn Код:

if(newstate == PLAYER_STATE_DRIVER)
    {
        if(BusCar(GetPlayerVehicleID(playerid))) // This works
        {
            if(PlayerInfo[playerid][pJob] == 1)
            {
                SendClientMessage(playerid,-1,"{1589FF} Welcome Bus Driver.");
            }
            else
            {
                SendClientMessage(playerid,COLOR_GREY,"This vehicle is LOCKED!"); 
                RemovePlayerFromVehicle(playerid);
            }
        }
        if(MechanicCar(GetPlayerVehicleID(playerid))) // All the code under this don't work whatever I put under here
        {
            if(PlayerInfo[playerid][pJob] == 2)
            {
                SendClientMessage(playerid,-1,"{1589FF} Welcome Mechanic.");
            }
            else
            {
                SendClientMessage(playerid,COLOR_GREY,"This vehicle is LOCKED!"); 
                RemovePlayerFromVehicle(playerid);
            }
        }
    }

[Muhamed.pwn]

Quote:

Originally Posted by BenzoAMG pawn Код: if(newstate == PLAYER_STATE_DRIVER)     {         if(BusCar(GetPlayerVehicleID(playerid))) // This works         {             if(PlayerInfo[playerid][pJob] == 1)             {                 SendClientMessage(playerid,-1,"{1589FF} Welcome Bus Driver.");             }             else             {                 SendClientMessage(playerid,COLOR_GREY,"This vehicle is LOCKED!");                 RemovePlayerFromVehicle(playerid);             }         }         if(MechanicCar(GetPlayerVehicleID(playerid))) // All the code under this don't work whatever I put under here         {             if(PlayerInfo[playerid][pJob] == 2)             {                 SendClientMessage(playerid,-1,"{1589FF} Welcome Mechanic.");             }             else             {                 SendClientMessage(playerid,COLOR_GREY,"This vehicle is LOCKED!");                 RemovePlayerFromVehicle(playerid);             }         }     }

It's still the same, I don't know what's wrong with this but it's weird.

[Threshold]

Did you change GetPlayerState(playerid) to newstate?
Make sure you copy my code exactly.

If that doesn't work, then the problem is most likely in your 'TowTrucks' array.