[N0FeaR]

I got some problem when i choose carid 400 = landstalker in the car menu its become a bravura?

pawn Код:

}
public OnPlayerModelSelection(playerid, response, listid, modelid)
{
    if(listid == carlist)
    {
        if(response)
        {
            new Float:pos[3]; GetPlayerPos(playerid, pos[0], pos[1], pos[2]);
            new string[128], price;
            switch(carlist)
            {
                case 0: {modelid = 400; price = 600;}
                case 1: {modelid = 401; price = 4000;}
                case 2: {modelid = 402; price = 6000;}
                case 3: {modelid = 403; price = 4000;}
                case 4: {modelid = 404; price = 6000;}
                case 5: {modelid = 405; price = 20000;}
                case 6: {modelid = 406; price = 25000;}
            }
            if(price > PlayerInfo[playerid][pMoney]) return SendClientMessage(playerid, COLOR_GREY, "You don't have enough money on you.");
            GiveDodMoney(playerid, -price);
            format(string, sizeof(string), " You have purchased a %s. (Unlock it using /lock)", RVN(modelid));
            SendClientMessage(playerid, COLOR_GREEN, string);
            //if(PlayerInfo[playerid][pVIP] >= 2 && PlayerInfo[playerid][vModel])
            if(PlayerInfo[playerid][vModel])
            {
                PlayerInfo[playerid][pVVeh] = CreateVehicle(modelid, 532.3726,-1287.4926,17.2422,272.3125,0,0,1200);
                SetVehicleParamsEx(PlayerInfo[playerid][pVVeh], 0, 0, 0, 1, 0, 0, 0);
                PlayerInfo[playerid][vVLocked] = 1;
                PlayerInfo[playerid][vVModel] = modelid;
                GetVehiclePos(PlayerInfo[playerid][pVVeh], PlayerInfo[playerid][vVX], PlayerInfo[playerid][vVY], PlayerInfo[playerid][vVZ]);
                GetVehicleZAngle(PlayerInfo[playerid][pVVeh], PlayerInfo[playerid][vVA]);
                PlayerInfo[playerid][vVC1] = 0;
                PlayerInfo[playerid][vVC2] = 0;
                return 1;
            }
            PlayerInfo[playerid][pVeh] = CreateVehicle(modelid, 532.3726,-1287.4926,17.2422,272.3125,0,0,1200);
            SetVehicleParamsEx(PlayerInfo[playerid][pVeh], 0, 0, 0, 1, 0, 0, 0);
            PlayerInfo[playerid][vLocked] = 1;
            PlayerInfo[playerid][vModel] = modelid;
            GetVehiclePos(PlayerInfo[playerid][pVeh], PlayerInfo[playerid][vX], PlayerInfo[playerid][vY], PlayerInfo[playerid][vZ]);
            GetVehicleZAngle(PlayerInfo[playerid][pVeh], PlayerInfo[playerid][vA]);
            PlayerInfo[playerid][vC1] = 0;
            PlayerInfo[playerid][vC2] = 0;
            //cVeh[i] = f(id, pos[0], pos[1], pos[2], pos[3], col[0], col[1], 1200);
        }
        else SendClientMessage(playerid, 0xFF0000FF, "Canceled vehicle selection");
        return 1;

[RajatPawar]

You should be switching 'modelid', not the list itself!

pawn Код:

switch(modelid)

[N0FeaR]

Quote:

Originally Posted by Rajat_Pawar You should be switching 'modelid', not the list itself! pawn Код: switch(modelid)

That's true, thank you =)

[RajatPawar]

You are doing it wrong, mate!
It should be something like:

pawn Код:

switch(modelid)
{
                case 400: {//mycarvariable = 400; price = 600;}
}

[N0FeaR]

Quote:

Originally Posted by Rajat_Pawar You are doing it wrong, mate! It should be something like: pawn Код: switch(modelid) {                 case 400: {//mycarvariable = 400; price = 600;} }

No, this code working fine

Код:

case 0: {modelid = 400; price = 600;}