[Areax]

Hello!

It's me again..Now I have another problem...I made a command /carcolor [color1] [color2], but doesn't work. I get no errors or warnings...

Code:

PHP код:

CMD:carcolor(playerid, params[])
{
    new color1;
    new color2;
    new string[200];
    new vehicleid;
    if(sscanf(params, "ii", color1, color2)) return SendClientMessage(playerid, COLOR_ORANGE, "Usage: /carcolor [color1] [color2]");
    if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, COLOR_RED, "ERROR: You Are not in a vehicle!");
    if(color1 < 0 || color1 > 20) return  SendClientMessage(playerid, COLOR_RED, "ERROR: Invalid Color! (0-20)");
    if(color2 < 0 || color2 > 20) return  SendClientMessage(playerid, COLOR_RED, "ERROR: Invalid Color! (0-20)");
    ChangeVehicleColor(vehicleid, color1, color2);
    format(string, sizeof(string), "[SYSTEM]: %s Has Changed Color Of His Vehicle To %d | %d!", GetName(playerid), color1, color2);
    SendClientMessageToAll(COLOR_DARKGREEN, string);
    return 1;
} 


Thanks

[Pottus]

Helps if you ..... vehicleid = GetPlayerVehicleID(playerid);

[Yashas]

You have an INVALID VEHICLE ID in vehicleid variable.You never initialized it!!

Use GetPlayerVehicleID(playerid) to get vehicle id

https://sampwiki.blast.hk/wiki/GetPlayerVehicleID

Код:

vehicleid = GetVehicleID(playerid);

[Areax]

Quote:

Originally Posted by [uL]Pottus Helps if you ..... vehicleid = GetPlayerVehicleID(playerid);

Thank you Rep+

[Babul]

may i suggest you to use the optional parameters for integers, so a player can use the command with ONE color only? like

pawn Код:

if(sscanf(params, "iI(126)", color1, color2))

if you make the second color -1 as default, you can later check for that, and then assign the first color to it:

pawn Код:

if(sscanf(params, "iI(-1)", color1, color2))

if(color2==-1)color2=color1;

btw, the really good colors start @ 127 now

[Yashas]

Yup!!All the colors I used above 127 shine a lot making car realistic
The actual color limit is more than 200!!!

[Areax]

Thanks guys