[oOegyptOo]

i think my code big O-o any way to make it shorter or its ok?

PHP код:

    if(!strcmp("/work", cmdtext, true))
    {
        if (minfo[playerid][CheckpointID] == 0)
        {
            new pvehiclemodel = GetVehicleModel(GetPlayerVehicleID(playerid));
            if (pvehiclemodel == 403 || pvehiclemodel == 514 || pvehiclemodel == 515)
            {
                if (IsTrailerAttachedToVehicle(GetPlayerVehicleID(playerid)))
                {
                    new string[200];
                    new rand = random(sizeof(TruckingMissionRandom));
                    CreatePlayerMission(playerid, TruckingMissionRandom[rand][UseTrailerCheck], TruckingMissionRandom[rand][MissionPay], TruckingMissionRandom[rand][loadx],TruckingMissionRandom[rand][loady], TruckingMissionRandom[rand][loadz], TruckingMissionRandom[rand][unloadx],TruckingMissionRandom[rand][unloady], TruckingMissionRandom[rand][unloadz]);
                    format(string, sizeof(string), "You are doing mission: %s", TruckingMissionRandom[rand][MissionName]);
                    SendClientMessage(playerid, 0x00FF00FF, string);
                }
                else
                {
                    SendClientMessage(playerid, 0xFF0000AA, "Im sorry but this mission requires that you use a vehicle that can pull a semi trailer Ex: Roadtrain with an Artict1!");
                }
            }
            else
            {
            SendClientMessage(playerid, 0xFF0000AA, "you need truck and tailer to be able to work");
            }
        }
        else
        {
        SendClientMessage(playerid, 0xFF0000AA, "you already have work");
        }
        return 1;
    } 


[FreshDoubleX]

Change your CreatePlayerMission line smaller:

pawn Код:

CreatePlayerMission(playerid, TruckingMissionRandom[rand][UseTrailerCheck], 
TruckingMissionRandom[rand][MissionPay], 
TruckingMissionRandom[rand][loadx],
TruckingMissionRandom[rand][loady], 
TruckingMissionRandom[rand][loadz], 
TruckingMissionRandom[rand][unloadx],
TruckingMissionRandom[rand][unloady], 
TruckingMissionRandom[rand][unloadz]);

[The_Gangstas]

pawn Код:

if(!strcmp("/work", cmdtext, true))
{
    if (minfo[playerid][CheckpointID] != 0) return SendClientMessage(playerid, 0xFF0000AA, "you already have work");
    new pvehiclemodel = GetVehicleModel(GetPlayerVehicleID(playerid));
    if (pvehiclemodel == 403 || pvehiclemodel == 514 || pvehiclemodel == 515) else { return SendClientMessage(playerid, 0xFF0000AA, "you need truck and tailer to be able to work");}
    if(!IsTrailerAttachedToVehicle(GetPlayerVehicleID(playerid))) return SendClientMessage(playerid, 0xFF0000AA, "This Mission Requres You To Have A Trailer Attached To Your Truck.");
    new string[128];
    new rand = random(sizeof(TruckingMissionRandom));
    CreatePlayerMission(playerid, TruckingMissionRandom[rand][UseTrailerCheck], TruckingMissionRandom[rand][MissionPay], TruckingMissionRandom[rand][loadx],TruckingMissionRandom[rand][loady], TruckingMissionRandom[rand][loadz],
    TruckingMissionRandom[rand][unloadx],TruckingMissionRandom[rand][unloady], TruckingMissionRandom[rand][unloadz]);
    format(string, sizeof(string), "You are doing mission: %s", TruckingMissionRandom[rand][MissionName]);
    SendClientMessage(playerid, 0x00FF00FF, string);
    return 1;
}

[oOegyptOo]

oO ur big

[oOegyptOo]

Quote:

Originally Posted by The_Gangstas pawn Код: if(!strcmp("/work", cmdtext, true)) {     if (minfo[playerid][CheckpointID] != 0) return SendClientMessage(playerid, 0xFF0000AA, "you already have work");     new pvehiclemodel = GetVehicleModel(GetPlayerVehicleID(playerid));     if (pvehiclemodel == 403 || pvehiclemodel == 514 || pvehiclemodel == 515) else { return SendClientMessage(playerid, 0xFF0000AA, "you need truck and tailer to be able to work");}     if (!IsTrailerAttachedToVehicle(GetPlayerVehicleID(playerid))) return SendClientMessage(playerid, 0xFF0000AA, "Im sorry but this mission requires that you use a vehicle that can pull a semi trailer Ex: Roadtrain with an Artict1!");     new string[128];     new rand = random(sizeof(TruckingMissionRandom));     CreatePlayerMission(playerid, TruckingMissionRandom[rand][UseTrailerCheck], TruckingMissionRandom[rand][MissionPay], TruckingMissionRandom[rand][loadx],TruckingMissionRandom[rand][loady], TruckingMissionRandom[rand][loadz],     TruckingMissionRandom[rand][unloadx],TruckingMissionRandom[rand][unloady], TruckingMissionRandom[rand][unloadz]);     format(string, sizeof(string), "You are doing mission: %s", TruckingMissionRandom[rand][MissionName]);     SendClientMessage(playerid, 0x00FF00FF, string);     return 1; }

really thanks

[FreshDoubleX]

Oh Yes, Thats if whole code is small, I thought that you had error because of too big line
Anyways, Good Luck !

[Steven82]

Your code aint that big. I once had a cmd on a script that used almost 150 lines.