Problem beim ersten Selfmade

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
  • hey leute


    Ich versuche mich auch grade an einem selfmadereallife


    ich habe bis jetzt nur 1 problem


    ich habe den befehl /pdhelp erstellt



    if(strcmp("/pdhelp",cmdtext,true) == 0)
    {
    if(pInfo[playerid][gTeam] != Polizei || pInfo[playerid][gLeader] != Polizei)return SendClientMessage(playerid, COLOR_RED, "Du bist kein Cop");
    {
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__Hier die Befehle der Polizei__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdhs [ID] || Handschellen anlegen");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdhsab [ID] || Handschellen ablegen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/jail [ID] || Jemanden in den Knast setzen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/unjail [ID] || Jemanden Rauslassen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdgw [ID] || Jemanden EIN Wanted geben__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/psc [ID] || Jemanden Clearen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pspc [ID] || Den Polizeicomputer anschalten__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdduty || Ausrüsten__");
    }
    return 1;
    }


    wenn ich nun aber /pdhelp mache kommt immer die meldung "Du Bist Kein Cop"


    es könnte auch am /makelader liegen




    if(!strcmp(strget(cmdtext,0),"/makeleader"))
    {
    new target = strval(strget(cmdtext,1)), fraktion = strval(strget(cmdtext,2)), saves[32], name[20];
    format(saves,sizeof(saves),"Spieler/%s.ini",name);
    if(!strlen(strget(cmdtext,1))) return SendClientMessage(playerid, COLOR_RED, "Benutzte: /makeleader [ID] [1-12]");
    {
    if(!strlen(strget(cmdtext,2))) return SendClientMessage(playerid, COLOR_RED, "Benutzte: /makeleader [ID] [1-12]");
    {
    if(fraktion == Polizei)
    {
    SetPlayerSkin(target, 288);
    SendClientMessage(target, COLOR_WHITE, "Du wurdest zum Leader der Polizei gemacht");
    pInfo[target][gLeader] = Polizei;
    dini_IntSet(save,"Fraktionsleader",pInfo[target][gLeader]);
    }


    Ich hoffe mir kann jemand helfen xD


    m.f.g hallo_wars

  • Hey hatt geklapt ok was solte es sonst in diesem Forum xD






    Danke


    //EDIT


    Ohh grade ingame getestet kommt immer noch das ich kein cop wäre ;(

  • if(strcmp("/pdhelp",cmdtext,true) == 0)
    {
    if(pInfo[playerid][gTeam] != Polizei || pInfo[playerid][gLeader] != Polizei);
    {
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__Hier die Befehle der Polizei__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdhs [ID] || Handschellen anlegen");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdhsab [ID] || Handschellen ablegen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/jail [ID] || Jemanden in den Knast setzen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/unjail [ID] || Jemanden Rauslassen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdgw [ID] || Jemanden EIN Wanted geben__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/psc [ID] || Jemanden Clearen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pspc [ID] || Den Polizeicomputer anschalten__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdduty || Ausrüsten__");
    }
    else
    {
    SendClientMessage(playerid, COLOR_RED, "Du bist kein Cop");
    }
    return 1;
    }
    http://pastebin.com/LwEuy41H


    Probiers mal damit..


    Ich bin mir nichtmehr sicher ob das so passt^^ Aufjedenfall danach mit Else machen


    (if Player) Wenn spieler cop ist
    ## zeige cophilfe ##
    (else) wenn nicht
    ## Schreibe Du bist kein Cop ##



    So musste dir das vorstelln^^ Weis nur nichtmehr wie das mitm Else geht :E



    - LolRiTTeR

    [align=center]

  • eigentlich ist alles hier falsch außer der post vom threadersteller ist einigermaßen gut ;)


    if(strcmp("/pdhelp",cmdtext,true) == 0)
    {
    if(pInfo[playerid][gTeam] != Polizei && pInfo[playerid][gLeader] != Polizei)return SendClientMessage(playerid, COLOR_RED, "Du bist kein Cop");
    {
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__Hier die Befehle der Polizei__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdhs [ID] || Handschellen anlegen");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdhsab [ID] || Handschellen ablegen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/jail [ID] || Jemanden in den Knast setzen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/unjail [ID] || Jemanden Rauslassen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdgw [ID] || Jemanden EIN Wanted geben__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/psc [ID] || Jemanden Clearen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pspc [ID] || Den Polizeicomputer anschalten__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdduty || Ausrüsten__");
    }
    return 1;
    }


    edit: zur erklärung wenn du || heißt das oder und da du verneinst wird es so ausschauen


    ist playerid kein Polizist oder kein Leader mach das , also musst du es mit && machen
    ist playerid kein polizist und kein leader mach das ...

  • if(strcmp("/pdhelp",cmdtext,true) == 0)
    {
    if(pInfo[playerid][gTeam] != Polizei && pInfo[playerid][gLeader] != Polizei) return SendClientMessage(playerid, COLOR_RED, "Du bist kein Cop");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__Hier die Befehle der Polizei__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdhs [ID] || Handschellen anlegen");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdhsab [ID] || Handschellen ablegen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/jail [ID] || Jemanden in den Knast setzen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/unjail [ID] || Jemanden Rauslassen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdgw [ID] || Jemanden EIN Wanted geben__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/psc [ID] || Jemanden Clearen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pspc [ID] || Den Polizeicomputer anschalten__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdduty || Ausrüsten__");
    return 1;
    }
    oder
    if(strcmp("/pdhelp",cmdtext,true) == 0)
    {
    if(pInfo[playerid][gTeam] == Polizei || pInfo[playerid][gLeader] == Polizei)
    {
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__Hier die Befehle der Polizei__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdhs [ID] || Handschellen anlegen");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdhsab [ID] || Handschellen ablegen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/jail [ID] || Jemanden in den Knast setzen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/unjail [ID] || Jemanden Rauslassen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdgw [ID] || Jemanden EIN Wanted geben__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/psc [ID] || Jemanden Clearen__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pspc [ID] || Den Polizeicomputer anschalten__");
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "__/pdduty || Ausrüsten__");
    return 1;
    }
    else return SendClientMessage(playerid, COLOR_RED, "Du bist kein Cop");
    }


    @[iP]Zabus
    Das was du da geschrieben hast würde auch nicht funktionieren ;)

    The fact is, I am right. And if you think I'm wrong, you are wrong.

  • ja bei mir sind die klammern zu viel das habe ich übersehen ;) , aber was die anderen für Lösungsvorschläge haben ist eine Katastrophe , aber dein zweiter lösungsvorschlag hat auch einen Fehler :P xD


    else return SendClientMessage(playerid, COLOR_RED, "Du bist kein Cop");


    in


    else {return SendClientMessage(playerid, COLOR_RED, "Du bist kein Cop");}


    keine Ahnung warum da else klammern verlangt aber habs gleich nochmal getestet und dann kommt ein Fehler beim compilen wenn man es ohne klammer macht ;)

  • nee klappt alles ne


    vieleicht liegt es ja am /makeleader




    if(!strcmp(strget(cmdtext,0),"/makeleader"))
    {
    new target = strval(strget(cmdtext,1)), fraktion = strval(strget(cmdtext,2)), saves[32], name[20];
    format(saves,sizeof(saves),"Spieler/%s.ini",name);
    if(!strlen(strget(cmdtext,1))) return SendClientMessage(playerid, COLOR_RED, "Benutzte: /makeleader [ID] [1-12]");
    {
    if(!strlen(strget(cmdtext,2))) return SendClientMessage(playerid, COLOR_RED, "Benutzte: /makeleader [ID] [1-12]");
    {
    if(fraktion == Polizei)
    {
    SetPlayerSkin(target, 288);
    SendClientMessage(target, COLOR_WHITE, "Du wurdest zum Leader der Polizei gemacht");
    pInfo[target][gLeader] = Polizei;
    dini_IntSet(save,"Fraktionsleader",pInfo[target][gLeader]);
    }


    ??