damage change on level

[AkaiNight]

i was trying to make a code that changes damage on level but it was working little broken it was does work but you must relogin for damage change so i tried to edit and make it work fine but now it won't even work and im not sure what is the problem im not getting any errors or something like it's like it's not even there

here is code

in master_postinit

	inst:ListenForEvent("equip", function()
	    local equippedweb = inst.components.inventory:GetEquippedItem(EQUIPSLOTS.HANDS)
		
		if equippedweb ~= nil and equippedweb:HasTag("shirayuki") then
			inst.components.combat.damagemultiplier = 1
		else
			inst.components.combat.damagemultiplier = 0
			
	
		end
	end)
	
	
		inst:ListenForEvent("unequip", function()
        local unequippedwep = inst.components.inventory:GetEquippedItem(EQUIPSLOTS.HANDS)
		
		if unequippedwep ~= nil and unequippedwep:HasTag("shirayuki") then
			inst.components.combat.damagemultiplier = 0
		else
			inst.components.combat.damagemultiplier = 0
				
		end
	end)

this is not working(and no error as well)

and i tried this but when i use this you must relogin for damage change

	inst:ListenForEvent("equip", function()
	    local equippedweb = inst.components.inventory:GetEquippedItem(EQUIPSLOTS.HANDS)
		
		if equippedweb ~= nil and equippedweb:HasTag("shirayuki") then
			inst.components.combat.damagemultiplier = 1 + inst.level
		else
			inst.components.combat.damagemultiplier = 0
			
	
		end
	end)
	
	
		inst:ListenForEvent("unequip", function()
        local unequippedwep = inst.components.inventory:GetEquippedItem(EQUIPSLOTS.HANDS)
		
		if unequippedwep ~= nil and unequippedwep:HasTag("shirayuki") then
			inst.components.combat.damagemultiplier = 0
		else
			inst.components.combat.damagemultiplier = 0
				
		end
	end)

and i tried this well

 

	inst:ListenForEvent("equip", function()
	    local equippedweb = inst.components.inventory:GetEquippedItem(EQUIPSLOTS.HANDS)
		
		if equippedweb ~= nil and equippedweb:HasTag("shirayuki") and inst.level == 0 then
			inst.components.combat.damagemultiplier = 1
		else
			inst.components.combat.damagemultiplier = 0
		elseif equippedweb ~= nil and equippedweb:HasTag("shirayuki") and inst.level == 1 then
			inst.components.combat.damagemultiplier = 2
		else
			inst.components.combat.damagemultiplier = 0
		.
		.
		.
		.
		end

	end)
	
	
		inst:ListenForEvent("unequip", function()
        local unequippedwep = inst.components.inventory:GetEquippedItem(EQUIPSLOTS.HANDS)
		
		if unequippedwep ~= nil and unequippedwep:HasTag("shirayuki") then
			inst.components.combat.damagemultiplier = 0
		else
			inst.components.combat.damagemultiplier = 0
				
		end
	end)

this one is not worked as well so im not sure what is wrong and im not sure what should i do

[Ultroman]

It doesn't only recalculate on relogin. It recalculates whenever something is equipped or unqeuipped. Those are the events you have chosen to listen to.

You should recalculate the damage whenever your character levels up. Since your character's damage multiplier is contingent on them having a certain set of HANDS equipment, you rightly also have to update when said item is equipped or unequipped. You have hooked up to two events which are fired whenever ANY item is equipped or unequipped. We can solve this by making them do the same job. The check will be roughly the same if we split it into separate equip/unequip versions, so unless this becomes more complex, this should be fine.

local updateDamageMultiplier = function(inst)
	local equippedweb = inst.components.inventory:GetEquippedItem(EQUIPSLOTS.HANDS)
	
	if equippedweb ~= nil and equippedweb:HasTag("shirayuki") then
		inst.components.combat.damagemultiplier = 1 + inst.level
	else
		inst.components.combat.damagemultiplier = 0
	end
end

inst:ListenForEvent("equip", function(inst, data)
	-- Remember, you get both the inst (listener, so here it is the character) and data which
	-- has different data depending on which event you listen to. In this case, we are listening
	-- to "equip", which is pushed like this:
	-- self.inst:PushEvent("equip", { item = item, eslot = eslot })
	-- and "unequip", which is pushed like this:
	-- self.inst:PushEvent("unequip", {item=item, eslot=equipslot, slip=slip})
	-- You can access these like so: data.eslot
	-- So we can ask which type of item was equipped or unequipped.
	-- If there is no data (there always is) or the equipped item is for the HANDS-eslot.
	if not data or data.eslot == EQUIPSLOTS.HANDS then
		updateDamageMultiplier(inst)
	end
	
	-- If you have other items or things which influence this multiplier,
	-- just call updateDamageMultiplier instead of having the if-statement.
end)

inst:ListenForEvent("unequip", function(inst, data)
	-- Ditto above.
	-- If there is no data (there always is) or the unequipped item is for the HANDS-eslot.
	if not data or data.eslot == EQUIPSLOTS.HANDS then
		updateDamageMultiplier(inst)
	end
	
	-- If you have other items or things which influence this multiplier,
	-- just call updateDamageMultiplier instead of having the if-statement.
end)

-- Also call updateDamageMultiplier wherever you count up the level variable.

 

Edited January 3, 2020 by Ultroman

[AkaiNight]

(quick fix) this should be like

if not data or data.eslot == EQUIPSLOT.HANDS then

this

if not data or data.eslot == EQUIPSLOTS.HANDS then

thanks for help ultroman.

[Ultroman]

You're welcome! I've fixed my code to your specifications