Ist OnPlayerUpdate zu fressend?

Rexo · 25. September 2015

Grüße!

Ich wollte ein Radarsystem bauen und das mit einem runden Umfang und einer Z Koordinate (also geht eine Gangzone schonmal nicht). Also habe ich mir gedacht das ich OnPlayerUpdate nutze, nachfrage ob man in der Nähe vom Radarpunkt ist und wenn ja einen Marker setzt. Wenn nicht setzt man den Marker aber auf unsichtbar. Wenn ich das jetzt mit mehreren Systemen mache bei mehreren Spielern, kommt mir die Frage auf ob das ein einfacher Server überhaupt schafft oder ob dann das Laggen losgeht.

Kennt sich jemand damit aus?

Jeffry · 25. September 2015

Sollte keine Probleme machen, sind ja nur ein paar if Abfragen.

DJ Deagle · 25. September 2015

Darf man fragen, warum du dafür nicht einen globalen Timer setzt, der dann durch alle Spieler läuft?
(Soll dir jetzt nicht die Option abschlagen, OnPlayerUpdate zu verwenden)

Da OnPlayerUpdate ca. 20x pro Sekunde aufgerufen wird.

Akino Kiritani · 25. September 2015

Zitat von DJ Deagle

Da OnPlayerUpdate ca. 20x pro Sekunde aufgerufen wird.

Selbst wenn das der Fall wäre, was es zwar nicht ist tut aber nichts zur Sache, kannst du ja messen wie lange die Ausführungszeit ist.

In welchen Zeitabständen der Spieler nun OnPlayerUpdate triggert in der Sekunde kann man ja gegen rechnen, so lange die Ausführungszeit mit der Zeit zwischen dem triggern des Callbacks sich nicht überschneidet ist alles im grünen Bereich.

**XeroX** · 25. September 2015

Wäre nicht CreateDynamicArea einfacher ?

Rexo · 25. September 2015

Werde ich mir mal anschauen, kenne ich noch gar nicht.

Jeffry · 26. September 2015

Zitat von BlackAce

Selbst wenn das der Fall wäre, was es zwar nicht ist tut aber nichts zur Sache, kannst du ja messen wie lange die Ausführungszeit ist.

DJ Deagle hat schon Recht, das wird so oft, bzw. noch öfter aufgerufen.

Rennend zu Fuß:

Code

[26/09/2015 12:31:32] OnPlayerUpdate called 50 times in 1 sec.
[26/09/2015 12:31:33] OnPlayerUpdate called 48 times in 1 sec.
[26/09/2015 12:31:34] OnPlayerUpdate called 53 times in 1 sec.
[26/09/2015 12:31:35] OnPlayerUpdate called 47 times in 1 sec.
[26/09/2015 12:31:36] OnPlayerUpdate called 51 times in 1 sec.
[26/09/2015 12:31:37] OnPlayerUpdate called 52 times in 1 sec.
[26/09/2015 12:31:38] OnPlayerUpdate called 46 times in 1 sec.
[26/09/2015 12:31:39] OnPlayerUpdate called 47 times in 1 sec.

Im Fahrzeug, fahrend:

Code

[26/09/2015 12:34:43] OnPlayerUpdate called 37 times in 1 sec.
[26/09/2015 12:34:44] OnPlayerUpdate called 35 times in 1 sec.
[26/09/2015 12:34:45] OnPlayerUpdate called 35 times in 1 sec.
[26/09/2015 12:34:46] OnPlayerUpdate called 34 times in 1 sec.
[26/09/2015 12:34:47] OnPlayerUpdate called 31 times in 1 sec.
[26/09/2015 12:34:48] OnPlayerUpdate called 36 times in 1 sec.

Stehend, ohne etwas zu machen:

Code

[26/09/2015 12:37:33] OnPlayerUpdate called 3 times in 1 sec.
[26/09/2015 12:37:34] OnPlayerUpdate called 1 times in 1 sec.
[26/09/2015 12:37:35] OnPlayerUpdate called 1 times in 1 sec.
[26/09/2015 12:37:36] OnPlayerUpdate called 2 times in 1 sec.
[26/09/2015 12:37:37] OnPlayerUpdate called 2 times in 1 sec.

