ADuty Befehl

RaPTiiC · 16. August 2020

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

Pawno compiled nicht mehr

PAWN

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

Melden Sie sich an oder registrieren Sie sich!

Anmeldung

Registrierung

Anmeldung über Drittanbieter

ADuty Befehl

Aus gegebenem Anlass: Sichert eure Backupcodes wenn ihr die Zwei-Faktor-Authentifizierung aktiviert habt!

Zum Hilfreichsten Beitrag springen

Ähnliche Themen

[SAMMELTHREAD] Kleine Scripting Fragen