Zu viele SendClientMessage

Zitat von Giovanni_LopeZ

ocmd:givecoinsall(playerid,params[])
{
for(new i = 0; i <MAX_PLAYERS; i++)
{
if(!IsPlayerConnected(i)) continue;
{
new coins,str[64];
if(!isPlayerAnAdmin(playerid,5)) return SendClientMessage(playerid,GRAU,"Du bist kein Admin!");
if(sscanf(params,"d",coins)) return SendClientMessage(playerid,GRAU,"Benutze: /givecoinsall [Anzahl]");
format(str,sizeof(str),"AdmCmd: Admin %s hat allen Spielern %d Coins gutgeschrieben!",SpielerName(playerid),coins);
SendClientMessageToAll(ROT,str);
SpielerInfo[pCoins] = coins;
SavePlayer(i);
}
}
return 1;
}

Wenn ich diesen Befehle eingebe kommt 10x die SendToAll Message? wieso und keiner bekommt coins.

Alles anzeigen

Man formatiert Strings nicht in einer Schleife...
//Edit:

SpielerInfo [i] [pCoins] = coins; (i) davor

Versuch es mal in der If Abfrage IsPlayerConnected(i)) ein return 1; reinmachen. Ich denke das es weiterhilft, wenn nicht, dann weiß ich es auch nicht

Ich hoffe Ich konnte Dir helfen.

Mit Freundlichen Grüßen,

TheBester

Zitat

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

SendClientMessageToAll(ROT,str);
Du hast die Message in der Schleife drinnen die durch alle Spieler geht.
Mach die auserhalb

ocmd:givecoinsall(playerid,params[])
{
for(new i = 0; i <MAX_PLAYERS; i++)
{
if(!IsPlayerConnected(i)) continue;
{
new coins,str[64];
if(!isPlayerAnAdmin(playerid,5)) return SendClientMessage(playerid,GRAU,"Du bist kein Admin!");
if(sscanf(params,"d",coins)) return SendClientMessage(playerid,GRAU,"Benutze: /givecoinsall [Anzahl]");
SpielerInfo[pCoins] = coins;
SavePlayer(i);
}
} format(str,sizeof(str),"AdmCmd: Admin %s hat allen Spielern %d Coins gutgeschrieben!",SpielerName(playerid),coins);
SendClientMessageToAll(ROT,str);
return 1;
}

So sollte es eig. gehen

Zitat von TheBester

Versuch es mal in der If Abfrage IsPlayerConnected(i)) ein return 1; reinmachen. Ich denke das es weiterhilft, wenn nicht, dann weiß ich es auch nicht

Völliger Quatsch, continue; ist schon richtig.
ocmd:givecoinsall(playerid,params[])
{
new coins,str[64];
if(!isPlayerAnAdmin(playerid,5)) return SendClientMessage(playerid,GRAU,"Du bist kein Admin!");
if(sscanf(params,"d",coins)) return SendClientMessage(playerid,GRAU,"Benutze: /givecoinsall [Anzahl]");
for(new i = 0; i <MAX_PLAYERS; i++)
{
if(!IsPlayerConnected(i)) continue;
SpielerInfo[i][pCoins] += coins;
SavePlayer(i);
}
format(str,sizeof(str),"AdmCmd: Admin %s hat allen Spielern %d Coins gutgeschrieben!",SpielerName(playerid),coins);
SendClientMessageToAll(ROT,str);
return 1;
}

Die Abfragen müssen aber nicht mit jedem Schleifendurchlauf wieder durchgeführt werden...
ocmd:givecoinsall(playerid,params[])
{
if(!isPlayerAnAdmin(playerid,5)) return SendClientMessage(playerid,GRAU,"Du bist kein Admin!");
new coins;
if(sscanf(params,"d",coins)) return SendClientMessage(playerid,GRAU,"Benutze: /givecoinsall [Anzahl]");
for(new i = 0; i <MAX_PLAYERS; i++)
{
if(!IsPlayerConnected(i)) continue;
SpielerInfo[pCoins] += coins;
SavePlayer(i);
}
new str[64];
format(str,sizeof(str),"AdmCmd: Admin %s hat allen Spielern %d Coins gutgeschrieben!",SpielerName(playerid),coins);
SendClientMessageToAll(ROT,str);
return 1;
}
/Edit da war wohl Jemand schneller
Allerdings würde ich bei "SpielerInfo[pCoins] = coins;" ein += setzen, da ja vllt manche Spieler mehr Coins habe & danach dementsprechend weniger hätten

Würde es eher so machen...

Müsste auch funktionieren...

ocmd:givecoinsall(playerid,params[])
{
if(!isPlayerAnAdmin(playerid,5)) return SendClientMessage(playerid,GRAU,"Du bist kein Admin!");
new coins,str[64];
if(sscanf(params,"d",coins)) return SendClientMessage(playerid,GRAU,"Benutze: /givecoinsall [Anzahl]");
for(new i = 0; i <MAX_PLAYERS; i++){
if(IsPlayerConnected(i)){
SpielerInfo[i][pCoins] = coins;
SavePlayer(i);
}
}
format(str,64,"AdmCmd: Admin %s hat allen Spielern %d Coins gutgeschrieben!",SpielerName(playerid),coins);
SendClientMessageToAll(ROT,str);
return 1;
}