Code dazu:

C

public OnPlayerUpdate(playerid)
{
	if(IsPlayerNPC(playerid)) return 1;
	static called[MAX_PLAYERS], count[MAX_PLAYERS];
	count[playerid]++;
	if(called[playerid]+1000 < GetTickCount())
	{
	    called[playerid] = GetTickCount();
	    printf("OnPlayerUpdate called %d times in 1 sec.", count[playerid]);
	    count[playerid] = 0;
	}
	return 1;
}

Alles anzeigen

Nimmt man einen Schnitt aus den ganzen Aktionen über einen längeren Zeitraum kommt man sicherlich auf 20-40x pro Sekunde, je nach dem was der Spieler macht. Steht er natürlich nur rum dann ist es wesentlich weniger.

Akino Kiritani · 26. September 2015

Zitat von Jeffry

DJ Deagle hat schon Recht, das wird so oft, bzw. noch häufiger aufgerufen.

Ich hab doch nie gesagt das das nicht der Fall ist, nur das seine Verallgemeinerung nicht stimmt, aus diesem Grund schrieb ich ja auch, das die Ausführungszeit nicht den durchschnittlichen Triggerzeitraum überschreiten sollte

Jeffry · 26. September 2015

Zitat von BlackAce

Ich hab doch nie gesagt das das nicht der Fall ist

hmm

Zitat von BlackAce

Selbst wenn das der Fall wäre, was es zwar nicht ist

Ist ja auch egal, du hast es wahrscheinlich anders gemeint, ging ja nur darum mal darzustellen, wie oft das wirklich aufgerufen wird, da viele das nicht wissen bzw. danach suchen/fragen, und ich es selbst auch nicht genau wusste, wie oft es wirklich ist.

Rexo · 1. Oktober 2015

Hab mich heute mal an das System gewagt und durfte feststellen das es bei OnPlayerUpdate zu viel war ([warning] client exceeded áckslimit´127..... <3006> Limit: 3000/sec).

Und bei CreateDynamicArea habe ich leider nichts gefunden.

Wenn ich das nun mit einen Timer mache den ich alle 3 sec abpsielen lasse, wäre das zu viel?
Und wenn nicht sollte ich einen Timer nehmen der für jeden Spieler einzelnd läuft (also Timer bei OnPlayerConnect) oder sollte ich das mit 2 Schleifen machen für die entsprechenden Playerid´s.

Es kann in dieser Abfrage zu 2 Abfragen außerhalb einer Schleife kommen und zu maximal 5 weiteren Abfragen in einer Schleife. Minimal sind es 2 außerhalb der Schleife und 3 innerhalb der Schleife.

Jeffry · 1. Oktober 2015

Poste doch mal den Code, den du da rein geschrieben hast, je nach dem kann man dir erst sagen, was besser ist.

Rexo · 1. Oktober 2015

Code

