aduty geht nicht

mach aus den

else if(aduty[playerid]==1)

ein:

else

Versuch mal so:
ocmd:aduty(playerid,params[])
{
new aduty[MAX_PLAYERS], astr[128];
if(!IstSpielerAdmin(playerid, 1))
{
return SendClientMessage(playerid, ROT, "Du bist kein Admin");
}
if(aduty[playerid] == 0)
{
format(astr, sizeof(astr), "Admin %s hat sich zum Support angemeldet.", SpielerName(playerid));
SendClientMessageToAll(BLAU, astr);
aduty[playerid] = 1;
return 1;
}
if(aduty[playerid] == 1)
{
format(astr, sizeof(astr), "Admin %s hat sich vom Support abgemeldet.", SpielerName(playerid));
SendClientMessageToAll(BLAU, astr);
aduty[playerid] = 0;
return 1;
}
return 1;
}

if(aduty[playerid]==0)
{
format(astr,sizeof(astr),"Admin %s hat sich zum Support angemeldet.",SpielerName(playerid));
SendClientMessageToAll(BLAU,astr);
aduty[playerid] = 1;
return 1;
}
if(aduty[playerid]==1)
{
format(astr1,sizeof(astr1),"Admin %s hat sich vom Support abgemeldet.",SpielerName(playerid));
SendClientMessageToAll(BLAU,astr1);
aduty[playerid] = 0;
return 1;
}

Versuchs mal so.

Benutzt du sonst irgendwo die Variable aduty??

ocmd:aduty(playerid,params[])
{
new string[128];
if(!isPlayerAnAdmin(playerid,1))return SendClientMessage(playerid,ROT,"Du bist kein Admin.");
if(!aduty{playerid})
{
format(string,sizeof(string),"**Administrator %s ist den Suppoter Dienst beigetreten.",SpielerName(playerid));
SendClientMessageToAll(0x00D7FFFF,string);
SetPlayerColor(playerid, rot);
aduty{playerid} = true;
}else{
if(!isPlayerAnAdmin(playerid,1))return SendClientMessage(playerid,ROT,"Du bist kein Admin.");
format(string,sizeof(string),"**Administrator %s hat den Supporter Dienst verlassen",SpielerName(playerid));
SendClientMessageToAll(0x00D7FFFF,string);
SetPlayerColor(playerid, 0xFFFFFFFF);
aduty{playerid} = false;
}
return 1;
}

SO müsste es gehen

Zitat von Nils_Alvarez

SO müsste es gehen

Wenn du nicht scripten kannst dann solltest du es lassen...

Bin anfänger und zugleich funktioniert der Befehl bestens :))

ach und setze das ganz oben in dein Script hinein.
new
bool:aduty[MAX_PLAYERS char];

Den

Zitat von Nils_Alvarez

Bin anfänger und zugleich funktioniert der Befehl bestens :))

ach und setze das ganz oben in dein Script hinein.
new
bool:aduty[MAX_PLAYERS char];

Den ersten vorschlag den du geschrieben hast ist völliger unsinn... Du kannst eine Variable nicht mit {} abfragen oder ändern!

Zitat von xXManuelXx

Wenn du nicht scripten kannst dann solltest du es lassen...

Wenn er Anfänger ist?? Sowelche Kommis sind richtig unnötig!

Zitat von Tonebag

Wenn er Anfänger ist?? Sowelche Kommis sind richtig unnötig!

Naja dann soll ers runter schreiben und nicht sagen "SO müsste es gehen"

Zitat von xXManuelXx

Den

Den ersten vorschlag den du geschrieben hast ist völliger unsinn... Du kannst eine Variable nicht mit {} abfragen oder ändern!

anscheinend bist du hier der anfänger, denn man kann Variablen mit { } machen!
Wenn die Variable mit new ...[MAX_PLAYERS char]; definiert ist, dann muss man { } statt [ ] benutzen!
Also erkundige dich erst!

MfG

ocmd:aduty(playerid,params[])
{
new aduty[MAX_PLAYERS],astr[128],astr1[128];
if(!IstSpielerAdmin(playerid,1)) return SendClientMessage(playerid,ROT,"Du bist kein Admin");
if(aduty[playerid]==0)
{
format(astr,sizeof(astr),"Admin %s hat sich zum Support angemeldet.",SpielerName(playerid));
SendClientMessageToAll(BLAU,astr);
aduty[playerid] = 1;
}
else if(aduty[playerid]==1)
{
format(astr1,sizeof(astr1),"Admin %s hat sich vom Support abgemeldet.",SpielerName(playerid));
SendClientMessageToAll(BLAU,astr1);
aduty[playerid] = 0;
}
return true;
}

deine returns...
sollte so funktionieren

Zitat von s h a i N

ocmd:aduty(playerid,params[])
{
new aduty[MAX_PLAYERS],astr[128],astr1[128];
if(!IstSpielerAdmin(playerid,1)) return SendClientMessage(playerid,ROT,"Du bist kein Admin");
if(aduty[playerid]==0)
{
format(astr,sizeof(astr),"Admin %s hat sich zum Support angemeldet.",SpielerName(playerid));
SendClientMessageToAll(BLAU,astr);
aduty[playerid] = 1;
}
else if(aduty[playerid]==1)
{
format(astr1,sizeof(astr1),"Admin %s hat sich vom Support abgemeldet.",SpielerName(playerid));
SendClientMessageToAll(BLAU,astr1);
aduty[playerid] = 0;
}
return true;
}

deine returns...
sollte so funktionieren

Alles anzeigen

Wie soll das denn funktionieren?!

//Hier eine resourcensparerende Methode:

//Oben bei deinen Globalen Variablen
new bool:AdminDuty[MAX_PLAYERS char]; // Hier kommen die { } Variablen!

//Der Befehl:
ocmd:aduty(playerid,params[])
{
if(!IstSpielerAdmin(playerid,1)) return SendClientMessage(playerid,ROT,"Du bist kein Admin");
new string[65]; // Wir brauchen nur 65, wenn man den Text zählt ;)
if(!AdminDuty{playerid})
{
format(string,sizeof(string),"Admin %s hat sich zum Support gemeldet.",SpielerName(playerid));
SendClientMessageToAll(BLAU,string);
AdminDuty{playerid} = true;
return 1;
}
format(string,sizeof(string),"Admin %s hat sich vom Support abgemeldet.",SpielerName(playerid));
SendClientMessageToAll(BLAU,string);
AdminDuty{playerid} = false;
return 1;
}

Das ist resourcensparend wegen des Char-Arrays und auch Zeilensparender.
Geht natürlich noch kürzer, verliert dann meiner meinung nach an übersichtlichkeit.

MfG