/admins (schwieriger als gedacht)

ocmd:admins(playerid,params[])
{
new text[128];
for(new i=0; i<GetMaxPlayers(); i++)
{
if(SpielerInfo[i][Adminlevel] == 0)continue;
format(text,sizeof(text),"%s",pName(i));
}
ShowPlayerDialog(playerid,DIALOG_SHOWADMINS,DIALOG_STYLE_MSGBOX,"Alle verfügbaren Admins",text,"Fertig","");
return 1;
}

if(SpielerInfo[i][Adminlevel] == 0)continue;
müsste meiner Meinung nach auch so aussehen:

if(SpielerInfo[i][Adminlevel] > 0)continue;

Verbessert mich, wenn ich falsch liege.

Zitat von 123marvin123

ocmd:admins(playerid,params[])
{
new text[128];
for(new i=0; i<GetMaxPlayers(); i++)
{
if(SpielerInfo[i][Adminlevel] == 0)continue;
format(text,sizeof(text),"%s",pName(i));
}
ShowPlayerDialog(playerid,DIALOG_SHOWADMINS,DIALOG_STYLE_MSGBOX,"Alle verfügbaren Admins",text,"Fertig","");
return 1;
}

Alles anzeigen

Das ist sowieso falsch...
So sollte es funktionieren, sofern ich es richtig verstanden habe, dass alle Admins in einem Dialog angezeigt werden sollen.

ocmd:admins(playerid,params[])
{
new text[128];
for(new i=0; i<GetMaxPlayers(); i++)
{
if(SpielerInfo[i][Adminlevel] >= 1){
format(text,sizeof(text),"%s\n%s",text, name); //Name anpassen
}

}
ShowPlayerDialog(playerid,DIALOG_SHOWADMINS,DIALOG_STYLE_MSGBOX,"Alle verfügbaren Admins\n%s",text,"Fertig","");
return 1;
}

if(SpielerInfo[i][Adminlevel] == 0)continue;

wenn adminlevel == 0 dann überspring die id und such solange weiter bis einer ein höres adminlevel hat.

Zitat von Panzerkampfwagen Tiger

format(text,sizeof(text),"%s\n%s",text, name); //Name anpassen

Wieso ist das erste %s text, text wird doch dort definiert?
pName(i) wäre es dann.

SoR Mirco, okay danke, also könnte ich diese auch verwenden gut zu wissen, muss es aber eine Abfrage sein und nicht im pName?

Definiert hast du es schon vorher per new in der Zeile:
new text[128];
Also kann man es da auch verwenden.
Es wird ja nichts Anderes getan, als die Variable "text" nur um einen Adminname + voranstehendem Zeilenumbruch zu erweitern.

ocmd:admins(playerid,params[])
{
new text[128];
for(new i=0; i<GetMaxPlayers(); i++)
{
if(SpielerInfo[i][Adminlevel] >= 1){
format(text,sizeof(text),"%s\n%s ID: %d",text, pName(i),i);
}

}
ShowPlayerDialog(playerid,DIALOG_SHOWADMINS,DIALOG_STYLE_MSGBOX,"Alle verfügbaren Admins\n%s",text,"Fertig","");
return 1;
}

Zitat von Panzerkampfwagen Tiger

Also wird mit text für %s mehr Namen anzeigen möglich, habe ich das richtig verstanden?
Und wie sieht es mit meiner Möglichkeit aus hinter den Namen die ID von denjenigen noch anzeigen zu lassen?

Sagen wir es mal so:

Es gibt 5 Spieler.
Spieler1, Spieler2, usw.
Spieler 1,2 und 4 sind Admins.

Durchlauf 1: text = Spieler1
Durchlauf 2: text = Spieler1\nSpieler2
Durchlauf 2: Schleife wird hier neu fortgesetzt, da kein Admin.
Durchlauf 4: text = Spieler1\nSpieler2\nSpieler4
Durchlauf 5: Siehe Durchlauf 2

Am Ende dann:
text = Spieler1\nSpieler2\nSpieler4

Du hast die Adminabfrage in der Schleife vergessen

Außerdem:

format(text,sizeof(text),"%s\n%s ID -> %i",text,pName(SpielerInfo[i][Adminlevel]>0),i);

durch

format(text,sizeof(text),"%s\n%s ID -> %i",text,pName(i),i);
ersetzen.

Das >0 entfernen und nicht Adminlevel mit ID vertauschen.

Und das GetMaxPlayers() passt doch auch nicht, oder? Weil wenn 500 Leute drauf sind und dann die IDs von 100-300 z.B. offline gehen, dann werden ja nur die IDs von 0-300 Überprüft, aber die von User mit 300-500 werden ausgelassen, oder irre ich mich?

Zitat von Überg'schnüzz'lt

passt doch auch nicht, oder? Weil wenn 500 Leute drauf sind und dann die IDs von 100-300 z.B. offline gehen, dann werden ja nur die IDs von 0-300 Überprüft, aber die von User mit 300-500 werden ausgelassen, oder irre ich mich?

Du irrst dich.

GetMaxPlayers() gibt dir die Anzahl der eingestellten Server Slots aus der server.cfg wieder

Sprich wie viele Leute maximal auf den Server joinen könen sieht immer so aus (0/100...0/200) << das ist das, dieses Maximum

mfg.

Okay, danke... Sorry Dachte, dass das die Anzahl der connecteten Spieler angibt
Naja, würde trotzdem auf ein Define zurückgreifen und nicht auf die Funktion

Zitat von Überg'schnüzz'lt

Naja, würde trotzdem auf ein Define zurückgreifen und nicht auf die Funktion

Er macht es ja so:

Zitat von Frosty

for(new i=0; i<GetMaxPlayers(); i++)

Das ist natürlich sehr schlecht. So wird nämlich immer die Funktion aufgerufen und somit dauert das wesentlich länger als das Define.

Was aber zeitgleich (fast bis auf ein paar hunderstel Nanosekunden) gleich ist, ist diese Schleife:

for(new i,l=GetMaxPlayers(); i<l; i++)

Dann würde man die Funktion gut nutzen und zeitlich gesehen, ist es genauso schnell

mfg.