Eficácia de scripts globalEvents ou CreatureScripts

[FeeTads 172]

salve rapaziada, estou fazendo uma quest no meu OT que é necessário faze-la durante 10 dias consecutivos, porém caso o player perca um dia, a storage da quest reseta.
Já tenho esses scripts prontos de 2 formas: global event que checa a storage de todos os player online no momento e caso ja tenha passado 24h ele tira a storage do player que está entre os 10 dias de quest.
E também tenho um creatureScript de onLogin() que quando o player loga, ele entra num loop de verificação a cada 60s

minha duvida: globalEvents vai checar todos os players online de uma só vez e fazer as alterações necessarias, isso pode lagar a distro, ou até mesmo crashar?
o creatureScript vai entrar num loop até o player deslogar, isso numa média de 250 pessoa são diversas verificações em momentos diferentes, pode acabar lagando ou crashando?

meu OT possui uma media de 300 pessoas online.

Script globalEvents é esse:

function onThink(interval, lastExecution)
    local players = {}
    local timer = os.time()
    for _, pid in pairs (getPlayersOnline()) do
        local storage = getPlayerStorageValue(pid, 1231234)
        if getPlayerStorageValue(pid,888251) > 0 and getPlayerStorageValue(pid,888251) <= 9 then
            if storage - timer <= 1  then
                table.insert(players, pid)
            end
        end
    end

    
    if #players > 0 then
        for i = 1, #players do
            doPlayerSendTextMessage(players[i],22,"seu dano voltou ao normal por vc nao ter feito a quest!")
            setPlayerStorageValue(players[i],888251,0)
        end
    end
return true
end

Script do Creature é esse:
local storage = getPlayerStorageValue(cid,1231234)
        function checkStorage(cid) local timer = os.time()
               if not isPlayer(cid) then return true end
               if getPlayerStorageValue(cid,888251) > 0 and getPlayerStorageValue(cid, 888251) <= 9 then
                       if storage - timer <= 1 then
                               setPlayerStorageValue(cid,888251,0)
                       end
             end
             addEvent(function()
                        checkStorage(cid)
              end, 60000)
end
function onLogin(cid)
        checkStorage(cid)
return true
end
function onLogout(cid)
       stopEvent(checkStorage[cid])
return true
end


caso os códigos nao estejam legiveis me avisem como arrumar em .lua pf

se quiserem usar os códigos podem usar a vontade kkkkkkk

[So volto tarde 64]

Este tópico foi movido para a seção de Suporte Otserv.

o creaturescript de onlogin seria o melhor nesse caso, aliás, me diz uma Coisa, porque verificar a cada 10 segundos, minutos, ou horas se a quest é diária? dá pra deixar somente o evento de login com algumas verificações 

[FeeTads 172]

tipo assim @So volto tarde preciso verificar pelo menos a cada 10 minutos, porque a quest é diária porém o portal abre 19h e fecha 19:03h, ai quem não entrar tem que perder a storage após passar 24h de ter clicado no baú, e não da pra fazer uma verificação diaria, pq se o cara logar após a quest ele n vai perder a storage, então eu precisaria de algo que a cada 10 minutos pelo menos verifique os players pra pode tirar essa storage se eles perderem a quest

 

[So volto tarde 64]

6 horas atrás, FeeTads disse:

tipo assim @So volto tarde preciso verificar pelo menos a cada 10 minutos, porque a quest é diária porém o portal abre 19h e fecha 19:03h, ai quem não entrar tem que perder a storage após passar 24h de ter clicado no baú, e não da pra fazer uma verificação diaria, pq se o cara logar após a quest ele n vai perder a storage, então eu precisaria de algo que a cada 10 minutos pelo menos verifique os players pra pode tirar essa storage se eles perderem a quest

 

saquei mais ou menos, mas se for pelo que deu para entender, utilizar um globalevent será a melhor opção mesmo, só um adendo:

vc está verificando se apenas os players online estão com tal valor de storage, se um player entrar durante o interval de 1 globalevent pro outro e clicar no bau durante esse meio tempo, eles vao ter perdido o dia, porém mesmo assim vão ter pego o baú

