[Need really help] Change displayed GUI

JimiIT92 · September 25, 2015

How can i display a different GUI from a Vanilla Block? For example, i want to display this GUI when clicking on the crafting table

where the additional 4 slots should be automatically filled with the current armor. Is there any way to make this? And if there is, how can i make it (any links to a tutorial or a previous post like this?)

EverythingGames · September 25, 2015

Replace the crafting inventory with your own. Your own should have all the extra slots handled.

        @SubscribeEvent
public void onEvent(GuiOpenEvent event)
{
	if(event.gui instanceof TheInventoryYouWantToReplace)
	{		
		event.gui = new YourNewInventory(Minecraft.getMinecraft().thePlayer);
	}
}

JimiIT92 · September 26, 2015

I don't understand what the YourNewInventory class should looks like. I'm following this example to create a GUI

http://www.minecraftforge.net/wiki/Containers_and_GUIs

but i can't figure out why you pass only the player to the constructor (since the class takes a TileEntity as second parameter)

EverythingGames · September 26, 2015

On 9/26/2015 at 10:33 AM, JimiIT92 said:
I don't understand what the YourNewInventory class should looks like. I'm following this example to create a GUI
http://www.minecraftforge.net/wiki/Containers_and_GUIs
but i can't figure out why you pass only the player to the constructor (since the class takes a TileEntity as second parameter)

By all means my above post was an example of code taken from my mod Armed - I had to give an EntityPlayer instance on the client for my GUI. Use your own constructor.

JimiIT92 · September 26, 2015

Thanks for your help I've created the GUI and now I have this handler

package blaze.gui;

import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.entity.player.InventoryPlayer;
import net.minecraft.init.Blocks;
import net.minecraft.inventory.Container;
import net.minecraft.inventory.IInventory;
import net.minecraft.inventory.InventoryCraftResult;
import net.minecraft.inventory.InventoryCrafting;
import net.minecraft.inventory.Slot;
import net.minecraft.inventory.SlotCrafting;
import net.minecraft.item.ItemStack;
import net.minecraft.item.crafting.CraftingManager;
import net.minecraft.util.BlockPos;
import net.minecraft.world.World;

public class ContainerNewWorkbench extends Container
{
    /** The crafting matrix inventory (3x3). */
    public InventoryCrafting craftMatrix = new InventoryCrafting(this, 3, 3);
    public IInventory craftResult = new InventoryCraftResult();
    private World worldObj;
    private BlockPos pos;
    
    public ContainerNewWorkbench(InventoryPlayer playerInventory, World worldIn, BlockPos position)
    {
        this.worldObj = worldIn;
        this.pos = position;
        this.addSlotToContainer(new SlotCrafting(playerInventory.player, this.craftMatrix, this.craftResult, 0, 124, 35));
        int i;
        int j;

        /**
         * SLOTS CRAFTING 0-9
         */
        for (i = 0; i < 3; ++i)
        {
            for (j = 0; j < 3; ++j)
            {
                this.addSlotToContainer(new Slot(this.craftMatrix, j + i * 3, 30 + j * 18, 17 + i * 18));
            }
        }

        
        /**
         * SLOTS 9-35
         */
        for (i = 0; i < 3; ++i)
        {
            for (j = 0; j < 9; ++j)
            {
                this.addSlotToContainer(new Slot(playerInventory, j + i * 9 + 9, 8 + j * 18, 84 + i * 18));
            }
        }

        /**
         * SLOTS 0-9
         */
        for (i = 0; i < 9; ++i)
        {
            this.addSlotToContainer(new Slot(playerInventory, i, 8 + i * 18, 142));
        }
        
        /**
         * START ADDING THE ADDITIONAL 4 SLOTS, THEY WILL BE THE SLOTS FOR THE HELMET, THE CHESTPLATE AND THE LEGGINGS AND THE BOOTS.
         * THIS SLOTS MUST BE AUTOMATICALLY FILLED WITH THE CURRENT ARMOR WEARED, ALSO IN CRAFTING 
         * IF THE PLAYER CRAFT AN ARMOR THIS WILL AUTOMATICALLY BE PUTTED HERE IF EMPTY, SO THE PLAYER
         * WILL WEAR IT AS SOON AS HE CRAFTED IT
         * SLOTS 36-39
         */
        for (i = 0; i < 4; ++i)
        {
            this.addSlotToContainer(new Slot(playerInventory, 36 + i, 8 + 9 * 18, 84 + i * 18));
        }
        
        this.onCraftMatrixChanged(this.craftMatrix);
    }

    /**
     * Callback for when the crafting matrix is changed.
     */
    public void onCraftMatrixChanged(IInventory inventoryIn)
    {
        this.craftResult.setInventorySlotContents(0, CraftingManager.getInstance().findMatchingRecipe(this.craftMatrix, this.worldObj));
    }

