Problem mit Ein/Auslogg nachricht

Guten Tag ich möchte machen das wenn man in den Einstellungen die Ein/Auslogg angeschaltet das man die nachricht bekommt nun ist es so das wenn 2 Spieler die Nachricht anhaben es auch 2 Mal angezeigt wird.

OnPlayerConnect:
for(new i=0;i<MAX_PLAYERS; i++) {
new text[100];
if(IsPlayerConnected(i)) {
if (PlayerInfo[i][togeinlogg] == 1) {
format(text,sizeof(text),"%s [ID: %d] hat den Server betreten",PlayerName(playerid),playerid);
Einlogtext(COLOR_RED,text);
}
}
}

OnPlayerDisconnect
for(new i=0;i<MAX_PLAYERS; i++)
{
new text[100],LocationZone[MAX_ZONE_NAME];
GetPlayer2DZone(playerid, LocationZone, MAX_ZONE_NAME);
if(IsPlayerConnected(i))
{
if (PlayerInfo[i][togeinlogg] == 1)
{
format(text,sizeof(text),"%s [ID: %d] hat den Server in %s verlassen",PlayerName(playerid),playerid,LocationZone);
Einlogtext(COLOR_RED,text);
}
}
}

forward Einlogtext(color,const string[]);
public Einlogtext(color,const string[])
{
for(new i = 0; i < MAX_PLAYERS; i++)
{
if(IsPlayerConnected(i))
{
if (PlayerInfo[i][togeinlogg] != 0)
{
SendClientMessage(i, color, string);
}
}
}
return 1;
}

Zitat von Kaliber

Viel zu viele Schleifen überall

Und du solltest keine Variablen in Schleifen deklarieren

Hier, mal etwas effizienter:

C

//OnPlayerConnect
new text[MAX_PLAYER_NAME + 35];
format(text,sizeof(text),"%s [ID: %d] hat den Server betreten",PlayerName(playerid),playerid);
for(new i; i<MAX_PLAYERS; i++)
{
    if(IsPlayerConnected(i) && PlayerInfo[i][togeinlogg] == 1) SendClientMessage(i,COLOR_RED,text);
}

C

//OnPlayerDisconnect
new text[MAX_PLAYER_NAME + MAX_ZONE_NAME + 35];
GetPlayer2DZone(playerid, text, MAX_ZONE_NAME);
format(text,sizeof(text),"%s [ID: %d] hat den Server in %s verlassen",PlayerName(playerid),playerid,text);
for(new i; i<MAX_PLAYERS; i++)
{
    if(IsPlayerConnected(i) && PlayerInfo[i][togeinlogg] == 1) SendClientMessage(i,COLOR_RED,text);
}

Alles anzeigen

Danki es geht