how to you get the world without a method. 1.7.10

[Draco18s]

Show the code for the

isOffCooldown(...)

method.  We need to know why you need access to a World there.

As well as why that function is calling

getCooldown(...)

 

Notably, because isOffCooldown is calling getCooldown, you could just add a World param to getCooldown and then pass it, then badabing, you have your world object (but still have an infinite loop).

[kauan99]

here is a way of implementing an item with a cooldown without the need of any hack:

 

/**
* you can extend this class to implement any item that has a cooldown
* @author Kauan99
* 
*/
public class CooldownItem extends Item
{
final int cooldown;//in ticks. ideally, minecraft runs at 20 ticks per second

public CooldownItem(int cooldown)
{
	super();//if you extend something else, you have to call the proper constructor
	this.cooldown = cooldown;
}

@Override
    public void onCreated(ItemStack stack, World world, EntityPlayer player)
    {
	/*
	 * this method can be used to create the NBT if the item is crafted on a workbench.
	 * wont be invoked in creative though (I think)
	 */
	if(stack.stackTagCompound == null)
		stack.stackTagCompound = new NBTTagCompound();

	stack.stackTagCompound.setLong("lastuse", world.getTotalWorldTime());
    }
    
@Override
    public ItemStack onItemRightClick(ItemStack stack, World world, EntityPlayer player)
    {
	if(!world.isRemote)
	{
		if(stack.stackTagCompound != null)
		{
	        if(!isOnCoolDown(stack, world))
	        {
	        	stack.stackTagCompound.setLong("lastuse", world.getTotalWorldTime());
	        	doAction(stack, world, player);
	        }
	        else
	        {
	        	/*
	        	 * this message here just for test purposes. you could remove this else clause entirely in your actual code
	        	 */
	        	player.addChatMessage(new ChatComponentText("you could not use the item because it's on cooldown. wait " + (cooldown - (world.getTotalWorldTime() - stack.stackTagCompound.getLong("lastuse"))) + " more ticks to use it"));
	        }
		}
		else
		{
			stack.stackTagCompound = new NBTTagCompound();
			stack.stackTagCompound.setLong("lastuse", world.getTotalWorldTime());

			/* 
			 * bellow we have an example of recursion. this method calls itself, but we are certain this will happen only
			 * once for each item stack
			 */
			return this.onItemRightClick(stack, world, player);
		}
	}
	return stack;
        /*
         * or you could make this class extend something that already does stuff when used. just replace line above with
         * return super.onRightClick(stack, world, player);
         */
    }	

/**
 * override this each item that has a cooldown
 * @param stack
 * @param world
 * @param player
 */
protected void doAction(ItemStack stack, World world, EntityPlayer player)
{
	//possibly cause damage to the item stack and whatever else you want it to do.
	//message bellow just for test purposes, remove it in your actual code.
	player.addChatMessage(new ChatComponentText("the item is doing its thing"));
}

public boolean isOnCoolDown(ItemStack stack, World world)
{
	long lastUse = stack.stackTagCompound.getLong("lastuse");
	long rightNow = world.getTotalWorldTime();
	return rightNow - lastUse < cooldown;	
}
}

 

edit: no idea why the indentation got all messed up, but if you paste that into eclipse and ctrl+i the code it will get properly indented.

[Choonster]

FYI: The OP's code is adapted from an example I provided in their previous thread.

[kauan99]

FYI: The OP's code is adapted from an example I provided in their previous thread.

 

pretty similar to what I came up with. this is probably the only decent way to do it, since we both came up with virtually the same solution.