Geld an alle User verteilen

Mach ne schleife die alle Spieler durchgeht und verteil das Geld?
Ist doch wirklich nicht schwer O.o

ocmd:givemoneyall(playerid,params[])
{
new anzahl;
if(adminabfrage)return SendClientMessage(playerid,Rot,"Kein Admin");
if(sscanf(params,"d",anzahl))return SendClientMessage(playerid,Gelb,"Verwendung: /givemoneyall anzahl");
for(new i=0; i<MAX_PLAYERS; i++)
{
if(IsPlayerConnected(i))
{
GivePlayerMoney(playerid,anzahl);
}
}
return 1;
}

ocmd:givemoneyall(playerid,params[])
{
//if(adminabfrage)
{
new betrag,string[128];
if(sscanf(params,"i",betrag))return SendClientMessage(playerid,0xFF0000FF,"Fehler: /givemoneyall [Betrag]");
{
for (new i=0;i<MAX_PLAYERS;i++)
{
format(string,sizeof(string),"Admin %s gibt jedem Spieler %i€!",PlayerName(playerid),betrag);
SendClientMessageToAll(0xFF0000FF,string);
if(IsPlayerConnected(i))
{
GivePlayerMoney(i,betrag);

}
}
}
}
return 1;
}

stock PlayerName(playerid)
{
new name[MAX_PLAYER_NAME];
GetPlayerName(playerid, name, MAX_PLAYER_NAME);
return name;
}

Zitat von Mr.Cro997

Danke, funktioniert bestens, aber eine Frage.
Wenn ich dann allen Usern Geld gebe wird der ganze Chat zugespammt, kann man auch machen, dass die Nachricht im Chat nur einmal kommt.
Sorry das ich es nicht selber machen kann, bin nicht so gut mit Schleifen und allgemein im Scripten mit solchen Sachen...

Oh sry

ocmd:givemoneyall(playerid,params[])
{
//if(adminabfrage)
{
new betrag,string[128];
if(sscanf(params,"i",betrag))return SendClientMessage(playerid,0xFF0000FF,"Fehler: /givemoneyall [Betrag]");
{
format(string,sizeof(string),"Admin %s gibt jedem Spieler %i€!",PlayerName(playerid),betrag);
SendClientMessageToAll(0xFF0000FF,string);
for (new i=0;i<MAX_PLAYERS;i++)
{
if(IsPlayerConnected(i))
{
GivePlayerMoney(i,betrag);

}
}
}
}
return 1;
}

so sollte es eigentlich gehen

mach das
format(string,sizeof(string),"Admin %s gibt jedem Spieler %i€!",PlayerName(playerid),betrag);
SendClientMessageToAll(0xFF0000FF,string);
einfach vor das
for(new i; i<MAX_PLAYERS; i++)

ocmd:givemoneyall(playerid,params[])
{
//if(adminabfrage)
{
new betrag,string[128];
if(sscanf(params,"i",betrag))return SendClientMessage(playerid,0xFF0000FF,"Fehler: /givemoneyall [Betrag]");
{
format(string,sizeof(string),"Admin %s gibt jedem Spieler %i€!",PlayerName(playerid),betrag);
SendClientMessageToAll(0xFF0000FF,string);
for (new i=0;i<MAX_PLAYERS;i++)
{
if(IsPlayerConnected(i))
{
GivePlayerMoney(i,betrag);

}
}
}
}
return 1;
}

stock PlayerName(playerid)
{
new name[MAX_PLAYER_NAME];
GetPlayerName(playerid, name, MAX_PLAYER_NAME);
return name;
}

einfach das format und SendClient... ausserhalb der Schleife lassen

ocmd:givemoneyall(playerid,params[]){
//if(adminabfrage){
new betrag,string[128], i;
if(sscanf(params,"i",betrag))return SendClientMessage(playerid,0xFF0000FF,"Fehler: /givemoneyall [Betrag]");{
for(;i<MAX_PLAYERS;i++){
if(IsPlayerConnected(i)){
GivePlayerMoney(i,betrag);
}
}
format(string, sizeof(string), "* Administrator %s gab allen Spielern $%d.", PlayerName(playerid), betrag);
SendClientMessageToAll(-1, string);
}
}
return 1;
}