max players ändern
mach nach ganz oben
#undef MAX_PLAYERS
#define MAX_PLAYERS 200
Es geht viel einfacher, du schreibst einfach eine Funktion die zählt wieviele Spieler on sind und diese returned.
Versuchs so:
stock GetOnlinePlayers()
{
new alle;
for(new count=0;count<=MAX_PLAYERS;count++)
{
if(IsPlayerConnected(i))
{
alle++;
}
}
return alle;
}
Und dann einfach das:
for(new i = 0; i < GetOnlinePlayers(); i++){
//Blabla
Um Himmels Willen.
Wenn IDs 0,1,2,3,4 und 7 on sind, geht die Schleife nur bis 5 und ID 7 hat Pech gehabt.
Immer ein "IsPlayerConnected" als erste Abfragen oder alternativ Y_Less' foreach benutzen, dann wird das schon.
dann könnte er es viel einfacher machen und einfach abfragen ob er online ist.
for(new i = 0; i < MAX_PLAYERS; i++)
{
if(IsPlayerConnected(i))
Wäre sinnvoller.. Aber er möchte glaube ich, das nicht mehr gemacht wird als nötig.
Bei deiner variante würde er ja 1 schleife komplett durchgehen und dann eine 2. Da könnte er genauso gut eine schleife komplett durchgehen und abfragen ob der spieler connected ist.
Würde es auch mit einer variablen machen.
Noch kürzer:
for(new i;i<MAX_PLAYERS&&IsPlayerConnected(i);i++)
so sehn bei mir quasi alle Schleifen aus
Die Schnellste Konventionelle Methode ist immernoch die hier:
for ( new slots = GetMaxPlayers( ), i; i < slots; i++ )
{
if ( !IsPlayerConnected( i ) ) continue;
//dein Code
}
Ansonsten solltest du mal darüber nachdenken auf foreach umzusteigen bei Spieler schleifen.
wozu getmayplayers? kann er etwa max_players nicht so definieren wv er aufn server lassen will?
oder mach getmaxplayers bei ongamemodeinit, aber nicht in jeder schleife...
Noch kürzer:
for(new i;i<MAX_PLAYERS&&IsPlayerConnected(i);i++)
so sehn bei mir quasi alle Schleifen aus
Dann funktionieren quasi alle deine Schleifen nicht richtig.
Nehmen wir an,wir haben Spieler mit den IDs:
Zitat0, 1 , 5 , 6
Deine Schleife beginnt.
0 < MAX_PLAYERS && IsPlayerConnected(0) - wahr / true
1 < MAX_PLAYERS && IsPlayerConnected(1) - wahr / true
2 < MAX_PLAYERS && IsPlayerConnected(2) - falsch / false
Schleife wird beendet.
ID 5 und 6 werden niemals erreicht.
Denn die Schleife läuft nur so lange,wie die Bedingung wahr ist.Sobald eine ID unbenutzte haben, wird die Schleife beendet. Egal ob danach noch irgendeine belegte pielerID kommt oder nicht.
Von daher,keine Gute Idee so die Schleifen zu machen.
Das mit dem GetMaxPlayers( ); sehe ich genau so als Schwachsinn an, da bei jeder Schleife er erneut auslesen muss, wie viele Spieler maximal auf den Server kommen können.
Bester Weg:
#undef MAX_PLAYERS
#define MAX_PLAYERS 32
Zu deiner Frage:
Also ich sehe es nicht als Problem an, es können 10 Spieler oder auch 100 Spieler sein und die Funktion würde funktionieren...
Deine Schleife ist schon richtig. Du musst jedoch nur noch eine if-Abfrage mit IsPlayerConnected(i) unterbringen..
Alles anzeigen
[...]
Deine Schleife beginnt.
0 < MAX_PLAYERS && IsPlayerConnected(0) - wahr / true
1 < MAX_PLAYERS && IsPlayerConnected(1) - wahr / true
2 < MAX_PLAYERS && IsPlayerConnected(2) - falsch / false
Schleife wird beendet.
ID 5 und 6 werden niemals erreicht.
Denn die Schleife läuft nur so lange,wie die Bedingung wahr ist.Sobald eine ID unbenutzte haben, wird die Schleife beendet. Egal ob danach noch irgendeine belegte pielerID kommt oder nicht.
Von daher,keine Gute Idee so die Schleifen zu machen.
Gut, das ist natürlich ein ziemlich mieser Logikfehler, obwohl ich sogar meine, dass DMA die Schleife in dieser Form vor etlicher Zeit mal gepostet hätte 
Was du dort hast,sieht nämlich so aus:
for(new i = 0 ; i < MAX_PLAYERS && IsPlayerConnected(i) ; i++) {
// Code
}
// ==
for(new i = 0 ; i < MAX_PLAYERS ; i++) {
if( !IsPlayerConnected(i) ) break;
// Code
}
Das es mit diesem break nicht sinnvoll ist,sollte klar sein :p.
Um nochmal auf den Sinn des Threads zu kommen.
for(new i = 0 ; i < MAX_PLAYERS ; i++) if(IsPlayerConnected(i) ) {
}
#define foreach(%1) \
for(new %1 = 0; %1 < MAX_PLAYERS ; %1++) if( IsPlayerConnected(%1) )
// Nutzung:
foreach(i) {
printf("ID's Online: %d",i);
}
Kannst also mit diesem define arbeiten.Das ist nicht das foreach, von dem hier gesprochen wurde!
Sie meinten nämlich folgendes : http://forum.sa-mp.com/showthread.php?t=92679
//Edit: Fehler :-0
Im Define von Goldkiller ist auch ein Fehler, dort fehlt eine Klammer.
Setz einfach ganz am schluss noch eine Klammer ran
#define foreach(%1) \
for(new %1 = 0; %1 < MAX_PLAYERS ; %1++) if(IsPlayerConnected(%1))
Hast recht, hab es falsch aus meinem Code kopiert:
// gAdmin.pwn
#define foreach(%1) \
for(new %1;%1<g_Max_Players;%1++) if(IsPlayerConnected(%1) && !IsPlayerNPC(%1) )
Hab es nämlich bei mir so, wohl falsch umgeschrieben 
.
Es fehlte hinten die Klammer ) , die die Bedingung schliet.
#define foreach(%1) \
for(new %1 = 0; %1 < MAX_PLAYERS ; %1++) if( IsPlayerConnected(%1) )
