[Server] Problem with GetTickCount

[RhaegarX]

I made an anti flood command script for my gamemode, in which the player can only type commands every 2 seconds. Everything worked fine, until yesterday I realized that this script had bugged, any command typed by the player regardless of the time between them is classified as spam.



So I decided to do a type of debugging, to see what value the GetTickCount was returning, and it is returning very large negative values.


The amazing thing about this problem is that it only occurs on my computer, on my notebook and on the gamemode host this problem does not occur everything works correctly. Does anyone have any idea what this is? Maybe some problem with some windows dll? I do not believe it is a gamemode problem, because as I mentioned in other places it works normally, only on my computer this problem occurs.

Here are the plugins / includes I use and the anti flood code snippet

PHP Code:

#include? ? <a_samp>
#define FIXES_Single 1
#define FIXES_ServerVarMsg 0
#define LOCAL_HOST true
// Redefini??es antes da Fixes
#if? ? ? defined MAX_PLAYERS
#undef? ? MAX_PLAYERS
#if LOCAL_HOST == true
#define? MAX_PLAYERS? (100)
#else
#define? MAX_PLAYERS? (30)
#endif
#endif
//
#include <fixes>
#include <crashdetect>
#include? ? <timerfix>
#include? ? <a_mysql>
#include? ? <foreach>
#include? ? <izcmd>
#include? ? <sscanf2>
#include? ? <MV_Youtube>
#include? ? <streamer>
#include? ? <callbacks>
#include? ? <mSelection>
#define AUTO_SETUP
#include? ? <gmtime>


public OnPlayerCommandReceived(playerid, cmdtext[])
{
? ? if (!IsPlayerConnected(playerid) || playerid == INVALID_PLAYER_ID)
{
return SendClientMessage(playerid, COLOR_INVALID, "You are not connected."), false;
}
if (GetTickCount() - timeUsedCommand[playerid] < 2000)
? ? {
return SendClientMessage(playerid, COLOR_GRAD1, "Wait 2 seconds to retype a command!"), false;
? ? }
timeUsedCommand[playerid] = GetTickCount();
? ? return 1;
} 


plugins crashdetect fixchars sscanf mysql streamer timerfix gmtime SAMPSON

And this problem has occurred with any code that uses the native GetTickCount.

[Virsenas]

Do you keep your computer on at all times?

[Radical]

I have no idea about GetTickCount() but you can use gettime() instead.

PHP Code:

public OnPlayerCommandReceived(playerid, cmdtext[])
{
? ? if (!IsPlayerConnected(playerid) || playerid == INVALID_PLAYER_ID)
{
return SendClientMessage(playerid, COLOR_INVALID, "You are not connected."), false;
}
if (gettime() < timeUsedCommand[playerid])
? ? {
return SendClientMessage(playerid, COLOR_GRAD1, "Wait 2 seconds to retype a command!"), false;
? ? }
timeUsedCommand[playerid] = gettime();
? ? return 1;
} 


[RhaegarX]

Do you keep your computer on at all times?

No. I usually use it for my daily tasks and then hang up. literally, overnight this problem appeared and what intrigues me the most is that it only happens on the computer.

[Virsenas]

Do you keep your computer on at all times?

No. I usually use it for my daily tasks and then hang up. literally, overnight this problem appeared and what intrigues me the most is that it only happens on the computer.

Do you get the same results without using timerfix and gmtime?

[RhaegarX]

I have no idea about GetTickCount() but you can use gettime() instead.

PHP Code:

public OnPlayerCommandReceived(playerid, cmdtext[])

{

? ? if (!IsPlayerConnected(playerid) || playerid == INVALID_PLAYER_ID)

{

return SendClientMessage(playerid, COLOR_INVALID, "You are not connected."), false;

}

if (gettime() < timeUsedCommand[playerid])

? ? {

return SendClientMessage(playerid, COLOR_GRAD1, "Wait 2 seconds to retype a command!"), false;

? ? }

timeUsedCommand[playerid] = gettime();

? ? return 1;

} 


It is an option too, but on other devices with GetTickCount it works perfectly. And since this function returns the time in seconds since the server was turned on, there is a problem with that, as it is returning a negative time. I wanted to understand why this problem arose

[RhaegarX]

Do you keep your computer on at all times?

No. I usually use it for my daily tasks and then hang up. literally, overnight this problem appeared and what intrigues me the most is that it only happens on the computer.

Do you get the same results without using timerfix and gmtime?

