[[1.7.10] Consume Logs]

  On 12/5/2014 at 3:31 PM, Draco18s said:

player.getHeldItem()

?

player.inventory.getStackInSlot(slotNum)

?

 

I know how to get the held item. I just want to know how I can decrease 1 item from that stack

[[1.7.10] Consume Logs]

  Quote

how do I only take 1 away?

 

Or in other words, HOW do I decrement the stack size

[[1.7.10] Consume Logs]

  On 12/5/2014 at 2:50 PM, Draco18s said:

Unfortunately that function doesn't deal with metadata.  You're better of checking the player's hand for spruce (item == Blocks.log && itemDamage == 4 (I think spruce is 4)), and if so, decrementing the stack size.

 

Ok, I've now got this code

 

player.inventory.setInventorySlotContents(player.inventory.currentItem, null);

 

But that takes the whole stack away, how do I only take 1 away?

[[1.7.10] Consume Logs]

I want to consume one log (ex. spruce) everytime I right click on a block. Here is where I've got so far

 

player.inventory.consumeInventoryItem(Item.getItemFromBlock(Blocks.log));

 

But that code consume only oak log, but I want it to consume spruce

 

 

[[1.7.10] Check If Birch Is In Hand?]

  On 12/3/2014 at 5:52 PM, Draco18s said:

And now you know why we use trace statements.  They tell you what the program is doing as it does it, and lets you locate the problem area of code rather than BLINDLY GUESSING.

 

Yea, so sorry about so much fuss over me being blind https://www.youtube.com/watch?v=YhSXAyybEIM

[[1.7.10] Check If Birch Is In Hand?]

  On 12/3/2014 at 5:37 PM, FLUFFY2 said:

For the love of god, why not just simply use java correctly?

 

if(entityplayer.getCurrentEquippedItem() != null && (entityplayer.getCurrentEquippedItem().getItem() == Item.getItemFromBlock(Blocks.log) && entityplayer.getCurrentEquippedItem().getItemDamage() == 2))

 

Bugzoo i see you are new to this so i just give you a simple advice. Use the vanilla src!

 

OMFG! I just realized I was running another if statement somewhere in the method which checked if the block was a log, and if it was it ran some code. That was the problem. But, on a sidenote, i'm not new to this. I have been programming in java for 4 years, programming minecraft mods for 2 years. Published 3 mods. So, i'm not really new to this.

[[1.7.10] Check If Birch Is In Hand?]

  On 12/3/2014 at 5:25 PM, Draco18s said:

Just above that if statement, put this:

 

System.out.println("Right clicked with: " + hand.getItem() + ":" + hand.getItemDamage());
System.out.println(hand.getItem() == Item.getItemFromBlock(Blocks.log));
System.out.println(hand.getItemDamage() == 0);
System.out.println((hand.getItem() == Item.getItemFromBlock(Blocks.log) && hand.getItemDamage() == 0));

 

Then tell me what it prints out when you use various blocks.

 

When I right click with oak it prints this

 

[com.bugzoo.GranterMod.BlockGranter:onBlockActivated:50]: Right clicked with: net.minecraft.item.ItemMultiTexture@1c29de2:0
true
true
true

 

When I right click with any other type of log it prints out this

 

[com.bugzoo.GranterMod.BlockGranter:onBlockActivated:50]: Right clicked with: net.minecraft.item.ItemMultiTexture@1c29de2:3
true
false
false

 

 

[[1.7.10] Check If Birch Is In Hand?]

  On 12/3/2014 at 5:08 PM, Draco18s said:

  Quote

Describe how it doesn't work.  What seems to be the issue?

 

I just noticed something. That line of code does work. So if I did

 

 if(hand.getItemDamage() == 0)

 

The if statement would run, only if the block is Spruce or Dark Oak. Though if I do this

 

 if(hand.getItem() == Item.getItemFromBlock(Blocks.log) && hand.getItemDamage() == 0)

 

It runs no-matter what log it is.

[[1.7.10] Check If Birch Is In Hand?]

  On 12/3/2014 at 4:29 PM, Draco18s said:

  Quote

hand.getItemDamage() == 1

 

?

 

That doesn't work.

[[1.7.10] Check If Birch Is In Hand?]

  On 12/2/2014 at 8:24 PM, Draco18s said:

You need to add the metadata check, duh.

 

I know that, but how do I check the metadata?

[[1.7.10] Check If Birch Is In Hand?]

  On 12/2/2014 at 6:15 PM, larsgerrits said:

Does it go INSIDE(!) the if-statement if holding, for example, oak log?

 

Yea, so this if statement

 

 if(hand.getItem() == Item.getItemFromBlock(Blocks.log))

 

Runs the code no-matter what log it is. I only want it to run if it is birch

[[1.7.10] Check If Birch Is In Hand?]

  On 12/1/2014 at 8:11 PM, larsgerrits said:

Sorry for that, little unclear. I mean, only if you are holding birch wood, it would go into the if-statement, else it shouldn't.

 

Yea, thats what i'm trying to do, but that code doesn't work. It runs the method no-matter what log it is

[[1.7.10] Check If Birch Is In Hand?]

  On 12/1/2014 at 7:10 PM, larsgerrits said:

Ok, that method is called for EVERY time you right click on the block. But it shouldn't go inside the if-statement.

 

