/admins buggt

"else if" - du tust bei jedem das "else" weg.

Ich gehe mal davon aus, dass dein IstSpielerAdmin überprüft, ob der Spieler den Rank hat, oder drüber ist.
Wenn das der Fall ist, ist das das Problem.

Du könntest die Abfragen tauschen, dass du zu erst die hohen Level abfragst, und dann die niedrigeren.

Gruß,
Neo.

Das liegt dann daran, dass du das else weg gemacht hast.
Es muss else if heißen, da sonst alle IF-Abfragen durchgegangen werden.

Gruß,
Neo.

€dit:

Hier noch eine Variante, die ich persönlich besser finde.

du solltest das in ner andren Reihenfolge machen,
wenn du als erstes admin rang 7 und als letztes adminrang 1 nimmst, sollte es besser funktionieren.

Zitat von Sry4pwn

du solltest das in ner andren Reihenfolge machen,
wenn du als erstes admin rang 7 und als letztes adminrang 1 nimmst, sollte es besser funktionieren.

Wo hast du das gehört?

Zitat von jeffrey blair

Wo hast du das gehört?

Er bezog das auf den Hauptpost. (Denke ich, da es nur da Sinn macht)

Wenn er zuerst abfragt, ob der Spieler Admin >= 7 ist, und danach die darunter, wird das höchste Admin-Level von dem Spieler ausgegeben.

So siehts aus

ocmd:admins(playerid,params[])
{
new text[256];
new string[500],name[MAX_PLAYER_NAME],var,level;
for(new i=0;i<MAX_PLAYERS;i++)
{
if(IsPlayerConnected(i) && isPlayerAnAdmin(i,1))
{
if(GetPVarInt(i,"Adminlevel")==1) {text = "Moderator"; }
if(GetPVarInt(i,"Adminlevel")==2) {text = "Lite Admin"; }
if(GetPVarInt(i,"Adminlevel")==3) {text = "Admin"; }
if(GetPVarInt(i,"Adminlevel")==4) {text = "Full Admin"; }
if(GetPVarInt(i,"Adminlevel")==5) {text = "Server Leader"; }
format(string, sizeof(string), "%s{F5FF00}%s (ID: %d) - {2DFF00}Level %d {F00000}(%s)\n",string,PlayerName(i),i,GetPVarInt(i,"Adminlevel"),text);
var++;
}
}
if(!var)
{
ShowPlayerDialog(playerid,0,DIALOG_STYLE_MSGBOX,"{F00000}Admins","{F00000}Zur Zeit sind keine Admins online!","Ok","");
}else
{
ShowPlayerDialog(playerid,0,DIALOG_STYLE_MSGBOX,"{F00000}Admins",string,"Ok","");
}
return 1;
}

Probier' das mal, das musst Du natürlich auf Dein Script anpassen.

ocmd:admins(playerid,params[])
{
new clas[128];
SendClientMessage(playerid,BLAU,"Admins Online:");
for(new i = 0; i < MAX_PLAYERS; i++)
{
if(IstSpielerAdmin(i,7))
format(clas,sizeof(clas),"Projektleitung: %s",SpielerName(i)),SendClientMessage(playerid,weis,clas);
else if(IstSpielerAdmin(i,6))
format(clas,sizeof(clas),"stelv.Projektleitung: %s",SpielerName(i)),SendClientMessage(playerid,weis,clas);
else if(IstSpielerAdmin(i,5))
format(clas,sizeof(clas),"Admin: %s",SpielerName(i)),SendClientMessage(playerid,weis,clas);
else if(IstSpielerAdmin(i,4))
format(clas,sizeof(clas),"Super-Moderator: %s",SpielerName(i)),SendClientMessage(playerid,weis,clas);
else if(IstSpielerAdmin(i,3))
format(clas,sizeof(clas),"Moderator: %s",SpielerName(i)),SendClientMessage(playerid,weis,clas);
else if(IstSpielerAdmin(i,2))
format(clas,sizeof(clas),"Supporter: %s",SpielerName(i)),SendClientMessage(playerid,weis,clas);
else if(IstSpielerAdmin(i,1))
format(clas,sizeof(clas),"Probe-Supporter: %s",SpielerName(i)),SendClientMessage(playerid,weis,clas);
}
return 1;
}