Alright so I have these achievements here:

if(player.inventory.hasItem(Item.getItemFromBlock(WinterBlocks.icedStone))){
		player.addStat(WinterAch.icedStone, 1);
	}
	if(player.inventory.hasItem(WinterItems.iceCrystal)){
		player.addStat(WinterAch.iceCrystal, 1);
	}
	if(player.inventory.hasItem(WinterItems.iceIngot)){
		player.addStat(WinterAch.iceIngot, 1);
	}

 

So basically I have these all in a PlayerTickEvent which can lead onto some lag. At least logically. So I was wondering if there was a different place I could put these items with the same effect. These items are crafted in a custom furnace so using the pickup or crafting events won't be useful here.

Also, cache the result of Item.getItemFromBlock(WinterBlocks.icedStone) so you don't have to do the lookup every time.

You can do it from your furnace, you'll probably need a custom Slot class for your output and then check when the player takes the stack out.

 

Thanks! Forgot I had that.

Also, cache the result of Item.getItemFromBlock(WinterBlocks.icedStone) so you don't have to do the lookup every time.

Also done!

 

Thanks for the help.