[Castle]

I've made this command: http://pastebin.com/7sJhYJEg (Old fashioned I know).
When I type '/veh' without any variables should show: USAGE: /veh [CARID] [COLOR1] [COLOR2]
Instead shows: ** Vehicle model ID can't be below 400 or abor 611!
Could anyone give me a hint where to look?

[(SF)Noobanatior]

pawn Code:

new tmp[128];
    new idx,car,color1,color2,scar;
    new Float:X,Float:Y,Float:Z,Float:A;
    tmp=strtok(cmdtext,idx);//<--here
    if(!strcmp("/veh",tmp,true,4)){//<--- then tmp not cmdtext here
        tmp=strtok(cmdtext,idx);
        if(!strlen(tmp))return SendClientMessage(playerid,COLOR_GREY,"USAGE: /veh [CARID] [COLOR1] [COLOR2]"); //No variable typed
        car=strval(tmp);
        if(car<400||car>611)return SendClientMessage(playerid,COLOR_GREY,"** Vehicle model ID can't be below 400 or abor 611!");//carid
        tmp=strtok(cmdtext,idx);
        if(!strlen(tmp))return SendClientMessage(playerid,COLOR_GREY,"USAGE: /veh [CARID] [COLOR1] [COLOR2]");//color 1
        color1=strval(tmp);
        if(color1<-1||color1>255)return SendClientMessage(playerid,COLOR_GREY,"**    Color must be between -1 to 255");
        tmp=strtok(cmdtext,idx);
        if(!strlen(tmp))return SendClientMessage(playerid,COLOR_GREY,"USAGE: /veh [CARID] [COLOR1] [COLOR2]");//color 2
        color1=strval(tmp);
        if(color2<-1||color2>255)return SendClientMessage(playerid,COLOR_GREY,"**    Color must be between -1 to 255");//color 2
        GetPlayerPos(playerid,X,Y,Z);
        GetPlayerFacingAngle(playerid,A);//player position
        scar=CreateVehicle(car,X,Y,Z,A,color1,color2,999999);
        PutPlayerInVehicle(playerid,scar,0);//spawning car
        return 1;
    }

like this you need to strip the command of the front of cmdtext first for it to work properly

[Castle]

Quote:

Originally Posted by (SF)Noobanatior pawn Code: new tmp[128];     new idx,car,color1,color2,scar;     new Float:X,Float:Y,Float:Z,Float:A;     tmp=strtok(cmdtext,idx);//<--here     if(!strcmp("/veh",tmp,true,4)){//<--- then tmp not cmdtext here         tmp=strtok(cmdtext,idx);         if(!strlen(tmp))return SendClientMessage(playerid,COLOR_GREY,"USAGE: /veh [CARID] [COLOR1] [COLOR2]"); //No variable typed         car=strval(tmp);         if(car<400||car>611)return SendClientMessage(playerid,COLOR_GREY,"** Vehicle model ID can't be below 400 or abor 611!");//carid         tmp=strtok(cmdtext,idx);         if(!strlen(tmp))return SendClientMessage(playerid,COLOR_GREY,"USAGE: /veh [CARID] [COLOR1] [COLOR2]");//color 1         color1=strval(tmp);         if(color1<-1||color1>255)return SendClientMessage(playerid,COLOR_GREY,"**    Color must be between -1 to 255");         tmp=strtok(cmdtext,idx);         if(!strlen(tmp))return SendClientMessage(playerid,COLOR_GREY,"USAGE: /veh [CARID] [COLOR1] [COLOR2]");//color 2         color1=strval(tmp);         if(color2<-1||color2>255)return SendClientMessage(playerid,COLOR_GREY,"**    Color must be between -1 to 255");//color 2         GetPlayerPos(playerid,X,Y,Z);         GetPlayerFacingAngle(playerid,A);//player position         scar=CreateVehicle(car,X,Y,Z,A,color1,color2,999999);         PutPlayerInVehicle(playerid,scar,0);//spawning car         return 1;     } like this you need to strip the command of the front of cmdtext first for it to work properly

Thanks, I didn't script for long time forgot nearly all the shit..