    /**
     * Called when the container is closed.
     */
    public void onContainerClosed(EntityPlayer playerIn)
    {
        super.onContainerClosed(playerIn);

        if (!this.worldObj.isRemote)
        {
            for (int i = 0; i < 9; ++i)
            {
                ItemStack itemstack = this.craftMatrix.getStackInSlotOnClosing(i);

                if (itemstack != null)
                {
                    playerIn.dropPlayerItemWithRandomChoice(itemstack, false);
                }
            }
        }
    }

    public boolean canInteractWith(EntityPlayer playerIn)
    {
        return this.worldObj.getBlockState(this.pos).getBlock() != Blocks.crafting_table ? false : playerIn.getDistanceSq((double)this.pos.getX() + 0.5D, (double)this.pos.getY() + 0.5D, (double)this.pos.getZ() + 0.5D) <= 64.0D;
    }

    /**
     * Take a stack from the specified inventory slot.
     */
    public ItemStack transferStackInSlot(EntityPlayer playerIn, int index)
    {
        ItemStack itemstack = null;
        Slot slot = (Slot)this.inventorySlots.get(index);
        
        if (slot != null && slot.getHasStack())
        {
            ItemStack itemstack1 = slot.getStack();
            itemstack = itemstack1.copy();

            if (index == 0)
            {
                if (!this.mergeItemStack(itemstack1, 10, 51, true))
                {
                    return null;
                }

                slot.onSlotChange(itemstack1, itemstack);
            }
            else if (index >= 10 && index < 37)
            {
                if (!this.mergeItemStack(itemstack1, 37, 51, false))
                {
                    return null;
                }
            }
            else if (index >= 37 && index < 46)
            {
                if (!this.mergeItemStack(itemstack1, 10, 37, false))
                {
                    return null;
                }
            }
            else if (index >= 47 && index < 51)
            {
                if (!this.mergeItemStack(itemstack1, 10, 47, false))
                {
                    return null;
                }
            }
            else if (!this.mergeItemStack(itemstack1, 10, 51, false))
            {
                return null;
            }

            if (itemstack1.stackSize == 0)
            {
                slot.putStack((ItemStack)null);
            }
            else
            {
                slot.onSlotChanged();
            }

            if (itemstack1.stackSize == itemstack.stackSize)
            {
                return null;
            }

            slot.onPickupFromSlot(playerIn, itemstack1);
        }

        return itemstack;
    }

    /**
     * Called to determine if the current slot is valid for the stack merging (double-click) code. The stack passed in
     * is null for the initial slot that was double-clicked.
     */
    public boolean canMergeSlot(ItemStack itemstack, Slot slot)
    {
        return slot.inventory != this.craftResult && super.canMergeSlot(itemstack, slot);
    }
}

But there are a lot of problems with the additional slots. The 4 new slots should be automatically filled with the current armor weared. Actually they do but reversed (video below). Also i want that in that 4 slots only armor pieces can be putted (or everything else that can be weared (like skulls or pumpkins). Also if i craft an armor piece this should go in the correspective new slot, so the player will instantly wear it How can i do that?

Another serious problem is that if i try to get something from that slot it disappear, gone forever, but if i try to put in ther it duplicates

https://www.youtube.com/watch?v=cojKwg4i1T8&feature=youtu.be

JimiIT92 · September 26, 2015

Ok, quick update, if i click on the extra slot it gives me this error

[20:33:38] [server thread/FATAL] [FML]: Exception caught executing FutureTask: java.util.concurrent.ExecutionException: java.lang.IndexOutOfBoundsException: Index: 48, Size: 46
java.util.concurrent.ExecutionException: java.lang.IndexOutOfBoundsException: Index: 48, Size: 46
at java.util.concurrent.FutureTask.report(Unknown Source) ~[?:1.8.0_51]
at java.util.concurrent.FutureTask.get(Unknown Source) ~[?:1.8.0_51]
at net.minecraftforge.fml.common.FMLCommonHandler.callFuture(FMLCommonHandler.java:710) [FMLCommonHandler.class:?]
at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:726) [MinecraftServer.class:?]
at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:669) [MinecraftServer.class:?]
at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:171) [integratedServer.class:?]
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:540) [MinecraftServer.class:?]
at java.lang.Thread.run(Unknown Source) [?:1.8.0_51]
Caused by: java.lang.IndexOutOfBoundsException: Index: 48, Size: 46
at java.util.ArrayList.rangeCheck(Unknown Source) ~[?:1.8.0_51]
at java.util.ArrayList.get(Unknown Source) ~[?:1.8.0_51]
at net.minecraft.inventory.Container.slotClick(Container.java:473) ~[Container.class:?]
at net.minecraft.network.NetHandlerPlayServer.processClickWindow(NetHandlerPlayServer.java:1039) ~[NetHandlerPlayServer.class:?]
at net.minecraft.network.play.client.C0EPacketClickWindow.processPacket(C0EPacketClickWindow.java:46) ~[C0EPacketClickWindow.class:?]
at net.minecraft.network.play.client.C0EPacketClickWindow.processPacket(C0EPacketClickWindow.java:110) ~[C0EPacketClickWindow.class:?]
at net.minecraft.network.PacketThreadUtil$1.run(PacketThreadUtil.java:24) ~[PacketThreadUtil$1.class:?]
at java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source) ~[?:1.8.0_51]
at java.util.concurrent.FutureTask.run(Unknown Source) ~[?:1.8.0_51]
at net.minecraftforge.fml.common.FMLCommonHandler.callFuture(FMLCommonHandler.java:709) ~[FMLCommonHandler.class:?]
... 5 more

