onItemUse vs onBlockActivated

[target.san]

Hello everyone!

 

I wanna add some special code which is executed when redstone block is ignited via flint'n'steel. So my question is, which place is more appropriate for this code - redstone block's onBlockActivated or flint'n'steel's onItemUse. The exact way of doing this via ASM is out of scope.

 

Thanks!

[SanAndreaP]

Why not use the PlayerInteractEvent? Check if the type is RIGHT_CLICK_BLOCK and it'll have the block coords of the block right-clicked. Check if the player has flint and steel and do your custom code if all conditions are met.

[jabelar]

Usually in programming there is more than one way to approach these things.  In general, if I had a custom item that was interacting with vanilla items it would of course make more sense to add the code in the custom item.  Conversely, if I had a custom block that was interacting with vanilla items then it would make more sense to add the code in the custom block.

 

Even more generally, as SanAndreasP points out, when changing behavior related to vanilla stuff it is best to look for events (hooks provided by FML) which are exactly intended to help you with such interceptions of vanilla processing.  Even with events though, there are usual several alternatives to choose from -- handle in the player, the block, the world tick, or whatever.