ADuty Befehl

RaPTiiC · 16. August 2020

Moin mal wieder kann mir jemand sagen was ich falsch mache?

Pawno compiled nicht mehr

Code

ocmd:aduty(playerid, params[])
{
    if(isAdmin(playerid, 1))return SCM(playerid, COLOR_RED, MESSAGE_NOADMIN) && if(aduty[playerid]==false)
    {
        new string[128];
        new Float:x, Float:y, Float:z;
        new Text3D:label = Create3DTextLabel("Teammitglied im Dienst!", COLOR_RED, x,y,z, 40.0, 0, 0);
        GetPlayerPos(playerid, x,y,z);
        oldskin[playerid] = GetPlayerSkin(playerid);
        SetPlayerHealth(playerid, 999999);
        SetPlayerArmour(playerid, 999999);
        SetPlayerColor(playerid, COLOR_RED);
        SetPlayerSkin(playerid, 217);
        format(string,sizeof(string),"Probe Supporter %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid));
        SendClientMessageToAll(COLOR_GOLD,string);
        Attach3DTextLabelToPlayer(label, playerid, 0.0, 0.0, 0.7);
        aduty[playerid] = true;
    }
    else if(isAdmin(playerid, 2))return SCM(playerid, COLOR_RED,MESSAGE_NOADMIN) && if(aduty[playerid]==false)
    {
        new string[128];
        new Float:x, Float:y, Float:z;
        new Text3D:label = Create3DTextLabel("Teammitglied im Dienst!", COLOR_RED, x,y,z, 40.0, 0, 0);
        GetPlayerPos(playerid, x,y,z);
        oldskin[playerid] = GetPlayerSkin(playerid);
        SetPlayerHealth(playerid, 999999);
        SetPlayerArmour(playerid, 999999);
        SetPlayerColor(playerid, COLOR_RED);
        SetPlayerSkin(playerid, 217);
        format(string,sizeof(string),"Supporter %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid));
        SendClientMessageToAll(COLOR_GOLD,string);
        Attach3DTextLabelToPlayer(label, playerid, 0.0, 0.0, 0.7);
        aduty[playerid] = true;
    }
    if(isAdmin(playerid, 3))return SCM(playerid, COLOR_RED,MESSAGE_NOADMIN) && if(aduty[playerid]==false)
    {
        new string[128];
        new Float:x, Float:y, Float:z;
        new Text3D:label = Create3DTextLabel("Teammitglied im Dienst!", COLOR_RED, x,y,z, 40.0, 0, 0);
        GetPlayerPos(playerid, x,y,z);
        oldskin[playerid] = GetPlayerSkin(playerid);
        SetPlayerHealth(playerid, 999999);
        SetPlayerArmour(playerid, 999999);
        SetPlayerColor(playerid, COLOR_RED);
        SetPlayerSkin(playerid, 217);
        format(string,sizeof(string),"Moderator %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid));
        SendClientMessageToAll(COLOR_GOLD,string);
        Attach3DTextLabelToPlayer(label, playerid, 0.0, 0.0, 0.7);
        aduty[playerid] = true;
    }
    if(isAdmin(playerid, 4))return SCM(playerid, COLOR_RED,MESSAGE_NOADMIN) && if(aduty[playerid]==false)
    {
        new string[128];
        new Float:x, Float:y, Float:z;
        new Text3D:label = Create3DTextLabel("Teammitglied im Dienst!", COLOR_RED, x,y,z, 40.0, 0, 0);
        GetPlayerPos(playerid, x,y,z);
        oldskin[playerid] = GetPlayerSkin(playerid);
        SetPlayerHealth(playerid, 999999);
        SetPlayerArmour(playerid, 999999);
        SetPlayerColor(playerid, COLOR_RED);
        SetPlayerSkin(playerid, 217);
        format(string,sizeof(string),"Administrator %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid));
        SendClientMessageToAll(COLOR_GOLD,string);
        Attach3DTextLabelToPlayer(label, playerid, 0.0, 0.0, 0.7);
        aduty[playerid] = true;
    }
    if(isAdmin(playerid, 4))return SCM(playerid, COLOR_RED,MESSAGE_NOADMIN) && if(aduty[playerid]==false)
    {
        new string[128];
        new Float:x, Float:y, Float:z;
        new Text3D:label = Create3DTextLabel("Teammitglied im Dienst!", COLOR_RED, x,y,z, 40.0, 0, 0);
        GetPlayerPos(playerid, x,y,z);
        oldskin[playerid] = GetPlayerSkin(playerid);
        SetPlayerHealth(playerid, 999999);
        SetPlayerArmour(playerid, 999999);
        SetPlayerColor(playerid, COLOR_RED);
        SetPlayerSkin(playerid, 217);
        format(string,sizeof(string),"Projektleiter %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid));
        SendClientMessageToAll(COLOR_GOLD,string);
        Attach3DTextLabelToPlayer(label, playerid, 0.0, 0.0, 0.7);
        aduty[playerid] = true;
    }
    else if(aduty[playerid]==true)
    {
         new string[128];
        new Float:x, Float:y, Float:z;
        new Text3D:label = Create3DTextLabel("Teammitglied im Dienst!", COLOR_RED, x,y,z, 40.0, 0, 0);
        Delete3DTextLabel(label);
        SetPlayerHealth(playerid, 100);
        SetPlayerSkin(playerid, oldskin[playerid]);
        format(string,sizeof(string),"Teammitglied %s hat den Dienst beendet!",GetName(playerid));
        SendClientMessageToAll(COLOR_GOLD,string);
        SetPlayerColor(playerid, COLOR_WHITE);
        aduty[playerid]=false;
    }
    return 1;
}

Alles anzeigen

**CaptainKevin** · 16. August 2020

Heyho,

du sendest einen Return & hängst direkt eine IF Abfrage dahinter, das funktioniert glaube ich nicht.

Probier das ganze mal so

Code

ocmd:aduty(playerid, params[])
{
    if(isAdmin(playerid, 1))return SCM(playerid, COLOR_RED, MESSAGE_NOADMIN)
    {
        if(aduty[playerid] == false) 
        {
            new string[128];
            new Float:x, Float:y, Float:z;
            new Text3D:label = Create3DTextLabel("Teammitglied im Dienst!", COLOR_RED, x,y,z, 40.0, 0, 0);
            GetPlayerPos(playerid, x,y,z);
            oldskin[playerid] = GetPlayerSkin(playerid);
            SetPlayerHealth(playerid, 999999);
            SetPlayerArmour(playerid, 999999);
            SetPlayerColor(playerid, COLOR_RED);
            SetPlayerSkin(playerid, 217);
            format(string,sizeof(string),"Probe Supporter %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid));
            SendClientMessageToAll(COLOR_GOLD,string);
            Attach3DTextLabelToPlayer(label, playerid, 0.0, 0.0, 0.7);
            aduty[playerid] = true;
        }
    }

Alles anzeigen

Wobei ich an deiner Stelle auch noch mit "Switch" arbeiten würde, um den Adminrang zu separieren, statt jedes Adminlevel einzelt durch zu gehen.
Außerdem brauchst du ja ein TextLabel für jeden Spieler & nicht nur eines, für den ganzen SERVER, ich glaube, das gibt auch Probleme.

Ich würde das warscheinlich so bauen (hab lange schon kein PAWN mehr programamiert):

Code

//Ganz oben im Script
new Text3D:Label[MAX_PLAYERS]; //Hier wird ein neues 3D Label erstellt mit der gesamten Anzahl an möglichen Spielern
new Aduty[MAX_PLAYERS]; //eine Variable, damit der Spieler zwischen dem Adminduty unterschieden werden kann

//Aduty
ocmd:aduty(playerid, params[]) //Befehl wird erstellt
{
    if(SpielerInfo[playerid][Adminlevel] < 1)return SCM(playerid, COLOR_RED, MESSAGE_NOADMIN); //Hier musst du natürlich selber über ein Enum abfragen, ob der Spieler ein Adminlevel mit größer 0 hat
    if(Aduty[playerid] == false) //Wenn der Spieler nicht im Adminduty ist
    {
        new string[128]; //wird ein neuer String erstellt
        new Float:x, Float:y, Float:z; //Variablen, damit wir die Position speichern können
        GetPlayerPos(playerid, x, y, z); //Position in die Variable schreiben
        oldskin[playerid] = GetPlayerSkin(playerid); //Der alte Skin wird in deine globale Variable geschrieben
        SetPlayerHealth(playerid, 999999); //Das Leben des Spielers wird auf Godmode gesetzt
        SetPlayerArmour(playerid, 999999); // das selbe gilt für die Armor
        SetPlayerSkin(playerid, 217); //der Spieler bekommt seinen Adminduty Skin
        Label[playerid] = Create3DTextLabel("Teammitglied im Dienst!", COLOR_RED, x, y, z, 40.0, 0, 0); //ein neues Label wird für den Spieler, der diesen Befehl eingibt erstellt
        Attach3DTextLabelToPlayer(Label[playerid], playerid, 0.0, 0.0, 0.7); //das Textlabel wird an den Spieler

        switch(SpielerInfo[playerid][Adminlevel]) //Eine Abfrage durch die Adminlevel
        {
            case 1: format(string, sizeof(string), "Probe-Supporter %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid)); //Bei Adminlevel 1           case 2: format(string, sizeof(string), "Supporter %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid)); //Bei Adminlevel 2                        case 3: format(string, sizeof(string), "Moderator %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid)); //bei Adminlevel 3            
            case 4: format(string, sizeof(string), "Administrator %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid)); //bei Adminlevel 4
            case 5: format(string, sizeof(string), "Projektleiter %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid)); //bei Adminlevel 5        
        }

        SendClientMessageToAll(COLOR_GOLD,string); //Eine Nachricht an alle Spieler senden
        Aduty[playerid] = true; //Adminlevel auf true setzen
    }
    else //Wenn der Spieler im Adminduty ist
    {
        new string[128]; //ein neuer String
        Aduty[playerid] = false; //Adminduty deaktivieren
        Delete3DTextLabel(Label[playerid]);//Text Label zerstören
        SetPlayerHealth(playerid, 100); //Leben wieder normal setzen
        SetPlayerSkin(playerid, oldskin[playerid]);//Skin wieder zurücksetzen
        format(string, sizeof(string), "Teammitglied %s hat den Dienst beendet!", GetName(playerid));//Nachricht formatieren
        SendClientMessageToAll(COLOR_GOLD, string);//Nachricht an alle schicken
        SetPlayerColor(playerid, COLOR_WHITE);//Farbe setzen
    }
    return 1;
}

Alles anzeigen

RaPTiiC · 16. August 2020

CaptainKevin erst einmal danke, jedes mal wenn doch jemand /aduty macht wird doch ein neues TextLabel erstellt oder nicht?

CaptainKevin

/edit bekomme jetzt diese Errors:

C:\Users\Dusti\Desktop\SA-MP Server\gamemodes\roleplay.pwn(1611) : error 029: invalid expression, assumed zero

C:\Users\Dusti\Desktop\SA-MP Server\gamemodes\roleplay.pwn(1611) : warning 215: expression has no effect

C:\Users\Dusti\Desktop\SA-MP Server\gamemodes\roleplay.pwn(1611) : error 001: expected token: ";", but found "if"

2 Errors.

Habe vorerst erst einmal einen neuen befehl erstellt:

Code

ocmd:aduty(playerid, params[])
{
    if(isAdmin(playerid, 1))return SCM(playerid, COLOR_RED, MESSAGE_NOADMIN);
    {
        if(aduty[playerid] == false)
        {
            new string[128];
            new Float:x, Float:y, Float:z;
            new Text3D:label = Create3DTextLabel("Teammitglied im Dienst!", COLOR_RED, x,y,z, 40.0, 0, 0);
            GetPlayerPos(playerid, x,y,z);
            oldskin[playerid] = GetPlayerSkin(playerid);
            SetPlayerHealth(playerid, 999999);
            SetPlayerArmour(playerid, 999999);
            SetPlayerColor(playerid, COLOR_RED);
            SetPlayerSkin(playerid, 217);
            format(string,sizeof(string),"Probe Supporter %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid));
            SendClientMessageToAll(COLOR_GOLD,string);
            Attach3DTextLabelToPlayer(label, playerid, 0.0, 0.0, 0.7);
            aduty[playerid] = true;
        }
    }
    else if(isAdmin(playerid, 2))return SCM(playerid, COLOR_RED, MESSAGE_NOADMIN); //1611
    {
        if(aduty[playerid] == false)
        {
            new string[128];
            new Float:x, Float:y, Float:z;
            new Text3D:label = Create3DTextLabel("Teammitglied im Dienst!", COLOR_RED, x,y,z, 40.0, 0, 0);
            GetPlayerPos(playerid, x,y,z);
            oldskin[playerid] = GetPlayerSkin(playerid);
            SetPlayerHealth(playerid, 999999);
            SetPlayerArmour(playerid, 999999);
            SetPlayerColor(playerid, COLOR_RED);
            SetPlayerSkin(playerid, 217);
            format(string,sizeof(string),"Probe Supporter %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid));
            SendClientMessageToAll(COLOR_GOLD,string);
            Attach3DTextLabelToPlayer(label, playerid, 0.0, 0.0, 0.7);
            aduty[playerid] = true;
        }
    }
    return 1;
}

Alles anzeigen

**CaptainKevin** · 16. August 2020

RaPTiiC : Ich habe meinen Beitrag oben einmal bearbeitet & dir einen komplett neuen Befehl erstellt.
Ich bearbeite diesen Beitrag jetzt nochmal & kommentiere dir den Code, damit du auch weist, was ich getan habe

RaPTiiC · 16. August 2020

CaptainKevin Danke nochmal ich sollte mir vielleicht angewöhnen mehr mit Cases zu arbeiten

/edit CaptainKevin

Bekomme leider immer noch ein Warning:

Code

ocmd:aduty(playerid, params[])
{
    if(PlayerInfo[playerid][pAdmin] < 1)return SCM(playerid, COLOR_RED, MESSAGE_NOADMIN);
    if(Aduty[playerid] == false) //Tag Mismatch
    {
        new string[128];
        new Float:x, Float:y, Float:z;
        GetPlayerPos(playerid, x, y, z);
        oldskin[playerid] = GetPlayerSkin(playerid);
        SetPlayerHealth(playerid, 999999);
        SetPlayerArmour(playerid, 999999);
        SetPlayerSkin(playerid, 217);
        Label[playerid] = Create3DTextLabel("Teammitglied im Dienst!", COLOR_RED, x, y, z, 40.0, 0, 0);
        Attach3DTextLabelToPlayer(Label[playerid], playerid, 0.0, 0.0, 0.7);
        switch(PlayerInfo[playerid][pAdmin])
        {
            case 1: format(string, sizeof(string), "Probe-Supporter %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid));
            case 2: format(string, sizeof(string), "Supporter %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid));
            case 3: format(string, sizeof(string), "Moderator %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid));
            case 4: format(string, sizeof(string), "Administrator %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid));
            case 5: format(string, sizeof(string), "Projektleiter %s ist in den Dient gegangen! Er ist Rot markiert!", GetName(playerid));
        }
        SendClientMessageToAll(COLOR_GOLD,string);
        Aduty[playerid] = true;
    }
    else
    {
        new string[128];
        Aduty[playerid] = false;
        Delete3DTextLabel(Label[playerid]);
        SetPlayerHealth(playerid, 100);
        SetPlayerSkin(playerid, oldskin[playerid]);
        format(string, sizeof(string), "Teammitglied %s hat den Dienst beendet!", GetName(playerid));
        SendClientMessageToAll(COLOR_GOLD, string);
        SetPlayerColor(playerid, COLOR_WHITE);
    }
    return 1;
}

Alles anzeigen

**CaptainKevin** · 16. August 2020

Welche Warning bekommst du denn?
Markiere mir einmal die Zeile, in welcher die Warnings auftreten.

RaPTiiC · 16. August 2020

CaptainKevin hab ich bereits im code Zeile 4 Tag Mismatch

**CaptainKevin** · 16. August 2020

Ansonsten ändere einfach die "true" bei allen Aduty[playerid] zu 1

und alle "false" zu 2

Ich weis leider nicht mehr genau, wie es bei Pawn mit den Bools war, ob die anders deklariert wurden oder so.

RaPTiiC · 16. August 2020

Hat alles geklappt vielen Dank nochmal CaptainKevin

**CaptainKevin** · 16. August 2020

Kein Problem
Sei bitte noch so nett & markiere meinen Beitrag mit "Lösung gefunden", damit auch alle anderen bescheid wissen, dass dieser Fall gelöst wurde.

Falls noch Fragen offen sind oder du eine Erklärung brauchst, gib mir einfach bescheid.

Schönen Abend noch

ADuty Befehl

Zum Hilfreichsten Beitrag springen

breadfish.de 12. März 2022

Ähnliche Themen

[SAMMELTHREAD] Kleine Scripting Fragen