Failender · September 26, 2015

seriously u should start to learn to look at vanilla and learn from it.

look how vanilla renders the player inventory maybe and then add 4 slots?

JimiIT92 · September 27, 2015

I've looked into vanilla inventory container class, so now the extra slot works as normal armor slots The only problem is that when i click in it it gives an outOfBoundException, and also the item disappear (but when reopening the GUI is in inventory)

So this is the Container class right now

package blaze.container;

import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.entity.player.InventoryPlayer;
import net.minecraft.init.Blocks;
import net.minecraft.inventory.Container;
import net.minecraft.inventory.IInventory;
import net.minecraft.inventory.InventoryCraftResult;
import net.minecraft.inventory.InventoryCrafting;
import net.minecraft.inventory.Slot;
import net.minecraft.inventory.SlotCrafting;
import net.minecraft.item.ItemArmor;
import net.minecraft.item.ItemStack;
import net.minecraft.item.crafting.CraftingManager;
import net.minecraft.util.BlockPos;
import net.minecraft.world.World;
import net.minecraftforge.fml.relauncher.Side;
import net.minecraftforge.fml.relauncher.SideOnly;

public class ContainerNewWorkbench extends Container
{
    /** The crafting matrix inventory (3x3). */
    public InventoryCrafting craftMatrix = new InventoryCrafting(this, 3, 3);
    public IInventory craftResult = new InventoryCraftResult();
    private World worldObj;
    private BlockPos pos;

    public ContainerNewWorkbench(final InventoryPlayer playerInventory, World worldIn, BlockPos position)
    {
        this.worldObj = worldIn;
        this.pos = position;
        this.addSlotToContainer(new SlotCrafting(playerInventory.player, this.craftMatrix, this.craftResult, 0, 124, 35));
        int i;
        int j;

        for (i = 0; i < 3; ++i)
        {
            for (j = 0; j < 3; ++j)
            {
                this.addSlotToContainer(new Slot(this.craftMatrix, j + i * 3, 30 + j * 18, 17 + i * 18));
            }
        }

        for (i = 0; i < 3; ++i)
        {
            for (j = 0; j < 9; ++j)
            {
                this.addSlotToContainer(new Slot(playerInventory, j + i * 9 + 9, 8 + j * 18, 84 + i * 18));
            }
        }

        for (i = 0; i < 9; ++i)
        {
            this.addSlotToContainer(new Slot(playerInventory, i, 8 + i * 18, 142));
        }

        for(i = 0; i < 4; i++)
        {
        	final int k = i;
        	this.addSlotToContainer(new Slot(playerInventory, playerInventory.getSizeInventory() - 1 - i, 8 + 9 * 18, 84 + i * 18)
            {
                /**
                 * Returns the maximum stack size for a given slot (usually the same as getInventoryStackLimit(), but 1
                 * in the case of armor slots)
                 */
                public int getSlotStackLimit()
                {
                    return 1;
                }
                /**
                 * Check if the stack is a valid item for this slot. Always true beside for the armor slots.
                 */
                public boolean isItemValid(ItemStack stack)
                {
                    if (stack == null) return false;
                    return stack.getItem().isValidArmor(stack, k, playerInventory.player);
                }
                @SideOnly(Side.CLIENT)
                public String getSlotTexture()
                {
                    return ItemArmor.EMPTY_SLOT_NAMES[k];
                }
            });
        }
      
        this.onCraftMatrixChanged(this.craftMatrix);
    }

    /**
     * Callback for when the crafting matrix is changed.
     */
    public void onCraftMatrixChanged(IInventory inventoryIn)
    {
        this.craftResult.setInventorySlotContents(0, CraftingManager.getInstance().findMatchingRecipe(this.craftMatrix, this.worldObj));
    }

    /**
     * Called when the container is closed.
     */
    public void onContainerClosed(EntityPlayer playerIn)
    {
        super.onContainerClosed(playerIn);

        if (!this.worldObj.isRemote)
        {
            for (int i = 0; i < 9; ++i)
            {
                ItemStack itemstack = this.craftMatrix.getStackInSlotOnClosing(i);

                if (itemstack != null)
                {
                    playerIn.dropPlayerItemWithRandomChoice(itemstack, false);
                }
            }
        }
    }

