Adminchat

ocmd:a(playerid,params[])
{
new inhalt[140];
if(sscanf(params,"s",inhalt)) return SCM(playerid,FARBE,"Nutze: /a [TEXT]");
format(inhalt,140,"[%s %s: %s]",AdminRankName(playerid),SpielerName(playerid),inhalt);
for(new i = 0; i < MAX_PLAYERS; i++)
{
if(IsPlayerConnected(i))
if(!IstSpielerAdmin(i,1))return SCM(i,FARBE,"Du bist kein Admin.");
{
SendClientMessage(i,FARBE,inhalt);
}
}
return 1;
}

So sollte es eigentlich funktionieren, "FARBE" durch deine Farbe ersetzen und gut ist.

Zitat von Mr.DubStep

So sollte es eigentlich funktionieren, "FARBE" durch deine Farbe ersetzen und gut ist.

Nein, das ist falsch !

Mache es so:

ocmd:a(playerid,params[])
{
new inhalt[140];
if(sscanf(params,"s",inhalt)) return SCM(playerid,FARBE,"Nutze: /a [TEXT]");
format(inhalt,140,"[%s %s: %s]",AdminRankName(playerid),SpielerName(playerid),inhalt);
for(new i = 0; i < MAX_PLAYERS; i++)
{
if(!IsPlayerConnected(i)) continue;
if(!IstSpielerAdmin(i,1))return SCM(i,FARBE,"Du bist kein Admin.");
SendClientMessage(i,FARBE,inhalt);
}
return 1;
}

mfg.

Zitat von Kaliber

continue;

Warum sollte das bei so einer Abfrage von Nöten sein?

Zitat von Mr.DubStep

Warum sollte das bei so einer Abfrage von Nöten sein?

Naja, ganz einfach, weil es eine for-Schleife ist, und diese fängt bei 0 an und endet bei MAX_PLAYERS und wenn ein Spieler nicht "connected" ist, springt er direkt auf den nächsten Spieler

Das ist ressourcen freundlicher

mfg.

Ersetze SCM durch SendClientMessage
und mit dem AdminRankName frag mal Mr.Dubstep

ocmd:a(playerid,params[])
{
if(!adminabfrage)return SendClientMessage(playerid,Rot,"Du bist kein Admin.");
new text[128], string[128],name[MAX_PLAYER_NAME];
if(sscanf(params,"s",text))return SendClientMessage(playerid,Rot,"Verwendung: /a [Text]");
GetPlayerName(playerid,name,MAX_PLAYER_NAME);
format(string,128,"%s: %s",name,text);
for(new i=0; i<MAX_PLAYERS; i++)
{
if(adminafrage(i,1))
{
SendClientMessage(i,Gelb,string);
}
}
return 1;
}
So sieht mein Code aus. Natürlich musst du alles anpassen.

Dann nimm das einfach raus.

//e:
Ansonsten könntest du es aber auch so machen:

stock AdminRankName(playerid)
{
new aname[24];
if(!IsPlayerAdmin(playerid))
{
switch(GetPVarInt(playerid,"Adminlevel"))
{
case 0:{aname="Kein Admin";}
case 1:{aname="Rank 1";}
case 2:{aname="Rank 2";}
case 3:{aname="Rank 3";}
case 4:{aname="Rank 4";}
case 5:{aname="Rank 5";}
}
}else{aname="RCON-Admin";}
return aname;
}