[Aprezt]

My problem is bikes dosent work when i write /engine. But Cars work what wrong ?

pawn Code:

if(!strcmp(cmd, "/engine", true))
    {
        if(IsPlayerConnected(playerid))
        {
            if(IsPlayerInAnyVehicle(playerid))
            {
                if(!engineOn[GetPlayerVehicleID(playerid)])
                {
                    if(GetPlayerState(playerid) == PLAYER_STATE_PASSENGER)
                    {
                        return 1;
                    }
                    if(IsAnOwnableCar(idcar))
                    {
                        if(PlayerInfo[playerid][pPcarkey] == idcar) { }
                        else if(PlayerInfo[playerid][pPcarkey2] == idcar) { }
                        else if(PlayerInfo[playerid][pPcarkey3] == idcar) { }
                        else { return 1; }
                    }
                    if(pveh == 510 || pveh == 462 || pveh == 481 || pveh == 509)
                    {
                        return 1;
                    }
                    if(idcar == 59 || idcar == 60)
                    {
                        return 1;
                    }
                    if(idcar >= 135 && idcar <= 154)
                    {
                        if(HireCar[playerid] != idcar)
                        {
                            return 1;
                        }
                    }
                    if(IsAPlane(idcar))
                    {
                        return 1;
                    }
                    if(IsAHarvest(idcar))
                    {
                        return 1;
                    }
                    if(IsADrugHarvest(idcar))
                    {
                        return 1;
                    }
                    if(IsASweeper(idcar))
                    {
                        return 1;
                    }
                    if(gEngine[playerid] == 1) { return 1; }

                    new playerveh = GetPlayerVehicleID(playerid);
                    PutPlayerInVehicle(playerid, playerveh, 0);
                    GetPlayerName(playerid, sendername, sizeof(sendername));
                    format(string, sizeof(string), "* %s spins a key and tries to start vehicle engine.", sendername);
                    ProxDetector(30.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
                    SetTimerEx("StartingTheVehicle",3500,0,"i",playerid);
                    GameTextForPlayer(playerid, "~w~Starting vehicle engine...",3500,3);
                    gEngine[playerid] = 1;
                    return 1;
                }
                if(engineOn[GetPlayerVehicleID(playerid)])
                {
                    if(GetPlayerState(playerid) == PLAYER_STATE_PASSENGER)
                    {
                        return 1;
                    }
                    if(IsAnOwnableCar(idcar))
                    {
                        if(PlayerInfo[playerid][pPcarkey] == idcar) { }
                        else if(PlayerInfo[playerid][pPcarkey2] == idcar) { }
                        else if(PlayerInfo[playerid][pPcarkey3] == idcar) { }
                        else { return 1; }
                    }
                    if(pveh == 510 || pveh == 462 || pveh == 481 || pveh == 509)
                    {
                        return 1;
                    }
                    if(idcar == 59 || idcar == 60)
                    {
                        return 1;
                    }
                    if(idcar >= 135 && idcar <= 154)
                    {
                        if(HireCar[playerid] != idcar)
                        {
                            return 1;
                        }
                    }
                    if(IsAHarvest(idcar))
                    {
                        return 1;
                    }
                    if(IsADrugHarvest(idcar))
                    {
                        return 1;
                    }
                    if(IsAPlane(idcar))
                    {
                        return 1;
                    }
                    if(IsASweeper(idcar))
                    {
                        return 1;
                    }
                    new engine,lights,alarm,doors,bonnet,boot,objective;
                    GetVehicleParamsEx(GetPlayerVehicleID(playerid), engine, lights, alarm, doors, bonnet, boot, objective);
                    
                    engineOn[GetPlayerVehicleID(playerid)] = false;
                    TogglePlayerControllable(playerid, 1);
                    GetPlayerName(playerid, sendername, sizeof(sendername));
                    format(string, sizeof(string), "* %s spins the vehicle key and turns off the engine.", sendername);
                    ProxDetector(30.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
                    gEngine[playerid] = 0;
                    SetVehicleParamsEx(GetPlayerVehicleID(playerid), 0, 0,alarm,doors,bonnet,boot,objective);
                    return 1;
                }
            }
        }
        return 1;
    }

[Admigo]

You have the id from bikes in your script?

[Aprezt]

pawn Code:

public IsABike(carid)
{
if((carid >= 102 && carid <= 107) || (carid >= 112 && carid <= 130) || (carid >= 262 && carid <= 267))
    {
        return 1;
    }
    return 0;
}

[Tee]

You mean cars don't work but bikes do? And that code above doesn't make sense. Use this:

pawn Code:

stock IsBike(vehicleid)
{
    new Bikes[] = {509,481,510,462,448,581,522,461,521,523,463,586,468,471};
    for(new i = 0; i < sizeof(Bikes); i++)
    {
        if(GetVehicleModel(vehicleid) == Bikes[i]) return 1;
    }
    return 0;
}

[Aprezt]

Bikes dosent work but cars work ....

[=WoR=Varth]

I don't see you call IsABike in your engine command.

[Aprezt]

I know mistake and i do this right. Thanx who wanted help.