    public boolean canInteractWith(EntityPlayer playerIn)
    {
        return this.worldObj.getBlockState(this.pos).getBlock() != Blocks.crafting_table ? false : playerIn.getDistanceSq((double)this.pos.getX() + 0.5D, (double)this.pos.getY() + 0.5D, (double)this.pos.getZ() + 0.5D) <= 64.0D;
    }

    /**
     * Take a stack from the specified inventory slot.
     */
    public ItemStack transferStackInSlot(EntityPlayer playerIn, int index)
    {
        ItemStack itemstack = null;
        Slot slot = (Slot)this.inventorySlots.get(index);

        if (slot != null && slot.getHasStack())
        {
            ItemStack itemstack1 = slot.getStack();
            itemstack = itemstack1.copy();

            if (index == 0)
            {
                if (!this.mergeItemStack(itemstack1, 10, 46, true))
                {
                    return null;
                }

                slot.onSlotChange(itemstack1, itemstack);
            }
            else if (index >= 10 && index < 37)
            {
                if (!this.mergeItemStack(itemstack1, 37, 46, false))
                {
                    return null;
                }
            }
            else if (index >= 37 && index < 46)
            {
                if (!this.mergeItemStack(itemstack1, 10, 37, false))
                {
                    return null;
                }
            }
            else if (!this.mergeItemStack(itemstack1, 10, 46, false))
            {
                return null;
            }
            else if (itemstack.getItem() instanceof ItemArmor && !((Slot)this.inventorySlots.get(36 + ((ItemArmor)itemstack.getItem()).armorType)).getHasStack())
            {
                int j = 36 + ((ItemArmor)itemstack.getItem()).armorType;

                if (!this.mergeItemStack(itemstack1, j, j + 1, false))
                {
                    return null;
                }
            }

            if (itemstack1.stackSize == 0)
            {
                slot.putStack((ItemStack)null);
            }
            else
            {
                slot.onSlotChanged();
            }

            if (itemstack1.stackSize == itemstack.stackSize)
            {
                return null;
            }

            slot.onPickupFromSlot(playerIn, itemstack1);
        }

        return itemstack;
    }

    /**
     * Called to determine if the current slot is valid for the stack merging (double-click) code. The stack passed in
     * is null for the initial slot that was double-clicked.
     */
    public boolean canMergeSlot(ItemStack itemstack, Slot slot)
    {
        return slot.inventory != this.craftResult && super.canMergeSlot(itemstack, slot);
    }
}

And this is the behavior described above

https://www.youtube.com/watch?v=X8t39b5utVI

and this is the error that it gives in eclipse

[15:16:38] [server thread/FATAL] [FML]: Exception caught executing FutureTask: java.util.concurrent.ExecutionException: java.lang.IndexOutOfBoundsException: Index: 46, Size: 46
java.util.concurrent.ExecutionException: java.lang.IndexOutOfBoundsException: Index: 46, Size: 46
at java.util.concurrent.FutureTask.report(Unknown Source) ~[?:1.8.0_51]
at java.util.concurrent.FutureTask.get(Unknown Source) ~[?:1.8.0_51]
at net.minecraftforge.fml.common.FMLCommonHandler.callFuture(FMLCommonHandler.java:710) [FMLCommonHandler.class:?]
at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:726) [MinecraftServer.class:?]
at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:669) [MinecraftServer.class:?]
at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:171) [integratedServer.class:?]
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:540) [MinecraftServer.class:?]
at java.lang.Thread.run(Unknown Source) [?:1.8.0_51]
Caused by: java.lang.IndexOutOfBoundsException: Index: 46, Size: 46
at java.util.ArrayList.rangeCheck(Unknown Source) ~[?:1.8.0_51]
at java.util.ArrayList.get(Unknown Source) ~[?:1.8.0_51]
at net.minecraft.inventory.Container.slotClick(Container.java:473) ~[Container.class:?]
at net.minecraft.network.NetHandlerPlayServer.processClickWindow(NetHandlerPlayServer.java:1039) ~[NetHandlerPlayServer.class:?]
at net.minecraft.network.play.client.C0EPacketClickWindow.processPacket(C0EPacketClickWindow.java:46) ~[C0EPacketClickWindow.class:?]
at net.minecraft.network.play.client.C0EPacketClickWindow.processPacket(C0EPacketClickWindow.java:110) ~[C0EPacketClickWindow.class:?]
at net.minecraft.network.PacketThreadUtil$1.run(PacketThreadUtil.java:24) ~[PacketThreadUtil$1.class:?]
at java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source) ~[?:1.8.0_51]
at java.util.concurrent.FutureTask.run(Unknown Source) ~[?:1.8.0_51]
at net.minecraftforge.fml.common.FMLCommonHandler.callFuture(FMLCommonHandler.java:709) ~[FMLCommonHandler.class:?]
... 5 more

