[davve95]

I'm making a vehicle selling system..

But there's pretty much job to write down every vehicle I have..
(Not that many one yet) But still pretty much job.

So are there any faster/smarter way??.. Not to hard one..


I have thought something like this, so far..

pawn Код:

else if(checkpointid == SellVeh)
    {
        
        if(GetVehicleModel(vehicleid) == 411) 
        {
            SendClientMessage(playerid, Red, "Test");
        }
    }
    return 1;
}

I need to check which car it is so I can have different price on them.


But I'm got a error code when I tired with that GetVehicleModel function..
While testing the code..

Код:

D:\Davids\Scripting\Server Las Venturas\LV.pwn(1699) : error 017: undefined symbol "vehicleid"
Pawn compiler 3.2.3664	 	 	Copyright © 1997-2006, ITB CompuPhase


1 Error.

Edit: Don't post a whole script just some example..

[fordawinzz]

use a variable

pawn Код:

new vehicleid = GetPlayerVehicleID(playerid);

or

pawn Код:

if(GetVehicleModel(GetPlayerVehicleID(playerid)) == 411)

[davve95]

Quote:

Originally Posted by fordawinzz use a variable pawn Код: new vehicleid = GetPlayerVehicleID(playerid); or pawn Код: if(GetVehicleModel(GetPlayerVehicleID(playerid)) == 411)

Thanks a lot I worked!.


Edit:


So you have to make like this on every vehicle if you want different price on every vehicle?:

pawn Код:

else if(checkpointid == SellVeh)
    {
        //Bikes:
        if(GetVehicleModel(GetPlayerVehicleID(playerid)) == 509) //Infernus
        {
            GivePlayerMoney(playerid, 5000);
        }
        
        if(GetVehicleModel(GetPlayerVehicleID(playerid)) == 481) //BMX
        {
            GivePlayerMoney(playerid, 5000);
        }
        
        if(GetVehicleModel(GetPlayerVehicleID(playerid)) == 510) //Mountain bike
        {
            GivePlayerMoney(playerid, 5000);
        }
        
        if(GetVehicleModel(GetPlayerVehicleID(playerid)) == 462) //Faggio
        {
            GivePlayerMoney(playerid, 5000);
        }
        
        if(GetVehicleModel(GetPlayerVehicleID(playerid)) == 448) //Pizzaboy
        {
            GivePlayerMoney(playerid, 5000);
        }

That are just a example, it isn't really done!..
It's like that temporary too.

[Excelize]

Quote:

Originally Posted by davve95 Thanks a lot I worked!. Edit: So you have to make like this on every vehicle if you want different price on every vehicle?: pawn Код: else if(checkpointid == SellVeh)     {         //Bikes:         if(GetVehicleModel(GetPlayerVehicleID(playerid)) == 509) //Infernus         {             GivePlayerMoney(playerid, 5000);         }                 if(GetVehicleModel(GetPlayerVehicleID(playerid)) == 481) //BMX         {             GivePlayerMoney(playerid, 5000);         }                 if(GetVehicleModel(GetPlayerVehicleID(playerid)) == 510) //Mountain bike         {             GivePlayerMoney(playerid, 5000);         }                 if(GetVehicleModel(GetPlayerVehicleID(playerid)) == 462) //Faggio         {             GivePlayerMoney(playerid, 5000);         }                 if(GetVehicleModel(GetPlayerVehicleID(playerid)) == 448) //Pizzaboy         {             GivePlayerMoney(playerid, 5000);         } That are just a example, it isn't really done!.. It's like that temporary too.

Hmm, you could make an enum, storing all the vehicle ids in it?

[newbienoob]

pawn Код:

else if(checkpointid == SellVeh)
    {
        new vehicleid = GetPlayerVehicleID(playerid);
        new model = GetVehicleModel(vehicleid);
        //Bikes:
        if(model == 509 || model == 481 || model == 510 || model == 462 || model == 448)
        }
            GivePlayerMoney(playerid, 5000);
        }

[cessil]

you should add all the vehicle information into a multi-dimensional array so you can do something like this

pawn Код:

else if(checkpointid == SellVeh)
    {
        new vehicleid = GetPlayerVehicleID(playerid);
        new model = GetVehicleModel(vehicleid);
        GivePlayerMoney(playerid, vehicleInfo[model][sellValue]);
        }

[davve95]

Quote:

Originally Posted by cessil you should add all the vehicle information into a multi-dimensional array so you can do something like this pawn Код: else if(checkpointid == SellVeh)     {         new vehicleid = GetPlayerVehicleID(playerid);         new model = GetVehicleModel(vehicleid);         GivePlayerMoney(playerid, vehicleInfo[model][sellValue]);         }

Sorry for a very long reply/bump, but I haven't scripted for a while
and I thought I did got it first time..

But I didn't got it to fully I noticed now when I checked back.
Anyway:

It's hard to explain what I don't get by it but I'll try to explain..

Don't I need to write every single vehicle I have with that method? You have showed?.

And are there anything more I have to do please show a example/explain .

Edit: I think I got it, but not fully sure xD.

Edit 2: I just don't know how I should do with the prices..