OnPlayerCommand Falsch? Fehler?

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
  • Hallo Liebe Community!
    Ich habe irgendwie ein Problem in meinem kleinem FS!
    Ich vermute ich habe die { und die } sowie die returns falsch gesetzt!


    FS:
    public OnPlayerCommandText(playerid, cmdtext[])
    {
    if (strcmp("/Mission", cmdtext, true, 10) == 0)
    {


    TogglePlayerControllable(playerid, 0);
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange, "Wie ich sehe, möchtest du eine Mission beginnen.");
    SendClientMessage(playerid,Orange,"Nun, diese geht sehr Einfach und ist eine Singleplayer Mission von Grand Theft Auto, San Andreas!");
    SendClientMessage(playerid,Orange,"Gleich wirst du Checkpoints Erhalten und eine Spraydose!");
    SendClientMessage(playerid,Orange,"In Dieser Mission musst du 10 Checkpoints abfahren und die Wände Ansprühen!");
    SendClientMessage(playerid,Orange,"Wenn du mit dem Letzten Fertig bist, wirst du etwas Geld Erhalten!");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Hellblau,"Ich hoffe, du hast dir alles Sorgfälltig durchgelesen!");
    SendClientMessage(playerid,Hellblau,"Bitte Bestätige mit /Mission931");
    }
    return 1;
    }
    { // zeile 111
    if (strcmp("/Mission931", cmdtext, true, 10) == 0) // Zeile 112
    {
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    TogglePlayerControllable(playerid, 1);
    SendClientMessage(playerid,Orange,"Wir hoffen, dass du diese Mission auch wirklich Erledigst!");
    }
    return 1; // Zeile 124
    }
    Fehler:
    MissionFeature.pwn(109) : warning 217: loose indentation
    MissionFeature.pwn(111) : error 055: start of function body without function header
    MissionFeature.pwn(112) : error 010: invalid function or declaration
    MissionFeature.pwn(124) : error 010: invalid function or declaration
    //MissionFeature.pwn(36) : warning 204: symbol is assigned a value that is never used: "pickup"



    3 Errors.

    Danke!

  • Jetzt mal im Ernst, das ist so schlecht, schreibe das mal so:


    public OnPlayerCommandText(playerid, cmdtext[])
    {
    if(!strcmp("/Mission", cmdtext, true))
    {
    TogglePlayerControllable(playerid, 0);
    for(new i; i<15; i++) SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange, "Wie ich sehe, möchtest du eine Mission beginnen.");
    SendClientMessage(playerid,Orange,"Nun, diese geht sehr Einfach und ist eine Singleplayer Mission von Grand Theft Auto, San Andreas!");
    SendClientMessage(playerid,Orange,"Gleich wirst du Checkpoints Erhalten und eine Spraydose!");
    SendClientMessage(playerid,Orange,"In Dieser Mission musst du 10 Checkpoints abfahren und die Wände Ansprühen!");
    SendClientMessage(playerid,Orange,"Wenn du mit dem Letzten Fertig bist, wirst du etwas Geld Erhalten!");
    for(new i; i<6; i++) SendClientMessage(playerid,Orange," ");
    SendClientMessage(playerid,Hellblau,"Ich hoffe, du hast dir alles Sorgfälltig durchgelesen!");
    SendClientMessage(playerid,Hellblau,"Bitte Bestätige mit /Mission931");
    return 1;
    }
    if(!strcmp("/Mission931", cmdtext, true))
    {
    for(new i; i<15; i++) SendClientMessage(playerid,Orange," ");
    TogglePlayerControllable(playerid, 1);
    SendClientMessage(playerid,Orange,"Wir hoffen, dass du diese Mission auch wirklich Erledigst!");
    return 1;
    }
    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

  • public OnPlayerCommandText(playerid, cmdtext[])
    {
    if (strcmp("/Mission", cmdtext, true, 10) == 0)
    {
    TogglePlayerControllable(playerid, 0);
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange, "Wie ich sehe, möchtest du eine Mission beginnen.");
    SendClientMessage(playerid,Orange,"Nun, diese geht sehr Einfach und ist eine Singleplayer Mission von Grand Theft Auto, San Andreas!");
    SendClientMessage(playerid,Orange,"Gleich wirst du Checkpoints Erhalten und eine Spraydose!");
    SendClientMessage(playerid,Orange,"In Dieser Mission musst du 10 Checkpoints abfahren und die Wände Ansprühen!");
    SendClientMessage(playerid,Orange,"Wenn du mit dem Letzten Fertig bist, wirst du etwas Geld Erhalten!");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Hellblau,"Ich hoffe, du hast dir alles Sorgfälltig durchgelesen!");
    SendClientMessage(playerid,Hellblau,"Bitte Bestätige mit /Mission931");
    return 1;
    }
    if (strcmp("/Mission931", cmdtext, true, 10) == 0) // Zeile 112
    {
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    SendClientMessage(playerid,Orange,"");
    TogglePlayerControllable(playerid, 1);
    SendClientMessage(playerid,Orange,"Wir hoffen, dass du diese Mission auch wirklich Erledigst!");
    return 1;
    }
    }

  • Danke, \MissionFeature.pwn(122) : warning 209: function "OnPlayerCommandText" should return a value Fehler? :D


    Nimm nicht das von PuSchKiiN_x3, nimm meins, über seinem Post, das ist besser und richtig :rolleyes:


    mfg. :thumbup:

    ast2ufdyxkb1.png


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

  • Jedoch, brauch ich die Leeren SendClientMessage's die Du entfernt hast


    Habe ich nicht, ich habe das durch eine Schleife ersetzt, was viel Platzspraender ist schau mal genauer hin :rolleyes:


    fehlt ein return am ende wahrscheinlich


    Ignoriere mich nicht so :(


    mfg. :rolleyes:

    ast2ufdyxkb1.png


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

  • do.de - Domain-Offensive - Domains für alle und zu super Preisen