JimiIT92 · September 28, 2015

Any idea? I really don't understand why it gives me the out of bound exception since the inv size is greater than the selected slot

Failender · September 28, 2015

GuiOpenEvent is fired Client-Only. Which means If you want to open a different Container u need to send a packet and let the server open the gui (because the server will inform the client)

JimiIT92 · September 28, 2015

How can i send that packet? I'm asking you this since it's the first time i work with GUI and events in general The GUI works right now, except for the 4 extra slots, they accept only respective armor items but i can't pick up anything from that slots, it disappear and it gives me that error. It's like that slots doesn't exist for the game

Failender · September 28, 2015

http://www.minecraftforge.net/forum/index.php/topic,20135.0.html

JimiIT92 · September 28, 2015

Thanks I'll check it

JimiIT92 · September 28, 2015

I've tried doing this in the main mod init method

NetworkRegistry.INSTANCE.registerGuiHandler(this, new GUINewWorkbenchHandler());

With this handler

package blaze.gui;

import blaze.container.ContainerNewWorkbench;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.util.BlockPos;
import net.minecraft.world.World;
import net.minecraftforge.fml.common.network.IGuiHandler;

public class GUINewWorkbenchHandler implements IGuiHandler {
//returns an instance of the Container you made earlier
@Override
public Object getServerGuiElement(int id, EntityPlayer player, World world, int x, int y, int z) {
	System.out.println("SERVER: " + id);
	return new ContainerNewWorkbench(player.inventory, world, new BlockPos(x, y , z));
}

//returns an instance of the Gui you made earlier
@Override
public Object getClientGuiElement(int id, EntityPlayer player, World world, int x, int y, int z) {
	System.out.println("CLIENT: " + id);
	return new GUINewWorkbench(player.inventory, world, new BlockPos(x, y , z));	
}
}

But it doesn't work Also the handler code is never called

Failender · September 28, 2015

how are u opening the gui

JimiIT92 · September 28, 2015

I'm opening the GUI from this Event Handler

@SubscribeEvent
public void onEvent(GuiOpenEvent event)
{
	if(event.gui instanceof GuiCrafting)
	{		
		event.gui = new GUINewWorkbench(Minecraft.getMinecraft().thePlayer.inventory, Minecraft.getMinecraft().theWorld);
	}
}

So basically right-clicking on the crafting table will open my custom GUI (and it actually does). This could be the reason why the GUIHandler is never called i guess

Failender · September 28, 2015

On 9/28/2015 at 9:29 AM, Failender said:

GuiOpenEvent is fired Client-Only. Which means If you want to open a different Container u need to send a packet and let the server open the gui (because the server will inform the client)

JimiIT92 · September 28, 2015

So i need to do this?

BL.network.sendToServer(new NetworkMessage("craftingTable"));

Where NetworkMessage is my IMessage class

package blaze.proxy;

import io.netty.buffer.ByteBuf;
import net.minecraft.util.IThreadListener;
import net.minecraft.world.WorldServer;
import net.minecraftforge.fml.common.network.ByteBufUtils;
import net.minecraftforge.fml.common.network.simpleimpl.IMessage;
import net.minecraftforge.fml.common.network.simpleimpl.IMessageHandler;
import net.minecraftforge.fml.common.network.simpleimpl.MessageContext;

public class NetworkMessage implements IMessage {
    
    private String text;

    public NetworkMessage() { }

    public NetworkMessage(String text) {
        this.text = text;
    }

    @Override
    public void fromBytes(ByteBuf buf) {
        text = ByteBufUtils.readUTF8String(buf);
    }

    @Override
    public void toBytes(ByteBuf buf) {
        ByteBufUtils.writeUTF8String(buf, text);
    }

    public static class Handler implements IMessageHandler<NetworkMessage, IMessage> {

        @Override
        public IMessage onMessage(final NetworkMessage message, final MessageContext ctx) {
            IThreadListener mainThread = (WorldServer) ctx.getServerHandler().playerEntity.worldObj;
            mainThread.addScheduledTask(new Runnable() {
                @Override
                public void run() {
                    System.out.println(String.format("Received %s from %s", message.text, ctx.getServerHandler().playerEntity.getDisplayName()));
                }
            });
            return null;
        }
    }
}

and it's registered like this

network = NetworkRegistry.INSTANCE.newSimpleChannel("BL_Channel");
	network.registerMessage(NetworkMessage.Handler.class, NetworkMessage.class, 0, Side.SERVER);

I've tried this but nothing changed

EDIT: in the EventReceiver class

@SubscribeEvent
public void onEvent(GuiOpenEvent event)
{
	if(event.gui instanceof GuiCrafting)
	{	
		BL.network.sendToServer(new NetworkMessage("craftingTable"));
		event.gui = new GUINewWorkbench(Minecraft.getMinecraft().thePlayer.inventory, Minecraft.getMinecraft().theWorld);
	}
}

