[1.8] [SOLVED] is effective on check

[statphantom]

I am wondering the best way to loop through a Item's isEffectiveOn() Block <set> in order to manually change the mining speed. Please help with your ideas? thanks.

 

I am currently doing....

if (isEffective(helditem, block)) {
		event.newSpeed = event.originalSpeed * 0.3f;
	}

private boolean isEffective(ItemStack helditem, Block block) {
	if (helditem.getItem() instanceof ItemTool) {
		Set tooltypes = helditem.getItem().getToolClasses(helditem);

		Iterator<String> i = tooltypes.iterator();

		while (i.hasNext()) {
			if (block.isToolEffective(i.next(), block.getDefaultState())) {
				return true;
			}
		}

	}
	return false;
}

[statphantom]

1. following tutorials it said to do this, and I have done this due to that my tool really is a tool and I take advantage of harvest levels, tool material etc. To me this seems simpler and more compatibly friendly

 

2. thanks fixed that

 

3. I had seen and fixed that but forgot to edit this post to show it

 

4. this is how I learnt to iterate through sets when learning java

[statphantom]

I just assumed using as much vanilla code as possible was best for comparability, so should I never extend ItemTool? and instead just extend Item?

[statphantom]

ohh I see sorry, misread what you meant

 

I have marked this one as solved but I am still having issues on the other topics you have been helping me with