is this overkill?

trying to have an easy way to track the different types of mobs for advancements. making advancements for every mob so when the player kills one they get a reward for a skill tree im developing. it runs once every tick with a tick.json tag file