Fehler Job "Müllmann"

Wichtiger Hinweis: Bitte ändert nicht manuell die Schriftfarbe auf schwarz sondern belasst es bei der Standardeinstellung. Somit tragt ihr dazu bei dass euer Text auch bei Verwendung unseren dunklen Forenstils noch lesbar ist!

Tipp: Ihr wollt längere Codeausschnitte oder Logfiles bereitstellen? Benutzt unseren eigenen PasteBin-Dienst Link

    Guten Abend, habe ein Problem wollte grade Scripten das wenn ich nicht im Auto bin /dienst nicht geht und wenn ich im Auto bin, /dienst funktioniert und ich die Checkpoints abfahren kann..
    Kann jemand weiterhelfen was ich daran falsch gemacht habe?


    if (strcmp("/dienst", cmd, true) == 0)
    {
    if(Mullabfuhr[playerid] == 0 && PlayerInfo[playerid][pJob] == 17)
    {
    if(IsPlayerInVehicle(playerid, 408))
    {
    if(GetVehicleModel(GetPlayerVehicleID(playerid)) == 408 || Mullabfuhr[playerid] == 0)
    {
    SendClientMessage(playerid, COLOR_LIGHTGREEN, "Du bist nun im Dienst, fahre bitte die Checkpoints ab!");
    SetPlayerCheckpoint(playerid, 351.8024,-1786.3950,5.0254, 5.0);
    MullabfuhrCP[playerid] = 1;
    Mullabfuhr[playerid] = 1;
    }
    else
    {
    SendClientMessage(playerid, COLOR_RED, "Du bist nicht im Müllwagen, oder hast bereits eine Runde gestartet!");
    }
    }
    else
    {
    SendClientMessage(playerid, COLOR_RED, "Du bist nicht im Trashmaster!");
    }
    return 1;
    }
    }


    Freue mich um Hilfe.


    Mfg Chicago

    Schreibe es mal so:


    if (strcmp("/dienst", cmd, true) == 0)
    {
    if(PlayerInfo[playerid][pJob] != 17) return SendClientMessage(playerid,COLOR_RED,"Du bist kein Müllmann!");
    if(GetVehicleModel(GetPlayerVehicleID(playerid)) != 408) return SendClientMessage(playerid, COLOR_RED, "Du bist nicht im Trashmaster!");
    if(Mullabfuhr[playerid]) return SendClientMessage(playerid, COLOR_RED, "Du hast bereits eine Runde gestartet!");
    SendClientMessage(playerid, COLOR_LIGHTGREEN, "Du bist nun im Dienst, fahre bitte die Checkpoints ab!");
    SetPlayerCheckpoint(playerid, 351.8024,-1786.3950,5.0254, 5.0);
    return MullabfuhrCP[playerid] = 1,Mullabfuhr[playerid] = 1,1;
    }


    mfg. :thumbup:

    ast2ufdyxkb1.png


    Leute, lernt scripten und versucht mal lieber etwas selber zu schreiben, als es aus einem GF zu kopieren. :S

    Zitat von xChicagoBulls

    if(IsPlayerInVehicle(playerid, 408))


    Das hier ist falsch, das prüft ob die vehicleid 408 ist, aber du willst wissen, ob die modelid 408 ist ;)


    Zitat von xChicagoBulls

    if(GetVehicleModel(GetPlayerVehicleID(playerid)) == 408 || Mullabfuhr[playerid] == 0)


    Hier prüfst du zwar ob die Modelid 408 ist, aber du hast ein || gemacht sprich wenn es 408 ist oder 0...richtig wäre aber ein && ;)


    mfg. :thumbup:

    ast2ufdyxkb1.png


    Leute, lernt scripten und versucht mal lieber etwas selber zu schreiben, als es aus einem GF zu kopieren. :S

    Achso okay das merk ich mir.. dankeschön :thumbup:
    Hab aber noch eine Frage undzwar möchte ich dass wenn man Waffen kauft Prods verliert.. wie mache ich das?


    BizzInfo[2][bTill] += 1151;


    Ich kenne nur das damit das Geld was man ausgibt in das Biz hinein kommt..