[FeeTads 172]

aaah fraguei então, o global event vai ficar só checando se tem a storage e como a storage do bau eh os.time() de 24h vai ficar certinho então, vlw meu mano +rep

 

[luanluciano93 1994]

Olá Feetads, poderia explicar melhor como funcionará? talvez tenha uma forma mais fácil.
 

Até onde entendi é parecido com o atual daily reward do tibia global. Eu fiz algo parecido no meu server, a diferença é que ele dá uma recompensa por dia, e caso fique 24hrs em pegar, ele volta na primeira.

Dê uma olhada: 

local days = {
		-- [dia] = {itemId, itemCount, "ordem"},
		[1] = {2160, 1, "first"},
		[2] = {2160, 2, "second"},
		[3] = {2160, 3, "third"},
		[4] = {2160, 4, "fourth"},
		[5] = {2160, 5, "fifth"},
		[6] = {12328, 6, "sixth"}, -- stamina refill
		[7] = {2160, 7, "seventh"}
},

function onUse(player, item, fromPosition, target, toPosition, isHotkey)
	
	local timeQuest = player:getStorageValue(Storage.dailyQuestTime) > 0 and player:getStorageValue(Storage.dailyQuestTime) or 0
	if (timeQuest + 86400) >= os.time() then
		player:sendTextMessage(MESSAGE_STATUS_SMALL, "You still can't get the next reward.")
		player:getPosition():sendMagicEffect(CONST_ME_POFF)
		return true
	elseif (timeQuest + 172800) <= os.time() then
		player:setStorageValue(Storage.dailyQuest, 1)
	end

	local day = player:getStorageValue(Storage.dailyQuest) > 0 and player:getStorageValue(Storage.dailyQuest) or 1
	local dayReward = days[day]
	if dayReward then
		local reward = ItemType(dayReward[1])
		if reward:getId() ~= 0 then
			if player:getFreeCapacity() < reward:getWeight() then
				player:sendTextMessage(MESSAGE_STATUS_SMALL, "You don't have capacity.")
				player:getPosition():sendMagicEffect(CONST_ME_POFF)
				return true
			end

			player:sendTextMessage(MESSAGE_EVENT_ADVANCE, "You received ".. dayReward[2] .." ".. reward:getName() .. " as a reward for the ".. dayReward[3] .." day of daily quest.")
			player:addItem(reward:getId(), dayReward[2])
			player:setStorageValue(Storage.dailyQuestTime, os.time())

			if day == 7 then
				player:setStorageValue(Storage.dailyQuest, 1)
			else
				player:setStorageValue(Storage.dailyQuest, day + 1)
			end
		end
	end

	return true
end

 

Editado Janeiro 24, 2022 por luanluciano93 (veja o histórico de edições)

[FeeTads 172]

@luanluciano93

então acho que eh um pouco diferente desse daily reward (alias gostei do teu me deu uma ideia genial kkkkkkkk posso roubar?)

o script funciona da seguinte maneira
o action do bau seta a storage tempo pra +24h
e seta a storage de dano extra pra +1

ai vem o creature ou o globalEvent
que faria o seguinte, a cada 1 minuto checa os players onlines, e caso tenha passado 24 horas desde a ultima vez que ele fez a quest ele perderia a storage de dano extra

e apareceria na tela dele que ele perdeu.
pq checar a cada 1 minuto? pro cara nao entrar na quest, fazer uma parte deslogar esperar o check logar.... digamos que burlar o sistema
então eu precisaria que algo ficasse constantemente checando se ja passou 24horas após ele fazer a quest. e cada vez que ele faz a storage é setada com os.time()+24h pra ele nao perder a storage e ficar tudo certinho.

e pq checar constantemente? pro player nao ficar com esse dano extra por muito tempo
eu poderia deixar pra checar a cada hora ou quando abrisse a quest por exemplo, mas teria algumas maneiras de burlar e continuar com o dano extra.