Failender · September 28, 2015

seriously the tutorial is straight forward. fix it urself

JimiIT92 · September 28, 2015

The tutorial says:

call NetworkRegistry.INSTANCE.newSimpleChannel("myChannel"). Store the result in a static field in your Mod class. DONE
register your Packet, call registerMessage(MyMessageHandler.class, MyMessage.class, packetID, receivingSide) on the wrapper you obtained. DONE
Implementing the packet class and the packet handler. DONE, it's the same class of the tutorial
send IMessage instances to various targets. DONE. If i have a println in the run method it prints

The only thing i really don't understand is this

The IMessageHandler simply requires one method, onMessage.
Do whatever you want your packet to do in this method.

Since i've never worked before with packets and all this stuff i don't know what to place in that method. A new instance of the container? A new one of the GUIHandler? I don't really know

Failender · September 28, 2015

show thje whole thing where u send the package. i guess its inside ur event handler. have u registered the event handler? show that

JimiIT92 · September 28, 2015

Main mod file (where i register the packet handler)

public static SimpleNetworkWrapper network;

@EventHandler
public void preInit(FMLPreInitializationEvent event)
{
	network = NetworkRegistry.INSTANCE.newSimpleChannel("BL_Channel");
	network.registerMessage(NetworkMessage.Handler.class, NetworkMessage.class, 0, Side.SERVER);
...
}

In the ClientProxy

MinecraftForge.EVENT_BUS.register(new EventReceiver());

The NetworkMessage class (with Handler inner class)

package blaze.proxy;

import io.netty.buffer.ByteBuf;
import net.minecraft.util.IThreadListener;
import net.minecraft.world.WorldServer;
import net.minecraftforge.fml.common.network.ByteBufUtils;
import net.minecraftforge.fml.common.network.simpleimpl.IMessage;
import net.minecraftforge.fml.common.network.simpleimpl.IMessageHandler;
import net.minecraftforge.fml.common.network.simpleimpl.MessageContext;

public class NetworkMessage implements IMessage {
    
    private String text;

    public NetworkMessage() { }

    public NetworkMessage(String text) {
        this.text = text;
    }

    @Override
    public void fromBytes(ByteBuf buf) {
        text = ByteBufUtils.readUTF8String(buf);
    }

    @Override
    public void toBytes(ByteBuf buf) {
        ByteBufUtils.writeUTF8String(buf, text);
    }

    public static class Handler implements IMessageHandler<NetworkMessage, IMessage> {

        @Override
        public IMessage onMessage(final NetworkMessage message, final MessageContext ctx) {
            IThreadListener mainThread = (WorldServer) ctx.getServerHandler().playerEntity.worldObj;
            mainThread.addScheduledTask(new Runnable() {
                @Override
                public void run() {
                    //TODO What's going on here??
                }
            });
            return null;
        }
    }
}

The container class

package blaze.gui.container;

import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.entity.player.InventoryPlayer;
import net.minecraft.init.Blocks;
import net.minecraft.inventory.Container;
import net.minecraft.inventory.IInventory;
import net.minecraft.inventory.InventoryCraftResult;
import net.minecraft.inventory.InventoryCrafting;
import net.minecraft.inventory.Slot;
import net.minecraft.inventory.SlotCrafting;
import net.minecraft.item.ItemArmor;
import net.minecraft.item.ItemStack;
import net.minecraft.item.crafting.CraftingManager;
import net.minecraft.util.BlockPos;
import net.minecraft.world.World;
import net.minecraftforge.fml.relauncher.Side;
import net.minecraftforge.fml.relauncher.SideOnly;

public class ContainerNewWorkbench extends Container
{
    /** The crafting matrix inventory (3x3). */
    public InventoryCrafting craftMatrix = new InventoryCrafting(this, 3, 3);
    public IInventory craftResult = new InventoryCraftResult();
    private World worldObj;
    private BlockPos pos;

