I am using a Tick event as a cooldown timer and am getting outputs from both client and server, breaking my timer length.

In this code, cooldown is printing twice every tick and incrementing both times, breaking my speed.

 

@SubscribeEvent
	public static void playerTick(TickEvent.PlayerTickEvent event) {
		cooldown++;
		System.out.println(cooldown);
	}

 

I know I'm missing something obvious here, is there anyway I can only get one of the two?

 

Thank you for your help.

Edited April 2, 20205 yr by ultra_reemun
Solved!

Check for only one side and don't increment your counter if its on the wrong side?

21 minutes ago, Draco18s said:

Check for only one side and don't increment your counter if its on the wrong side?

Thanks for the suggestion! I'm new to modding, do you know how I would check for only one side?

36 minutes ago, ultra_reemun said:

Thanks for the suggestion! I'm new to modding, do you know how I would check for only one side?

You check for logical sides by using World::isRemote. if it is false the world is on the logical server if it is true you are on the logical client.

You could also set the sides of your event bus subscriber to prevent your subscribers from being loaded on a specific side.

My bad.

Edited April 2, 20205 yr by DavidM

4 hours ago, diesieben07 said:

That checks distribution (dedicated server vs client). Client in this context still includes the integrated server, which is not what you want.

 

@ultra_reemun Please refer to the documentation article about sides.

Also note that tick events always fire twice per tick, you must check TickEvent#phase.

Thank you