e pq entre global ou creature?
global > checka todos de uma vez (pode ser um peso grande pra distro pegar 300 pessoas online e checkar um por um, setar a storage....)
creature > quando o player loga, ele começa um loop de onLogin a cada 1 minuto ele chama a função pra checar a storage, porém são varios loops em tempos aleatorios, e checar 300 players em tempos aleatorios pode fazer com que todo segundo esteja executando um loop e isso cause lag

Editado Janeiro 25, 2022 por FeeTads
faltou explicar sobre o pq precisar ser constante (veja o histórico de edições)

[luanluciano93 1994]

22 horas atrás, FeeTads disse:

então acho que eh um pouco diferente desse daily reward (alias gostei do teu me deu uma ideia genial kkkkkkkk posso roubar?)

CLARO.

 

talvez um evento "onTargetCombat" não resolveria toda essa verificação?

Por onde você checa esse dano extra?

[FeeTads 172]

25 minutos atrás, luanluciano93 disse:

CLARO.

 

talvez um evento "onTargetCombat" não resolveria toda essa verificação?

Por onde você checa esse dano extra?

o proprio sistema do reset system ja checa
 

function ResetSystem:applyBonuses(pid)
	local bonus = self:getInfo(pid)
	if (bonus and bonus.damage_percent) then
		if getPlayerStorageValue(pid, 888251) >= 1	then		--se a storage da quest for mais ou igual a 1 
			bonusquest = getPlayerStorageValue(pid, 888251)	-- recebe conforme a quantidade de vezes que fez a quest
		else
			bonusquest = 0;				--se não seta pra 0
		end
		setPlayerDamageMultiplier(pid, 1.0 + ((bonus.damage_percent+bonusquest) / 100.0))
	else
		setPlayerDamageMultiplier(pid, 1.0)
	end
end

caso ele tenha a storage maior que 1, significa que ele fez a quest tantas vezes, o bonus recebe a storage e depois aplica, esse sistema de damage é direto na source tbm
 

[luanluciano93 1994]

7 minutos atrás, FeeTads disse:

getPlayerStorageValue(pid, 888251) >= 1

e se você verificasse aqui o tempo da quest?

[FeeTads 172]

Agora, luanluciano93 disse:

e se você verificasse aqui o tempo da quest?

não iria lagar muito? conferir em cada hit do player se o tempo ja passou?

[luanluciano93 1994]

Agora, FeeTads disse:

não iria lagar muito? conferir em cada hit do player se o tempo ja passou?

Você já faz a verificação se ele tem a storage, o que muda você trocar pela storage de tempo?

[FeeTads 172]

1 minuto atrás, luanluciano93 disse:

Você já faz a verificação se ele tem a storage, o que muda você trocar pela storage de tempo?

não entendi mt bem, vc diz trocar a storage 888251 pela do tempo? tipo
 

getPlayerStorageValue(pid, 888251) >= 1

 pra 

getPlayerStorageValue(pid, storangeTempo) < 1

mas de qualquer forma eu teria que saber qual o valor da storage né?
ou vc diz colocar mais uma verificação?
 

if getPlayerStorageValue(pid, 888251) >= 1 and ..storagetempo < 1 then  ??

eh que eu não fraguei mt bem, pq a storage de tempo é por os.time, eu acho que ficaria muita verificação, e esse codigo é do reset system que aplica o bonus por hit
acho que isso poderia lagar muito

[luanluciano93 1994]

Algo assim 

function ResetSystem:applyBonuses(pid)
	local bonus = self:getInfo(pid)
	if (bonus and bonus.damage_percent) then
		local storageTime = getPlayerStorageValue(pid, 1111) - os.time()
		if storageTime >= 1	then		--se a storage da quest for mais ou igual a 1 
			bonusquest = getPlayerStorageValue(pid, 888251)	-- recebe conforme a quantidade de vezes que fez a quest
		else
			bonusquest = 0;				--se não seta pra 0
		end
		setPlayerDamageMultiplier(pid, 1.0 + ((bonus.damage_percent+bonusquest) / 100.0))
	else
		setPlayerDamageMultiplier(pid, 1.0)
	end
end