[Donvalley]

any way to make it shorter? thanks guys

pawn Код:

if(IsPlayerInVehicle(playerid, 448) || IsPlayerInVehicle(playerid, 462) || IsPlayerInVehicle(playerid, 408) || IsPlayerInVehicle(playerid, 574) || IsPlayerInVehicle(playerid, 609) || IsPlayerInVehicle(playerid, 498) || IsPlayerInVehicle(playerid, 403) || IsPlayerInVehicle(playerid, 435) || IsPlayerInVehicle(playerid, 400) || IsPlayerInVehicle(playerid, 414) || IsPlayerInVehicle(playerid, 456) || IsPlayerInVehicle(playerid, 482) || IsPlayerInVehicle(playerid, 414) || IsPlayerInVehicle(playerid, 455) || IsPlayerInVehicle(playerid, 482))

[iJumbo]

pawn Код:

if(IsPlayerInVehicle(playerid, 448) || IsPlayerInVehicle(playerid, 462) || IsPlayerInVehicle(playerid, 408) ||
IsPlayerInVehicle(playerid, 574) || IsPlayerInVehicle(playerid, 609) || IsPlayerInVehicle(playerid, 498) ||
IsPlayerInVehicle(playerid, 403) || IsPlayerInVehicle(playerid, 435) || IsPlayerInVehicle(playerid, 400) ||
IsPlayerInVehicle(playerid, 414) || IsPlayerInVehicle(playerid, 456) || IsPlayerInVehicle(playerid, 482) ||
IsPlayerInVehicle(playerid, 414) || IsPlayerInVehicle(playerid, 455) || IsPlayerInVehicle(playerid, 482)) {
    Code
}

[Donvalley]

Cheers Mate

[dEcooR]

Код:

#define v IsPlayerInVehicle

	if(v(playerid, 448) || v(playerid, 462) || v(playerid, 408) || 
           v(playerid, 574) || v(playerid, 609) || v(playerid, 498) ||
           v(playerid, 403) || v(playerid, 435) || v(playerid, 400) ||
	   v(playerid, 414) || v(playerid, 456) || v(playerid, 482) ||
           v(playerid, 414) || v(playerid, 455) ||v(playerid, 482))

[Vince]

A switch is the answer to your question. Also I'm pretty sure that you're checking vehicle models, not ids.

pawn Код:

switch(GetVehicleModel(GetPlayerVehicleID(playerid)))
{
    case 448,462,408,574,609,498,403,435,400,414,456,482,414,455,482:
    {
        // do stuff
    }
    default:
    {
        // not one of these
    }
}

[Donvalley]

Thanks for all your help guys but iJumbo answered this for me thanks though