Ok, the way my code works, is if you right click with a birch log it runs the code within the if statement, so why shouldn't it go in the if statement?

[[1.7.10] Check If Birch Is In Hand?]

  On 12/1/2014 at 6:57 PM, larsgerrits said:

Full method, so including the method name and its signatures.

 

public boolean onBlockActivated(World world, int x, int y, int z, EntityPlayer player, int par6, float par7, float par8, float par9){
  if(player.getCurrentEquippedItem() != null)
      {
     ItemStack hand = player.getCurrentEquippedItem();
     
     if(hand.getItem() == Item.getItemFromBlock(Blocks.log) && hand.getItemDamage() == 1)
     {
    player.inventory.addItemStackToInventory(new ItemStack(Blocks.log, 1, 1));
    player.inventory.consumeInventoryItem(Item.getItemFromBlock(Blocks.log));
	RenderGranter.isAnimated = true;
	RenderGranter.renderOak = true;
	System.out.println(hand.getItem().getMetadata(1));
	world.spawnEntityInWorld((new EntityLightningBolt(world, x, y+1, z)));
	world.createExplosion((Entity)player, (double)x, (double)y, (double)z, 2F, false);
	}
}
}

 

 

[[1.7.10] Check If Birch Is In Hand?]

  On 12/1/2014 at 6:40 PM, larsgerrits said:

Show the full method with that if-statement. The method is probably run for EVERYTHING, and if you check for birch wood, it should only go inside that if-statement if so.

 

if(player.getCurrentEquippedItem() != null)
      {
     ItemStack hand = player.getCurrentEquippedItem();
     
     if(hand.getItem() == Item.getItemFromBlock(Blocks.log) && hand.getItemDamage() == 1)
     {
    player.inventory.addItemStackToInventory(new ItemStack(Blocks.log, 1, 1));
    player.inventory.consumeInventoryItem(Item.getItemFromBlock(Blocks.log));
	RenderGranter.isAnimated = true;
	RenderGranter.renderOak = true;
	System.out.println(hand.getItem().getMetadata(1));
	world.spawnEntityInWorld((new EntityLightningBolt(world, x, y+1, z)));
	world.createExplosion((Entity)player, (double)x, (double)y, (double)z, 2F, false);
	}
}
}

[[1.7.10] Check If Birch Is In Hand?]

  On 11/30/2014 at 4:34 PM, larsgerrits said:

You mean, it goes inside the if statements, as it shouldn't. But the method you have that if statement in, of course it will run it for every type of wood because you haven't checked for birch specifically.

 

How do I check for birch specifically?

[[1.7.10] Check If Birch Is In Hand?]

  On 11/30/2014 at 3:53 PM, Draco18s said:

Describe how it doesn't work.  What seems to be the issue?

 

It runs the method if you right click with any type of wood

[[1.7.10] Check If Birch Is In Hand?]

  On 11/30/2014 at 3:36 PM, Draco18s said:

The number of the slot you wish to change....and null.

 

Thanks, that works, but now going back in time to this.

 

  Quote

hand.getItemDamage() == birch damage

Hard???

 

I've got this code

if(hand.getItem() == Item.getItemFromBlock(Blocks.log) && hand.getItemDamage() == 1)

 

But this doesn't work

[[1.7.10] Check If Birch Is In Hand?]

  On 11/30/2014 at 3:27 PM, diesieben07 said:

InventoryPlayer implements IInventory so it has setInventorySlotContents.

 

I've got this code

player.inventory.setInventorySlotContents(int, itemstack);

So what arguments do I have to put in?

[[1.7.10] Check If Birch Is In Hand?]

  On 11/30/2014 at 3:07 PM, diesieben07 said:

*facedesk*

 

Of course that will throw an exception. Learn about autoboxing.

You need to set the Slot to null, not the currentItem field.

 

Lol, well how do I get the slot?

[[1.7.10] Check If Birch Is In Hand?]

  On 11/30/2014 at 2:30 PM, diesieben07 said:

Just set the current item slot (player.inventory.currentItem) to null.

 

Tried this and it crashed my game

player.inventory.currentItem = (Integer) null;

[[1.7.10] Check If Birch Is In Hand?]

  On 11/30/2014 at 1:33 PM, larsgerrits said:

hand.getItemDamage() == birch damage

Hard???

 

I've got that, but now I need to consume the birch wood

 player.inventory.consumeInventoryItem(Item.getItemFromBlock(Blocks.log));

[[1.7.10] Check If Birch Is In Hand?]

  On 11/30/2014 at 1:15 PM, larsgerrits said:

1) hand can be null, so check if it's not before calling

hand.getItem()

2) you can't compare Items to Blocks. Use

hand.getItem() == Item.getItemFromBlock(Blocks.log)

.

3) if you can check if

hand.getItemDamage()

is the damage of birch wood.

 

Ok, that works for oak wood, but how do I do it for birch using hand.getItemDamage?

[[1.7.10] Check If Birch Is In Hand?]

I want to check if Birch wood is in the players hand

 if(hand.getItem() == Blocks.log)

I have this code, but I get an error and thats oak and not birch

[[1.7.10] Animate Rendered Block?]

  On 11/26/2014 at 5:30 PM, diesieben07 said:

"On a Block"? Any Block? Or the Block that renders the Item?

 

The diamond block that I render on my item