SpawnerSpawnEvent is called once per entity spawn about iridiumskyblock

[Nubebuster]

Though my solution seems very hack, it's performance efficient and it works.

[dlsf]

Starting a thread for every spawner is not efficient imo

[Nubebuster]

Starting a thread for every spawner is not efficient imo

The code I wrote creates a runnable which is executed by an existing scheduler thread in bukkit. There is no new thread created, only a runnable. That is relatively a very inexpensive instruction. All the code is doing is putting an instruction into a list that is executed either the next game tick or at the end of the current tick on the main game thread. Either of thoseThis is good enough because all entity spawns will be executed before this happens.

See https://hub.spigotmc.org/stash/projects/SPIGOT/repos/craftbukkit/browse/src/main/java/org/bukkit/craftbukkit/scheduler/CraftScheduler.java

[Nubebuster]

Alternatively, we could check if the spawner already spawned a monster during the current tick using Bukkit.getCurrentTick(). But I can't quickly think of a way to store when the last tick was the spawner spawned something, since there are not any methods in java that I know of that have expiring data objects which would run more efficiently than this