[FaZeRs]

When i try to put fare it doesnt work

CMD

PHP код:

CMD:taxi(playerid, params[])
{
    new string[128];
    new option[11], secoption, vehicle = GetPlayerVehicleID(playerid);
    if(PlayerInfo[playerid][pMember] == 10||PlayerInfo[playerid][pLeader] == 10|| PlayerInfo[playerid][pJob] == 17 || PlayerInfo[playerid][pJob2] == 17 || PlayerInfo[playerid][pTaxiLicense] == 1)
    {
    if(sscanf(params,"s[32]", option, secoption))
    {
        SendClientMessage(playerid,-1, "/taxi [option]"); // Error here 
        SendClientMessageEx(playerid, COLOR_GREY, "Options: | duty | start | stop | fare | accept |");
        return 1;
    }
    if(CompareStrings(option, "duty"))
    {
        if(!IsDriver(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You must be inside a Taxi/Cabbie as a driver.");
        if(!IsATaxi(vehicle)) return SendClientMessage(playerid, COLOR_LIGHTRED, "You must be inside a Taxi/Cabbie.");
        if(TaxiDuty{playerid})
        {
            SendClientMessage(playerid, COLOR_YELLOW, "[TAXI]: You are now OFF duty taxi and won't recieve calls.");
            SetPlayerToTeamColor(playerid);
            TaxiDuty{playerid} = false;
            TaxiMade[playerid] = 0;
        }
        else
        {
            SendClientMessage(playerid, COLOR_YELLOW, "[TAXI]: You are now ON duty taxi and will recieve calls.");
            SetPlayerToTeamColor(playerid);
            TaxiDuty{playerid} = true;
            TaxiMade[playerid] = 0;
        }
    }
    if(CompareStrings(option, "start"))
    {
        if(!IsDriver(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You must be inside a Taxi/Cabbie as a driver.");
        if(!IsATaxi(vehicle)) return SendClientMessage(playerid, COLOR_LIGHTRED, "You must be inside a Taxi/Cabbie.");
        if(!TaxiDuty{playerid}) return SendClientMessage(playerid, COLOR_GREY, "You are not on duty.");
        if(TaxiFare{playerid} == 0) return SendClientMessage(playerid, COLOR_GREY, "You haven't set a fare yet, /taxi fare ($1-$25)");
        if(TaxiStart{playerid}) return SendClientMessage(playerid, COLOR_GREY, "Your taxi fare has already started.");
        SendClientMessage(playerid, COLOR_YELLOW, "[TAXI]: Your taxi fare has started and charging all people inside your vehicle.");
        TaxiStart{playerid} = true;
    }
    if(CompareStrings(option, "fare"))
    {
        new fare;
        if(!IsDriver(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You must be inside a Taxi/Cabbie as a driver.");
        if(!IsATaxi(vehicle)) return SendClientMessage(playerid, COLOR_LIGHTRED, "You must be inside a Taxi/Cabbie.");
        if(sscanf(params, "d",fare)) return SendClientMessageEx(playerid, COLOR_WHITE, "/taxi fare [amount]");
        if(!TaxiDuty{playerid}) return SendClientMessageEx(playerid, COLOR_GREY, "You are not on duty.");
        if(fare < 1 || fare > 25) return SendClientMessage(playerid, COLOR_GREY, "There is a limit for taxi fare, ($1-$25).");
        format(string, sizeof(string), "[TAXI]: You have changed your taxi fare to $%d.", fare);
        SendClientMessage(playerid, COLOR_YELLOW, string);
        TaxiFare{playerid} = fare;
    }
    if(CompareStrings(option, "accept"))
    {
        new id;
        if(!IsDriver(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You must be inside a Taxi/Cabbie as a driver.");
        if(!IsATaxi(vehicle)) return SendClientMessage(playerid, COLOR_LIGHTRED, "You must be inside a Taxi/Cabbie.");
        if(sscanf(params,"i",id)) return SendClientMessageEx(playerid, COLOR_WHITE, "/taxi accept [playerid]");
        if(!TaxiDuty{playerid}) return SendClientMessageEx(playerid, COLOR_GREY, "You are not on duty.");
        if(!IsPlayerConnected(id)) return Error(playerid, "That player is not connected");
        if(GetPVarInt(id, "NeedTaxi") == 0) return SendClientMessage(playerid, COLOR_LIGHTRED, "This player hasn't requested any taxi.");
        format(string, sizeof(string), "* %s has accepted your taxi call, he charges %d per second!", GetPlayerNameEx(playerid), TaxiFare{playerid});
        SendClientMessage(id, COLOR_YELLOW, string);
        format(string, sizeof(string), "* You have accepted %s taxi call.", GetPlayerNameEx(id));
        SendClientMessage(playerid, COLOR_YELLOW, string);
    }
    if(CompareStrings(option, "stop"))
    {
        if(!IsDriver(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You must be inside a Taxi/Cabbie as a driver.");
        if(!IsATaxi(vehicle)) return SendClientMessage(playerid, COLOR_LIGHTRED, "You must be inside a Taxi/Cabbie.");
        if(!TaxiDuty{playerid}) return SendClientMessage(playerid, COLOR_GREY, "You are not on duty.");
        if(!TaxiStart{playerid}) return SendClientMessage(playerid, COLOR_GREY, "Your taxi fare is not started.");
        SendClientMessage(playerid, COLOR_YELLOW, "[TAXI]: Your taxi fare has stopped and charges all people inside.");
        format(string, sizeof(string), "[TAXI]: The final fare of your taxi has reached to, $%d.", TaxiMade[playerid]);
        SendClientMessage(playerid, COLOR_YELLOW, string);
        foreach(Player, i)
        {
            if(PlayerInCar(i) && GetPlayerState(i) != PLAYER_STATE_DRIVER && GetPlayerVehicleID(i) == GetPlayerVehicleID(playerid))
            {
                ChargePerson(i);
            }
        }
        TaxiStart{playerid} = false;
        TaxiMade[playerid] = 0;
    }
        }
    else
    {
        SendClientMessageEx(playerid,COLOR_GREY,"You're not a licensed taxi/bus driver!");
        return 1;
    }
    return 1;
} 


[CuervO]

You're storing two parameters in the sscanf check but only checking for one.

Quote:

if(sscanf(params,"s[32]", option, secoption))

V

Quote:

if(sscanf(params,"s[32]I(65535)", option, secoption))

If you do sscanf once again below in the command it will not begin after one parameter is checked. Remove the other sscanf checks you use below in the code. I has a default value of 65535 so it can be used with players too.

[FaZeRs]

But when i set fare i have

Quote:

if(sscanf(params, "d",fare))