[Hilfe] Errors :/

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
  • Gute Morgen Breadfisch!
    Ich bekomme bei dem Code 1nen Error :P
    Ich bin Neuling im Scripten :)


    [pwn=5056]
    if (strcmp("/perso", cmdtext, true) == 0)
    {
    if (!IsPlayerInRangeOfPoint(playerid,2.0,358.4083,165.5182,1008.3828))
    {
    return SendClientMessage(playerid,COLOR_RED,"du bist nicht in der Stadthalle !");
    }
    if(GetPVarInt(playerid,"Perso") == 1)
    {
    return SendClientMessage(playerid,COLOR_RED,"Du hast bereits einen Personalausweiß!");
    }
    else
    {
    SetTimerEx("Persotimer1",9*1000*100,0,"i",playerid);
    SendClientMessage(playerid,BLAU,"Spiele nun einfach weiter , du wirst eine Nachricht bekommen wenn dein Perso fertig ist!");
    return 1;
    }
    if(GetPVarInt(playerid,"Persotimer") == 0)
    {
    SendClientMessage(playerid,COLOR_RED,"Der Antrag ist in arbeit!");
    return 1;
    }
    else
    {
    SetPVarInt(playerid,"Perso",1);
    SendClientMessage(playerid,COLOR_GREEN,"Du hast nun einen Personalausweiß!");
    return 1;
    }
    }
    [/pwn]


    Code
    (5072) : warning 225: unreachable code

    [tabmenu][tab=Produktion][subtab='iProduction']

    [subtab='Projekt']

  • Der Quellcode ist sehr kaotisch und ich bin mir aufgrund der unvorhandenen Kenntnisse, was die einzelnen Variablen tuen nicht richtig in der Lage dir eine zu 100% funktionierende Lösung zu sagen, aber probiere es mal so:


    if (strcmp(cmdtext, "/perso", true) == 0)//Das kannst du auch so machen wie du es gemacht hast, finde es aber so schöner xD
    {
    if (!IsPlayerInRangeOfPoint(playerid,2.0,358.4083,165.5182,1008.3828)) return SendClientMessage(playerid,COLOR_RED,"du bist nicht in der Stadthalle !");
    if(GetPVarInt(playerid,"Perso") == 1) return SendClientMessage(playerid,COLOR_RED,"Du hast bereits einen Personalausweiß!");
    if(GetPVarInt(playerid,"Persotimer") == 1) return SendClientMessage(playerid,COLOR_RED,"Der Antrag ist in arbeit!");
    if(GetPVarInt(playerid,"Persotimer") == 0)
    {
    SetPVarInt(playerid,"Perso",1);
    SendClientMessage(playerid,COLOR_GREEN,"Du hast nun einen Personalausweiß!");
    return 1;
    }
    SetTimerEx("Persotimer1",9*1000*100,0,"i",playerid);
    SendClientMessage(playerid,BLAU,"Spiele nun einfach weiter , du wirst eine Nachricht bekommen wenn dein Perso fertig ist!");
    return 1;
    }


    mfg. :thumbup:

    ast2ufdyxkb1.png


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

  • Soo jetzt habe ich eine Frage :P
    In dem Beispiel Zeile 21-22!
    [pwn=10]if (strcmp(cmdtext, "/perso", true) == 0)//Das kannst du auch so machen wie du es gemacht hast, finde es aber so schöner xD
    {
    if (!IsPlayerInRangeOfPoint(playerid,2.0,358.4083,165.5182,1008.3828)) return SendClientMessage(playerid,COLOR_RED,"du bist nicht in der Stadthalle !");
    if(GetPVarInt(playerid,"Perso") == 1) return SendClientMessage(playerid,COLOR_RED,"Du hast bereits einen Personalausweiß!");
    if(GetPVarInt(playerid,"Persotimer") == 1) return SendClientMessage(playerid,COLOR_RED,"Der Antrag ist in arbeit!");
    if(GetPVarInt(playerid,"Persotimer") == 0)
    {
    SetPVarInt(playerid,"Perso",1);
    SendClientMessage(playerid,COLOR_GREEN,"Du hast nun einen Personalausweiß!");
    return 1;
    }
    SetTimerEx("Persotimer1",9*1000*100,0,"i",playerid);
    SendClientMessage(playerid,BLAU,"Spiele nun einfach weiter , du wirst eine Nachricht bekommen wenn dein Perso fertig ist!");
    return 1;
    }[/pwn]


    könnten die beiden Zeilen auch so sein ?


    [pwn=10]if (strcmp(cmdtext, "/perso", true) == 0)//Das kannst du auch so machen wie du es gemacht hast, finde es aber so schöner xD
    {
    if (!IsPlayerInRangeOfPoint(playerid,2.0,358.4083,165.5182,1008.3828)) return SendClientMessage(playerid,COLOR_RED,"du bist nicht in der Stadthalle !");
    if(GetPVarInt(playerid,"Perso") == 1) return SendClientMessage(playerid,COLOR_RED,"Du hast bereits einen Personalausweiß!");
    if(GetPVarInt(playerid,"Persotimer") == 1) return SendClientMessage(playerid,COLOR_RED,"Der Antrag ist in arbeit!");
    SetTimerEx("Persotimer1",9*1000*100,0,"i",playerid);
    SendClientMessage(playerid,BLAU,"Spiele nun einfach weiter , du wirst eine Nachricht bekommen wenn dein Perso fertig ist!");
    if(GetPVarInt(playerid,"Persotimer") == 0)
    {
    SetPVarInt(playerid,"Perso",1);
    SendClientMessage(playerid,COLOR_GREEN,"Du hast nun einen Personalausweiß!");
    return 1;
    }
    return 1;
    }[/pwn]

    [tabmenu][tab=Produktion][subtab='iProduction']

    [subtab='Projekt']

  • Hä dort ist doch nur ein Warning


    Ja, aber der Warning steht da zurecht, da das return 1; den weiterführenden Code blockiert ;)


    könnten die beiden Zeilen auch so sein ?


    Nein, da er dann wenn er den Personalausweiß bekommen soll, nochmal der Timer startet ^^


    mfg. :thumbup:

    ast2ufdyxkb1.png


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