if(pInfo[playerid][pTeam] == 1)
{
GetVehiclePos(NibiruTransporter[0], NibiruTransporterPos[0],NibiruTransporterPos[1], NibiruTransporterPos[2]);
		SetPlayerMapIcon(playerid, 3, NibiruTransporterPos[0],NibiruTransporterPos[1],NibiruTransporterPos[2], 42, 0, MAPICON_GLOBAL);
	}
    if(pInfo[playerid][pTeam] == 2)
	{
		GetVehiclePos(SeparatistenTransporter[0], SeparatistenTransporterPos[0], SeparatistenTransporterPos[1], SeparatistenTransporterPos[2]);
		SetPlayerMapIcon(playerid, 3, SeparatistenTransporterPos[0],SeparatistenTransporterPos[1],SeparatistenTransporterPos[2], 42, 0, MAPICON_GLOBAL);
	}
	if(AmmuKiste3[playerid] != 1)
	{
		if(IsPlayerInRangeOfPoint(playerid, 700.0, 247.43230, 1255.15662, 646.68610))//Nibiru Antenne
		{
			for(new p;p<MAX_PLAYERS;p++)
  			{
			    if(NibiruAntenneHeil == 1)
			    {
		            if(pInfo[p][pTeam] == 1)
		            {
			            if(pInfo[playerid][pTeam] == 1){ SetPlayerMarkerForPlayer(p, playerid, 0x0000FFFF);}//Spieler wird angezeigt
						else if(pInfo[playerid][pTeam] == 2){ SetPlayerMarkerForPlayer(p, playerid, 0xFF0000FF);}//Spieler wird angezeigt
						else{ SetPlayerMarkerForPlayer(p, playerid, 0xFFFFFFFF);}//Spieler wird angezeigt
					}
		        }
				else
				{
					if(pInfo[playerid][pTeam] == 1){ SetPlayerMarkerForPlayer(playerid, p, 0x0000FFFF & 0xFFFFFF00);}//Spieler wird verdeckt
					else if(pInfo[playerid][pTeam] == 2){ SetPlayerMarkerForPlayer(playerid, p, 0xFF0000FF & 0xFFFFFF00);}//Spieler wird verdeckt
					else{ SetPlayerMarkerForPlayer(p, playerid, 0xFFFFFFFF & 0xFFFFFF00);}//Spieler wird verdeckt
				}
			}
	    }
	    else if(IsPlayerInRangeOfPoint(playerid, 700.0, -930.95032, 1900.95923, 620.97052))//Separatisten Antenne
	    {
		    for(new p;p<MAX_PLAYERS;p++)
  			{
			    if(SeparatistenAntenneHeil == 1)
			    {
		            if(pInfo[p][pTeam] == 2)
		            {
			            if(pInfo[playerid][pTeam] == 1){ SetPlayerMarkerForPlayer(p, playerid, 0x0000FFFF);}
						else if(pInfo[playerid][pTeam] == 2){ SetPlayerMarkerForPlayer(p, playerid, 0xFF0000FF);}
						else{ SetPlayerMarkerForPlayer(p, playerid, 0xFFFFFFFF);}
					}
		        }
				else
				{
					if(pInfo[playerid][pTeam] == 1){ SetPlayerMarkerForPlayer(p, playerid, 0x0000FFFF & 0xFFFFFF00);}
					else if(pInfo[playerid][pTeam] == 2){ SetPlayerMarkerForPlayer(p, playerid, 0xFF0000FF & 0xFFFFFF00);}
					else{ SetPlayerMarkerForPlayer(p, playerid, 0xFFFFFFFF & 0xFFFFFF00);}
				}
			}
	    }
	    else
	    {
	        for(new p;p<MAX_PLAYERS;p++)
	        {
				if(pInfo[playerid][pTeam] == 1){ SetPlayerMarkerForPlayer(p, playerid, 0x0000FFFF & 0xFFFFFF00);}
				else if(pInfo[playerid][pTeam] == 2){ SetPlayerMarkerForPlayer(p, playerid, 0xFF0000FF & 0xFFFFFF00);}
				else{ SetPlayerMarkerForPlayer(p, playerid, 0xFFFFFFFF & 0xFFFFFF00);}
	        }
		}
    }

Alles anzeigen

http://pastebin.com/pUyZd5CV

Jeffry · 1. Oktober 2015

Dieser Code an sich sollte auch bei OnPlayerUpdate eigentlich keinen Lag verursachen.
Versuche es aber mal so:

C

