Jump to content
Search the Forums BEFORE making a new post.
Currently supported: All versions

[1.6.4] Problem With addItemStackToInventory

Bugzoo

By Bugzoo
in Modder Support

 Share

Recommended Posts

Bugzoo Diamond Finder

Bugzoo

Posted
Posted

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

 

Bugzoo Diamond Finder

Bugzoo

Posted
  • Author
Posted

Im sending a packet.

 

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

 

 

Elyon Diamond Finder

Elyon

Posted
Posted

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 Diamond Finder

Bugzoo

Posted
  • Author
Posted

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 Diamond Finder

Elyon

Posted
Posted

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 Diamond Finder

Bugzoo

Posted
  • Author
Posted

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 Diamond Finder

Elyon

Posted
Posted

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 Diamond Finder

Bugzoo

Posted
  • Author
Posted

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 Diamond Finder

Bugzoo

Posted
  • Author
Posted

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

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
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
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
Go to topic listing

Announcements



  • Recently Browsing

    • No registered users viewing this page.

  • Posts

    • TileEntity
      "The game crashed: rendering overlayError: java.lang.NullPointerException: Rendering overlay -1 "

      By TileEntity · Posted

      Add the crash-report or latest.log (logs-folder) with sites like https://mclo.gs/ and paste the link to it here  
    • TileEntity
      Modded Minecraft 1.20.1 Crash ("Player" is null error)

      By TileEntity · Posted

      Make a test without stevescarts
    • TileEntity
      Game freezes when certain chunks are loaded on modded world

      By TileEntity · Posted

      Hmm I have no idea
    • Whisperwind
      Server crash when launched

      By Whisperwind · Posted

      So i have a forge modded aternos server that worked just fine for a month untill today it suddenly crashes most of the time giving errors and idk which mod is causing the error or its smth else here is the crash log link https://mclo.gs/gGkzGKT
    • GreyM7
      Modded Minecraft 1.20.1 Crash ("Player" is null error)

      By GreyM7 · Posted

      Struggling to decipher a crash report I'm getting in a custom modpack I'm tinkering with. The crash happens on startup, but weirdly, only some of the time. It seems to be related to Steves Carts, but weirdly it only started happening recently, and I can't identify if another mod is conflicting, or why it is only happening some of the time:   java.lang.NullPointerException: Cannot invoke "net.minecraft.world.entity.player.Player.m_20202_()" because "player" is null at vswe.stevescarts.events.OverlayEventHandler.onRenderTick(OverlayEventHandler.java:24) ~[stevescarts-1.20.1-1.1.14.jar%23527!/:1.20.1-1.1.14] {re:classloading} at net.minecraftforge.eventbus.EventBus.doCastFilter(EventBus.java:260) ~[eventbus-6.0.5.jar%23127!/:?] {} at net.minecraftforge.eventbus.EventBus.lambda$addListener$11(EventBus.java:252) ~[eventbus-6.0.5.jar%23127!/:?] {} at net.minecraftforge.eventbus.EventBus.post(EventBus.java:315) ~[eventbus-6.0.5.jar%23127!/:?] {} at net.minecraftforge.eventbus.EventBus.post(EventBus.java:296) ~[eventbus-6.0.5.jar%23127!/:?] {} at net.minecraftforge.event.ForgeEventFactory.onRenderTickEnd(ForgeEventFactory.java:919) ~[forge-1.20.1-47.1.106-universal.jar%23581!/:?] {re:mixin,re:classloading,pl:mixin:APP:modernfix-forge.mixins.json:perf.potential_spawns_alloc.ForgeEventFactoryMixin,pl:mixin:A} at net.minecraft.client.Minecraft.m_91383_(Minecraft.java:1148) ~[client-1.20.1-20230612.114412-srg.jar%23576!/:?] {re:mixin,pl:accesstransformer:B,xf:fml:xaeroworldmap:xaero_wm_minecraft_runtick,xf:fml:xaeroworldmap:xaero_wm_minecraft_runtick_render_call,xf:fml:xaerominimap:xaero_minecraftclient,xf:fml:xaeroworldmap:xaero_wm_minecraftclient,pl:runtimedistcleaner:A,re:classloading,pl:accesstransformer:B,xf:fml:xaeroworldmap:xaero_wm_minecraft_runtick,xf:fml:xaeroworldmap:xaero_wm_minecraft_runtick_render_call,xf:fml:xaerominimap:xaero_minecraftclient,xf:fml:xaeroworldmap:xaero_wm_minecraftclient,pl:mixin:A,pl:runtimedistcleaner:A} at net.minecraft.client.Minecraft.m_91374_(Minecraft.java:718) ~[client-1.20.1-20230612.114412-srg.jar%23576!/:?] {re:mixin,pl:accesstransformer:B,xf:fml:xaeroworldmap:xaero_wm_minecraft_runtick,xf:fml:xaeroworldmap:xaero_wm_minecraft_runtick_render_call,xf:fml:xaerominimap:xaero_minecraftclient,xf:fml:xaeroworldmap:xaero_wm_minecraftclient,pl:runtimedistcleaner:A,re:classloading,pl:accesstransformer:B,xf:fml:xaeroworldmap:xaero_wm_minecraft_runtick,xf:fml:xaeroworldmap:xaero_wm_minecraft_runtick_render_call,xf:fml:xaerominimap:xaero_minecraftclient,xf:fml:xaeroworldmap:xaero_wm_minecraftclient,pl:mixin:A,pl:runtimedistcleaner:A} at net.minecraft.client.main.Main.main(Main.java:218) ~[minecraft-1.20.1-client.jar:?] {re:mixin,pl:runtimedistcleaner:A,re:classloading,pl:mixin:APP:flywheel.mixins.json:ClientMainMixin,pl:mixin:A,pl:runtimedistcleaner:A}     Including some extra info in case any of it is relevant^. Appreciate any ideas/advice!    

  • Topics

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.

  I accept