I haven?t tested by removing these plugins, I?ll test it now and I?ll give you the answer shortly

[RhaegarX]

Do you keep your computer on at all times?

No. I usually use it for my daily tasks and then hang up. literally, overnight this problem appeared and what intrigues me the most is that it only happens on the computer.

Do you get the same results without using timerfix and gmtime?

I removed the plugins and the problem still persists.
OnPlayerText also has the same system using GetTickCount and the problem is exactly the same as well.

[RhaegarX]

I ran the gettickcount, on my computer and notebook at the same time to see what results would be obtained, below is the data:



Computer:





Notebook:

[RhaegarX]

folks I found the following information on the open mp wiki that talks about gettickcount uptime problems.



This is my computer's uptime


EDIT: I tested the include indicated on the wiki and it didn't work.

[RhaegarX]

I found the source of the problem, the GetTickCount () actually has problems with long periods of the connected server. I searched on some forums and windows 10 has a fastboot system that keeps some data saved in memory, such as uptime even after the computer is turned off. the computer is only actually "turned off" when you restart it. when restarting my computer my uptime reset and the function started working again. So, I think it's really better to use gettime (), rather than GetTickCount () to avoid problems.

[Virsenas]

folks I found the following information on the open mp wiki that talks about gettickcount uptime problems.







This is my computer's uptime





EDIT: I tested the include indicated on the wiki and it didn't work.

I thought I asked if you shut down your computer regularly.

Do you keep your computer on at all times?

I guess I didn't.

[RhaegarX]

folks I found the following information on the open mp wiki that talks about gettickcount uptime problems.



This is my computer's uptime


EDIT: I tested the include indicated on the wiki and it didn't work.

I thought I asked if you shut down your computer regularly.

Do you keep your computer on at all times?

I guess I didn't.

I turn off the computer during the night, but I keep it connected to power, as there is no use. But this windows fastboot system does not reset the computer uptime even when it is turned off (I believe that if you remove the computer from the power outlet it will reset)

https://social.technet.microsoft.com/For...progeneral

[Y_Less]

The wiki is sort of correct, but the solution is very very wrong. You just need to use a subtraction.



While the code might look like this before:

Code:

if (newTime < oldTime  2000)

{

    // Flooding.

}

The solution is amazingly simple - just rearrange the formula

Code:

if (newTime - oldTime < 2000)

{

    // Flooding.

}

That will totally account for all wrap-around and negative numbers (I'm not going to prove it mathematically here though).

[RhaegarX]

Thank you all for your help!

[Pinch]

Thank you all for your help!

You can (and should) use tickcount() as it's same as GetTickCount() except it's the time since THE SERVER is running, not THE COMPUTER.

https://open.mp/docs/scripting/functions/Tickcount

[rasheed]

You can also use this: https://github.com/ScavengeSurvive/tick-difference

[Y_Less]

Thank you all for your help!

You can (and should) use tickcount() as it's same as GetTickCount() except it's the time since THE SERVER is running, not THE COMPUTER.



https://open.mp/docs/scripting/functions/Tickcount

It makes no difference if you use the correct code (i.e. the code I posted).

You can also use this: https://github.com/ScavengeSurvive/tick-difference

No, you shouldn't. It makes the same mistake as I mentioned earlier. Well, not quite mistake - the result is eventually correct, but in an extremely round-about manner.

[Pinch]

Thank you all for your help!

You can (and should) use tickcount() as it's same as GetTickCount() except it's the time since THE SERVER is running, not THE COMPUTER.

https://open.mp/docs/scripting/functions/Tickcount

It makes no difference if you use the correct code (i.e. the code I posted).

Bench it.

Ik it has no difference tho (the formula)

[Y_Less]

Yeah I'm not talking about performance, nor do I care about the performance of a single simple native - and neither should you. Whatever your code, that won't be the bottleneck and you should spend your time improving something else.

[Virsenas]

Yeah I'm not talking about performance, nor do I care about the performance of a single simple native - and neither should you.? Whatever your code, that won't be the bottleneck and you should spend your time improving something else.

Agreed. Should never over do things with optimizations, become obsessed with it and pay attention to even smallest things. That is exactly how relationships are ruined ...

[Pinch]

Yeah I'm not talking about performance, nor do I care about the performance of a single simple native - and neither should you. Whatever your code, that won't be the bottleneck and you should spend your time improving something else.

Nawh I was just interested, didn't say it means a lot :D