Der Befehl /aduty funktioniert nicht richtig.

Zitat von Sonderzeichen

Hallo,
ich habe in meinem Selfmade einen /aduty Befehl eingebaut. Jedoch funktioniert der nicht richtig.
Es wird immer angezeigt, dass der Spieler sich als Administrator angemeldet, jedoch nicht abgemeldet hat.

ocmd:aduty(playerid, params[])
{
if (SpielerInfo[playerid][pAdminlevel] > 0)
{
new isduty[MAX_PLAYERS] = 0;
new username[128];
GetPlayerName(playerid, username, sizeof(username));
if(isduty[playerid] == 0)
{
new string[128];
format(string, sizeof(string), ""COLOR_WHITE"» "COLOR_RED"%s hat sich als Administrator angemeldet.", username);
SendClientMessageToAll(0xFFFFFFFF, string);
isduty[playerid] = 1;
}
else
{
new string[128];
format(string, sizeof(string), ""COLOR_WHITE"» "COLOR_RED"%s hat sich als Administrator abgemeldet.", username);
SendClientMessageToAll(0xFFFFFFFF, string);
isduty[playerid] = 0;
}
}
else
{
SendClientMessage(playerid, 0xFFFFFFFF, ADM_CMDERR);
}
return 1;
}

Lg Sonderzeichen

Alles anzeigen

Mach anstatt eine einfach else eine else if z.b else if(isduty[playerid] = 1)

ocmd:aduty(playerid, params[])
{
if (SpielerInfo[playerid][pAdminlevel] > 0)
{
new isduty[MAX_PLAYERS] = 0;
new username[128];
GetPlayerName(playerid, username, sizeof(username));
if(isduty[playerid] == 0)
{
new string[128];
format(string, sizeof(string), ""COLOR_WHITE"» "COLOR_RED"%s hat sich als Administrator angemeldet.", username);
SendClientMessageToAll(0xFFFFFFFF, string);
isduty[playerid] = 1;
}
if(isduty[playerid] = 1;
{
new string[128];
format(string, sizeof(string), ""COLOR_WHITE"» "COLOR_RED"%s hat sich als Administrator abgemeldet.", username);
SendClientMessageToAll(0xFFFFFFFF, string);
isduty[playerid] = 0;
}
}
else
{
SendClientMessage(playerid, 0xFFFFFFFF, ADM_CMDERR);
}
return 1;
}

Dann auch richtig:

else if(isduty[playerid] == 1)

Edit:

if(isduty[playerid] == 0)
{
new string[128];
format(string, sizeof(string), ""COLOR_WHITE"» "COLOR_RED"%s hat sich als Administrator angemeldet.", username);
SendClientMessageToAll(0xFFFFFFFF, string);
isduty[playerid] = 1;
}
else if(isduty[playerid] == 1;
{
new string[128];
format(string, sizeof(string), ""COLOR_WHITE"» "COLOR_RED"%s hat sich als Administrator abgemeldet.", username);
SendClientMessageToAll(0xFFFFFFFF, string);
isduty[playerid] = 0;
}

Wenn es so nicht geht, dann weiß ich auch nicht :o

Die Variable new isduty[MAX_PLAYERS] = 0; muss ausserhalb des Befehls erstellt werden, damit eine globale Wirkung erzielt wird. Diese Zeile irgendwo im Gamemode oben einfügen.