[CloW]

Код:

Warning: client exceeded 'ackslimit' 5.43.68.93:52706 (9840) Limit: 8000/sec

How to fix this?

[PowerPC603]

Do you have alot of code in OnPlayerUpdate, especially loops?

This is the result of too many messages per second between server and client.

A good example could be having "Kill(playerid)" inside OnPlayerDeath.
This one was posted a few weeks ago on the forum.

Killing a player called OnPlayerDeath, which killed the player again, calling OnPlayerDeath again.
A never-ending loop, which communicates alot between server and client.

The default max-limit is set to 3000 and using normal and good written code should never even come close to that.
You have 9840 messages sent between server and client every second.
Search for big loops, fast timers and such things to spot the problem.

[CloW]

Код:

public OnPlayerUpdate(playerid)
{
	if(!IsPlayerConnected(playerid)) return 0;

	if(GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
	{
		for(new i = 1; i < sizeof(SpikeInfo); i++)
		{
			if(IsPlayerInRangeOfPoint(playerid, 3.0, SpikeInfo[i][sgX], SpikeInfo[i][sgY], SpikeInfo[i][sgZ]))
			{
				if(SpikeInfo[i][sgCreated] == 1)
				{
					new panels, tires;
					new carid = GetPlayerVehicleID(playerid);
					GetVehicleDamageStatus(carid, panels, doors, lights, tires);
					tires = encode_tires(1, 1, 1, 1);
					UpdateVehicleDamageStatus(carid, panels, doors, lights, tires);
					return 0;
				}
			}
		}
	}

	return 1;
}

This is my OnPlayerUpdate.

[CloW]

BUMP!

[bigboy81]

put this on the end line in server.cfg

ackslimit 100000
messageholelimit 100000
messageslimit 100000
playertimeout 100000

[CloW]

I want to fix this,i set ackslimit on 40000 then block players because acks is bigger than ackslimit.
How to fix this?

[ball]

Код:

if(GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
	{
		for(new i = 1; i < sizeof(SpikeInfo); i++)
		{
			if(IsPlayerInRangeOfPoint(playerid, 3.0, SpikeInfo[i][sgX], SpikeInfo[i][sgY], SpikeInfo[i][sgZ]))
			{
				if(SpikeInfo[i][sgCreated] == 1)
				{
					new panels, tires;
					new carid = GetPlayerVehicleID(playerid);
					GetVehicleDamageStatus(carid, panels, doors, lights, tires);
					tires = encode_tires(1, 1, 1, 1);
					UpdateVehicleDamageStatus(carid, panels, doors, lights, tires);
					return 0;
				}
			}
		}
	}

Man, OPU is called very frequently per second (something like timer for 40ms). Move this code to timer for 1000ms.

[CloW]

Thanks man.I think this is fixed,if don't work i will open thread.