[Ajuda] OnPlayerUpdate + gettime usando muita CPU pra detectar players pausados/afk

[hiwyn]

Com esse set no OnPlayerUpdate

Código:

public OnPlayerUpdate(playerid)
{
    AFKTime[playerid] = gettime();
}

E depois um check simples numa outra função que roda a cada 1 segundo

Código:

// ongamemodeinit
SetTimerEx("PlayerOneSec", 1000, true, "i", playerid);

Código:

public PlayerOneSec(i) {
        // check paused
        if(paused[i] == false) {
            // 3 segundos sem resposta.
            if(GetTickCount() - AFKTime[i] > 3) { OnPlayerPause(i); }
        }
        if(paused[i] == true) {
            if(GetTickCount() - AFKTime[i] < 3) { OnPlayerUnpause(i); }
        }

O problema é que isso está consumindo muita CPU do meu servidor


Alguém sabe uma forma mais leve de fazer isso ou alguma forma de melhorar essa verificação?

[well]

Já pensou em criar uma callback e criar um timer pra chama-la a cada segundo?

Edit.
OnPlayerUpdate pode ser chamada até 30 vezes por segundo, então acho que fica inviável para o que tu deseja fazer.

[xbruno1000x]

O ideal é realmente criar um timer independente. A OnPlayerUpdate é chamada muitas vezes por segundo, e a detecção de AFK não tem uma necessidade tão grande.

[hiwyn]

@well e @xbruno1000x
eu já uso uma função de 1 segundo, ela valida se o player tá afk (editei o tópico pra ficar mais claro, eu tinha explicado mal)
o onplayerupdate só seta o afktime

o problema é que se eu tirar o

Código:

AFKTime[playerid] = gettime();

do OnPlayerUpdate e passar pra função que é chamada em 1 segundo

Código:

public PlayerOneSec(i) {
    AFKTime[playerid] = gettime();
        // check paused
        if(paused[i] == false) {
            // 3 segundos sem resposta.
            if(GetTickCount() - AFKTime[i] > 3) { OnPlayerPause(i); }
        }
        if(paused[i] == true) {
            if(GetTickCount() - AFKTime[i] < 3) { OnPlayerUnpause(i); }
        }

o player fica afk, da pause, mas o servidor nunca detecta
parece que quando o player pausa a função de PlayerOneSec não chega nele e a OnPlayerUpdate chega
então se eu tiro do OnPlayerUpdate para de funcionar

vcs sabem o pq isso acontece?
e como arrumar?

[RodrigoMSR]

Na prática esse código na OnPlayerUpdate não deveria causar grandes impactos na performance. De qualquer forma, use a função GetTickCount, ela é muito mais rápida que gettime. Será necessário adaptar algumas coisas no código, já que ela retorna milissegundos, e cuidado com o overflow que acontece com o servidor ligado por muitos dias (veja na wiki).

[hiwyn]

Na prática esse código na OnPlayerUpdate não deveria causar grandes impactos na performance. De qualquer forma, use a função GetTickCount, ela é muito mais rápida que gettime. Será necessário adaptar algumas coisas no código, já que ela retorna milissegundos, e cuidado com o overflow que acontece com o servidor ligado por muitos dias (veja na wiki).

Troquei pra GetTickCount como você sugeriu, ta funcionando agora eu to preocupado com o overflow, não entendi direito, o que eu devo adaptar no codigo pra não ter problema com isso?

Código:

public OnPlayerUpdate(playerid)

{

    AFKTime[playerid] = GetTickCount();

}

Código:

// ongamemodeinit

SetTimerEx("PlayerOneSec", 1000, true, "i", playerid);

Código:

public PlayerOneSec(i) {

        // check paused

        if(paused[i] == false) {

            // 3 segundos sem resposta.

            if(GetTickCount() - AFKTime[i] > 3000) { OnPlayerPause(i); }

        }

        if(paused[i] == true) {

            if(GetTickCount() - AFKTime[i] < 3000) { OnPlayerUnpause(i); }

        }

[RodrigoMSR]

https://team.sa-mp.com/wiki/GetTickCount.html

Após o servidor ficar 24 dias ligado, a função começará a retornar números negativos, o que pode quebrar os sistemas que estão usando ela. Se reiniciar o servidor periodicamente, não terá problema. Outra solução é usar a função GetTickDiff criada pela comunidade: https://gist.github.com/ziggi/5d7d8dc42f...924c608e73

[hiwyn]

https://team.sa-mp.com/wiki/GetTickCount.html

Após o servidor ficar 24 dias ligado, a função começará a retornar números negativos, o que pode quebrar os sistemas que estão usando ela. Se reiniciar o servidor periodicamente, não terá problema. Outra solução é usar a função GetTickDiff criada pela comunidade: https://gist.github.com/ziggi/5d7d8dc42f...924c608e73

só trocar o PlayerOneSec

de:

Código:

public PlayerOneSec(i) {

        // check paused

        if(paused[i] == false) {

            // 3 segundos sem resposta.

            if(GetTickCount() - AFKTime[i] > 3000) { OnPlayerPause(i); }

        }

        if(paused[i] == true) {

            if(GetTickCount() - AFKTime[i] < 3000) { OnPlayerUnpause(i); }

        }
}

pra:
de:

Código:

public PlayerOneSec(i) {
    new interval = GetTickDiff(GetTickCount(), AFKTime[playerid]);
    if(paused[i] == false) {
        // 3 segundos sem resposta.
        if(interval > 3000) { OnPlayerPause(i); }
    }
    if(paused[i] == true) {
        if(interval < 3000) { OnPlayerUnpause(i); }
    }
}

certo?



uma duvida:
com mais essa adição de stock, o `GetTickCount` não ficaria mais pesado que o `gettime`?

[RodrigoMSR]

Certo.

GetTickDiff está no timer de 1 segundo, não na OnPlayerUpdate, então mesmo que seja um pouco mais lento, não vai afetar em nada. E mesmo se estivesse, ainda é melhor que usar gettime.

Benchmark:

Código:

new tick = GetTickCount();
for(new i = 0; i < 2000000; i++)
{
    gettime();
}
printf("gettime: %dms", GetTickCount() - tick);


tick = GetTickCount();
for(new i = 0; i < 2000000; i++)
{
    GetTickDiff(GetTickCount(), tick);
}
printf("GetTickCount + GetTickDiff: %dms", GetTickCount() - tick);

Resultado (Linux):

gettime: 1676ms
GetTickCount + GetTickDiff: 87ms

No Windows a diferença é bem menor, mas no Linux foi quase 20x mais lento.

[hiwyn]

Certo.

GetTickDiff está no timer de 1 segundo, não na OnPlayerUpdate, então mesmo que seja um pouco mais lento, não vai afetar em nada. E mesmo se estivesse, ainda é melhor que usar gettime.

Benchmark:

Código:

new tick = GetTickCount();
for(new i = 0; i < 2000000; i++)
{
    gettime();
}
printf("gettime: %dms", GetTickCount() - tick);


tick = GetTickCount();
for(new i = 0; i < 2000000; i++)
{
    GetTickDiff(GetTickCount(), tick);
}
printf("GetTickCount + GetTickDiff: %dms", GetTickCount() - tick);

Resultado (Linux):

gettime: 1676ms
GetTickCount + GetTickDiff: 87ms

No Windows a diferença é bem menor, mas no Linux foi quase 20x mais lento.

muito obrigado, melhor impossivel!

eu fiz certo a resolução pra fugir do overflow?

mais uma coisa, pensei numa parada pra deixar o onplayerupdate mais leve

Código:

public OnPlayerUpdate(playerid)
{
    if(updatingAFKTime[playerid] == false) {
        AFKTime[playerid] = GetTickCount();
        updatingAFKTime[playerid] = true;
        SetTimerEx("ResetUpdatingAFKTime", 250, false, "i", playerid);
    }
    return true;
}

public ResetUpdatingAFKTime(playerid) {
    updatingAFKTime[playerid] = false;
}

Será que é bom?