    public ContainerNewWorkbench(final InventoryPlayer playerInventory, World worldIn, BlockPos position)
    {
        this.worldObj = worldIn;
        this.pos = position;
        this.addSlotToContainer(new SlotCrafting(playerInventory.player, this.craftMatrix, this.craftResult, 0, 124, 35));
        int i;
        int j;

        /**
         * SLOT 0-8
         */
        for (i = 0; i < 3; ++i)
        {
            for (j = 0; j < 3; ++j)
            {
                this.addSlotToContainer(new Slot(this.craftMatrix, j + i * 3, 30 + j * 18, 17 + i * 18));
            }
        }

        /**
         * SLOT 9-35
         */
        for (i = 0; i < 3; ++i)
        {
            for (j = 0; j < 9; ++j)
            {
                this.addSlotToContainer(new Slot(playerInventory, j + i * 9 + 9, 8 + j * 18, 84 + i * 18));
            }
        }

        /**
         * SLOT 36-44
         */
        for (i = 0; i < 9; ++i)
        {
            this.addSlotToContainer(new Slot(playerInventory, i, 8 + i * 18, 142));
        }

        /**
         * 45-48
         */
        for(i = 0; i < 4; i++)
        {
        	final int k = i;
        	this.addSlotToContainer(new Slot(playerInventory, playerInventory.getSizeInventory() - 1 - i, 8 + 9 * 18, 84 + i * 18)
            {
                /**
                 * Returns the maximum stack size for a given slot (usually the same as getInventoryStackLimit(), but 1
                 * in the case of armor slots)
                 */
                public int getSlotStackLimit()
                {
                    return 1;
                }
                /**
                 * Check if the stack is a valid item for this slot. Always true beside for the armor slots.
                 */
                public boolean isItemValid(ItemStack stack)
                {
                    if (stack == null) return false;
                    return stack.getItem().isValidArmor(stack, k, playerInventory.player);
                }
                @SideOnly(Side.CLIENT)
                public String getSlotTexture()
                {
                    return ItemArmor.EMPTY_SLOT_NAMES[k];
                }
            });
        }
      
        this.onCraftMatrixChanged(this.craftMatrix);
    }

    /**
     * Callback for when the crafting matrix is changed.
     */
    public void onCraftMatrixChanged(IInventory inventoryIn)
    {
        this.craftResult.setInventorySlotContents(0, CraftingManager.getInstance().findMatchingRecipe(this.craftMatrix, this.worldObj));
    }

    /**
     * Called when the container is closed.
     */
    public void onContainerClosed(EntityPlayer playerIn)
    {
        super.onContainerClosed(playerIn);

        if (!this.worldObj.isRemote)
        {
            for (int i = 0; i < 9; ++i)
            {
                ItemStack itemstack = this.craftMatrix.getStackInSlotOnClosing(i);

                if (itemstack != null)
                {
                    playerIn.dropPlayerItemWithRandomChoice(itemstack, false);
                }
            }
        }
    }

    public boolean canInteractWith(EntityPlayer playerIn)
    {
        return this.worldObj.getBlockState(this.pos).getBlock() != Blocks.crafting_table ? false : playerIn.getDistanceSq((double)this.pos.getX() + 0.5D, (double)this.pos.getY() + 0.5D, (double)this.pos.getZ() + 0.5D) <= 64.0D;
    }

    /**
     * Take a stack from the specified inventory slot.
     */
    public ItemStack transferStackInSlot(EntityPlayer playerIn, int index)
    {
        ItemStack itemstack = null;
        Slot slot = (Slot)this.inventorySlots.get(index);

        if (slot != null && slot.getHasStack())
        {
            ItemStack itemstack1 = slot.getStack();
            itemstack = itemstack1.copy();

            /**
             * FROM THE CRAFT RESULT SLOT
             */
            if (index == 0)
            {
                if (!this.mergeItemStack(itemstack1, 10, 46, true))
                {
                    return null;
                }

                slot.onSlotChange(itemstack1, itemstack);
            }
            /**
             * FROM THE INV
             */
            else if (index >= 10 && index < 37)
            {
            	System.out.println("INV");
                if (!this.mergeItemStack(itemstack1, 37, 46, false))
                {
                    return null;
                }
            }
            /**
             * FROM THE HOTBAR
             */
            else if (index >= 37 && index < 46)
            {
            	System.out.println("HOTBAR");
                if (!this.mergeItemStack(itemstack1, 10, 37, false))
                {
                    return null;
                }
            }
            /**
             * FROM THE ARMOR
             */
            else if ((index >= 47 && index < 50) && itemstack.getItem() instanceof ItemArmor && !((Slot)this.inventorySlots.get(47 + ((ItemArmor)itemstack.getItem()).armorType)).getHasStack())
            {
            	System.out.println("ARMOR");
                int j = 47 + ((ItemArmor)itemstack.getItem()).armorType;

                if (!this.mergeItemStack(itemstack1, j, j + 1, false))
                {
                    return null;
                }
            }
            /**
             * FROM CRAFTING GRID
             */
            else if (!this.mergeItemStack(itemstack1, 10, 46, false))
            {
            	System.out.println("GRID!");
                return null;
            }
            

            if (itemstack1.stackSize == 0)
            {
                slot.putStack((ItemStack)null);
            }
            else
            {
                slot.onSlotChanged();
            }

            if (itemstack1.stackSize == itemstack.stackSize)
            {
                return null;
            }

            slot.onPickupFromSlot(playerIn, itemstack1);
        }

        return itemstack;
    }

    /**
     * Called to determine if the current slot is valid for the stack merging (double-click) code. The stack passed in
     * is null for the initial slot that was double-clicked.
     */
    public boolean canMergeSlot(ItemStack itemstack, Slot slot)
    {
        return slot.inventory != this.craftResult && super.canMergeSlot(itemstack, slot);
    }
}