if(pInfo[playerid][pTeam] == 1)
{
	GetVehiclePos(NibiruTransporter[0], NibiruTransporterPos[0],NibiruTransporterPos[1], NibiruTransporterPos[2]);
	SetPlayerMapIcon(playerid, 3, NibiruTransporterPos[0],NibiruTransporterPos[1],NibiruTransporterPos[2], 42, 0, MAPICON_GLOBAL);
}
if(pInfo[playerid][pTeam] == 2)
{
	GetVehiclePos(SeparatistenTransporter[0], SeparatistenTransporterPos[0], SeparatistenTransporterPos[1], SeparatistenTransporterPos[2]);
	SetPlayerMapIcon(playerid, 3, SeparatistenTransporterPos[0],SeparatistenTransporterPos[1],SeparatistenTransporterPos[2], 42, 0, MAPICON_GLOBAL);
}
if(AmmuKiste3[playerid] != 1)
{
	if(IsPlayerInRangeOfPoint(playerid, 700.0, 247.43230, 1255.15662, 646.68610))//Nibiru Antenne
	{
		for(new p;p<MAX_PLAYERS;p++)
		{
			if(!IsPlayerConnected(p)) continue;
			if(NibiruAntenneHeil == 1)
			{
				if(pInfo[p][pTeam] == 1)
				{
					if(pInfo[playerid][pTeam] == 1){ SetPlayerMarkerForPlayer(p, playerid, 0x0000FFFF);}//Spieler wird angezeigt
					else if(pInfo[playerid][pTeam] == 2){ SetPlayerMarkerForPlayer(p, playerid, 0xFF0000FF);}//Spieler wird angezeigt
					else{ SetPlayerMarkerForPlayer(p, playerid, 0xFFFFFFFF);}//Spieler wird angezeigt
				}
			}
			else
			{
				if(pInfo[playerid][pTeam] == 1){ SetPlayerMarkerForPlayer(playerid, p, 0x0000FFFF & 0xFFFFFF00);}//Spieler wird verdeckt
				else if(pInfo[playerid][pTeam] == 2){ SetPlayerMarkerForPlayer(playerid, p, 0xFF0000FF & 0xFFFFFF00);}//Spieler wird verdeckt
				else{ SetPlayerMarkerForPlayer(p, playerid, 0xFFFFFFFF & 0xFFFFFF00);}//Spieler wird verdeckt
			}
		}
	}
	else if(IsPlayerInRangeOfPoint(playerid, 700.0, -930.95032, 1900.95923, 620.97052))//Separatisten Antenne
	{
		for(new p;p<MAX_PLAYERS;p++)
		{
			if(!IsPlayerConnected(p)) continue;
			if(SeparatistenAntenneHeil == 1)
			{
				if(pInfo[p][pTeam] == 2)
				{
					if(pInfo[playerid][pTeam] == 1){ SetPlayerMarkerForPlayer(p, playerid, 0x0000FFFF);}
					else if(pInfo[playerid][pTeam] == 2){ SetPlayerMarkerForPlayer(p, playerid, 0xFF0000FF);}
					else{ SetPlayerMarkerForPlayer(p, playerid, 0xFFFFFFFF);}
				}
			}
			else
			{
				if(pInfo[playerid][pTeam] == 1){ SetPlayerMarkerForPlayer(p, playerid, 0x0000FFFF & 0xFFFFFF00);}
				else if(pInfo[playerid][pTeam] == 2){ SetPlayerMarkerForPlayer(p, playerid, 0xFF0000FF & 0xFFFFFF00);}
				else{ SetPlayerMarkerForPlayer(p, playerid, 0xFFFFFFFF & 0xFFFFFF00);}
			}
		}
	}
	else
	{
		for(new p;p<MAX_PLAYERS;p++)
		{
			if(!IsPlayerConnected(p)) continue;
			if(pInfo[playerid][pTeam] == 1){ SetPlayerMarkerForPlayer(p, playerid, 0x0000FFFF & 0xFFFFFF00);}
			else if(pInfo[playerid][pTeam] == 2){ SetPlayerMarkerForPlayer(p, playerid, 0xFF0000FF & 0xFFFFFF00);}
			else{ SetPlayerMarkerForPlayer(p, playerid, 0xFFFFFFFF & 0xFFFFFF00);}
		}
	}
}

Alles anzeigen

Wenn das nicht geht, dann würde ich dir zu einem 1-Sekunden Timer raten, der für alle Spieler ausgeführt wird, also einmal bei OnGameModeInit gestartet wird.
Ich würde mich aber wundern, wenn es bei OnPlayerUpdate nicht geht.

breadfish.de 12. März 2022