Jump to content
Search In
  • More options...
Find results that contain...
Find results in...

how to you get the world without a method. 1.7.10


BoonieQuafter-CrAfTeR
 Share

Recommended Posts

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).

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

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.

WIP mods: easyautomation, easyenergy, easyelectronics, easymoney, easytrasportation, easysecurity, easymultiverse, easyfactions, easymagick, easyalchemy, easyseasons

Link to comment
Share on other sites

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.

WIP mods: easyautomation, easyenergy, easyelectronics, easymoney, easytrasportation, easysecurity, easymultiverse, easyfactions, easymagick, easyalchemy, easyseasons

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share



×
×
  • Create New...

Important Information

By using this site, you agree to our Privacy Policy.