The GUI Class

package blaze.gui;

import blaze.gui.container.ContainerNewWorkbench;
import net.minecraft.client.gui.inventory.GuiContainer;
import net.minecraft.client.renderer.GlStateManager;
import net.minecraft.client.resources.I18n;
import net.minecraft.entity.player.InventoryPlayer;
import net.minecraft.util.BlockPos;
import net.minecraft.util.ResourceLocation;
import net.minecraft.world.World;
import net.minecraftforge.fml.relauncher.Side;
import net.minecraftforge.fml.relauncher.SideOnly;

@SideOnly(Side.CLIENT)
public class GUINewWorkbench extends GuiContainer
{
    private static final ResourceLocation craftingTableGuiTextures = new ResourceLocation("blaze:textures/gui/container/crafting_table.png");

    public GUINewWorkbench(InventoryPlayer playerInv, World worldIn)
    {
        this(playerInv, worldIn, BlockPos.ORIGIN);
    }

    public GUINewWorkbench(InventoryPlayer playerInv, World worldIn, BlockPos blockPosition)
    {
        super(new ContainerNewWorkbench(playerInv, worldIn, blockPosition));
    }

    /**
     * Draw the foreground layer for the GuiContainer (everything in front of the items). Args : mouseX, mouseY
     */
    protected void drawGuiContainerForegroundLayer(int mouseX, int mouseY)
    {
        this.fontRendererObj.drawString(I18n.format("container.crafting", new Object[0]), 28, 6, 4210752);
        this.fontRendererObj.drawString(I18n.format("container.inventory", new Object[0]), 8, this.ySize - 96 + 2, 4210752);
    }

    /**
     * Args : renderPartialTicks, mouseX, mouseY
     */
    protected void drawGuiContainerBackgroundLayer(float partialTicks, int mouseX, int mouseY)
    {
        GlStateManager.color(1.0F, 1.0F, 1.0F, 1.0F);
        this.mc.getTextureManager().bindTexture(craftingTableGuiTextures);
        int k = (this.width - this.xSize) / 2;
        int l = (this.height - this.ySize) / 2;
        this.drawTexturedModalRect(k, l, 0, 0, 194, this.ySize);
    }
}

The GUIHandler Class

package blaze.gui;

import blaze.gui.container.ContainerNewWorkbench;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.util.BlockPos;
import net.minecraft.world.World;
import net.minecraftforge.fml.common.network.IGuiHandler;

public class GUINewWorkbenchHandler implements IGuiHandler {

public GUINewWorkbenchHandler() {
	// TODO Auto-generated constructor stub
}
//returns an instance of the Container you made earlier
@Override
public Object getServerGuiElement(int id, EntityPlayer player, World world, int x, int y, int z) {
	System.out.println("SERVER: " + id);
	return new ContainerNewWorkbench(player.inventory, world, new BlockPos(x, y , z));
}

//returns an instance of the Gui you made earlier
@Override
public Object getClientGuiElement(int id, EntityPlayer player, World world, int x, int y, int z) {
	System.out.println("CLIENT: " + id);
	return new GUINewWorkbench(player.inventory, world, new BlockPos(x, y , z));	
}
}

If i missed something please tell me and i will provide any missing class

EDIT: in the EventReceiver class

@SubscribeEvent
public void onEvent(GuiOpenEvent event)
{
	if(event.gui instanceof GuiCrafting)
	{	
		BL.network.sendToServer(new NetworkMessage("craftingTable"));
		event.gui = new GUINewWorkbench(Minecraft.getMinecraft().thePlayer.inventory, Minecraft.getMinecraft().theWorld);
	}
}

Failender · September 28, 2015

are u calling the clientproxy codeß

JimiIT92 · September 28, 2015

Yes, i'm calling from the init method by doing this

@EventHandler
public void init(FMLInitializationEvent event)
{
	if(event.getSide() == Side.CLIENT)
	{
		RenderItem renderItem = Minecraft.getMinecraft().getRenderItem();

                        ....

		proxy.registerRenderThings(renderItem);
	}
}

Failender · September 28, 2015

u should consider reading a tutorial about the proxies and why u need to use them. the simple reason to use proxies is to NOT use event.getSide()

Sign In

[Need really help] Change displayed GUI

Recommended Posts

JimiIT92

EverythingGames

JimiIT92

EverythingGames

JimiIT92

JimiIT92

Failender

JimiIT92

JimiIT92

Failender

JimiIT92

Failender

JimiIT92

JimiIT92

Failender

JimiIT92

Failender

JimiIT92

Failender

JimiIT92

Failender

JimiIT92

Failender

JimiIT92

Failender

Join the conversation

Announcements

Recently Browsing

Posts

Topics

Who's Online (See full list)

Browse

Activity

Important Information