[1.6.4] Problem With addItemStackToInventory

[Bugzoo]

Im using addItemStackToInventory and i keep getting random numbers of items back. Here is the code im using

((EntityPlayer)player).inventory.addItemStackToInventory(new ItemStack(Minespresso.SugaryMagicCappuccino, 1));

 

[Elyon]

Can you post your entire method or class?

 

In what context are you calling

addItemStackToInventory

?

[Bugzoo]

Im sending a packet.

 

if (packet.channel.equals("DrinksNetwork") && packet.sentID == 1) {
            	((EntityPlayer)player).inventory.addItemStackToInventory(new ItemStack(Minespresso.Cappuccino, 1));
            }

 

 

[Elyon]

Can you post your entire method or class?

 

Under what circumstances is this packet sent? Where in your class/method setup are you calling

addItemStackToInventory

(what method in what class)?

 

You have to help us to help you

[Bugzoo]

Can you post your entire method or class?

 

Under what circumstances is this packet sent? Where in your class/method setup are you calling

addItemStackToInventory

(what method in what class)?

 

You have to help us to help you

Im handling a packet so its pretty obvious what class its in but heres my code

package MinespressoMod;

import java.io.ByteArrayInputStream;
import java.io.DataInputStream;
import java.io.IOException;

import net.minecraft.client.Minecraft;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.item.ItemStack;
import net.minecraft.network.INetworkManager;
import net.minecraft.network.packet.Packet250CustomPayload;
import cpw.mods.fml.common.network.IPacketHandler;
import cpw.mods.fml.common.network.Player;

public class PacketHandler implements IPacketHandler {


    @Override
    public void onPacketData(INetworkManager manager,
                    Packet250CustomPayload packet, Player player) {

    	//Cappuccino
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 1) {
            	((EntityPlayer)player).inventory.addItemStackToInventory(new ItemStack(Minespresso.Cappuccino, 1));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 2) {
            	((EntityPlayer)player).inventory.addItemStackToInventory(new ItemStack(Minespresso.SugaryCappuccino, 1));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 3) {
            	((EntityPlayer)player).inventory.addItemStackToInventory(new ItemStack(Minespresso.MagicCappuccino, 1));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 4) {
            	((EntityPlayer)player).inventory.addItemStackToInventory(new ItemStack(Minespresso.SugaryMagicCappuccino, 1));
            }
            
         //Coffee
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 10) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.Coffee));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 11) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.SugaryCoffee));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 12) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.MagicCoffee));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 13) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.SugaryMagicCoffee));
            }
           
          //Minespresso
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 20) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.Minespresso));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 21) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.SugaryMinespresso));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 22) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.MagicMinespresso));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 23) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.SugaryMagicMinespresso));
            }
            
          //Latte
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 30) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.Latte));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 31) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.SugaryLatte));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 32) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.MagicLatte));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 33) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.SugaryMagicLatte));
            }
            
            //Tea
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 40) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.Tea));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 41) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.SugaryTea));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 42) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.MagicTea));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 43) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.SugaryMagicTea));
            }
            
          //HotChocolate
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 50) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.HotChocolate));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 51) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.SugaryHotChocolate));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 52) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.MagicHotChocolate));
            }
            if (packet.channel.equals("DrinksNetwork") && packet.sentID == 53) {
            	((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.SugaryMagicHotChocolate));
            }
           
    }
   
    private void handleRandom(Packet250CustomPayload packet) {
            DataInputStream inputStream = new DataInputStream(new ByteArrayInputStream(packet.data));
           
            int randomInt1;
            int randomInt2;
           
            try {
                    randomInt1 = inputStream.readInt();
                    randomInt2 = inputStream.readInt();
            } catch (IOException e) {
                    e.printStackTrace();
                    return;
            }
           
            System.out.println(randomInt1 + " " + randomInt2);
    }

}

[Elyon]

Under what circumstances is this packet sent?

 

It is possibly either being sent or handled multiple times, or it might be related to the lack of @SideOnly annotations.

 

...

 

Also:

• I did not ask where you handle your packets
  
• You wrote that you were sending a packet, not handling one
  
• The code you wrote earlier turned out to be from packet handling, not sending
  
• Handling or sending a packet does not make it obvious what custom class you are handling/sending it in, or what the contents of that class are
  
• You still haven't pasted any code from where you send the packet
  

 

Hardly obvious, no need to be snappy. We're trying to help, here.

 

Finally, please use http:/gist.github.com/ for longer pastes.

[Bugzoo]

Under what circumstances is this packet sent?

 

It is possibly either being sent or handled multiple times, or it might be related to the lack of @SideOnly annotations.

 

...

 

Also:

• I did not ask where you handle your packets
  
• You wrote that you were sending a packet, not handling one
  
• The code you wrote earlier turned out to be from packet handling, not sending
  
• Handling or sending a packet does not make it obvious what custom class you are handling/sending it in, or what the contents of that class are
  
• You still haven't pasted any code from where you send the packet
  

 

Hardly obvious, no need to be snappy. We're trying to help, here.

 

Finally, please use http:/gist.github.com/ for longer pastes.

Sorry...

public void actionPerformed(GuiButton guibutton){
	if(guibutton.id == 1){
		Packet250CustomPayload packet = new Packet250CustomPayload();
            packet.channel = "DrinksNetwork"; 
            packet.sentID = 50;
            PacketDispatcher.sendPacketToServer(packet);       
	}

[Elyon]

No worries; we're here to help

 

Anyway, the code you pasted has

sentID

set to 50, which would execute this line:

((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.HotChocolate));

 

You may want to change the

sentID

to 1, or chance the line above to use

addItemStackToInventory

.

[Bugzoo]

No worries; we're here to help

 

Anyway, the code you pasted has

sentID

set to 50, which would execute this line:

((EntityPlayer)player).inventory.setInventorySlotContents(1, new ItemStack(Minespresso.HotChocolate));

 

You may want to change the

sentID

to 1, or chance the line above to use

addItemStackToInventory

.

Still doesnt make a difference

[Bugzoo]

I found out that when I press the button, it glitches and keeps pressing the button. So if i press it ones it actually presses tons of times. do you know how to fix this, or why its happening

[Bugzoo]

Got it fixed. I was adding the buttons in the drawScreen method instead of the initGui