Jump to content

[Need really help] Change displayed GUI


JimiIT92

Recommended Posts

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

COvHreq.png

 

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?) :)

Don't blame me if i always ask for your help. I just want to learn to be better :)

Link to comment
Share on other sites

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

Development of Plugins [2012 - 2014] Development of Mods [2012 - Current]

Link to comment
Share on other sites

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)

Don't blame me if i always ask for your help. I just want to learn to be better :)

Link to comment
Share on other sites

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.

Development of Plugins [2012 - 2014] Development of Mods [2012 - Current]

Link to comment
Share on other sites

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

Don't blame me if i always ask for your help. I just want to learn to be better :)

Link to comment
Share on other sites

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

Don't blame me if i always ask for your help. I just want to learn to be better :)

Link to comment
Share on other sites

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

Don't blame me if i always ask for your help. I just want to learn to be better :)

Link to comment
Share on other sites

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 :/

Don't blame me if i always ask for your help. I just want to learn to be better :)

Link to comment
Share on other sites

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 :/

Don't blame me if i always ask for your help. I just want to learn to be better :)

Link to comment
Share on other sites

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 :/

Don't blame me if i always ask for your help. I just want to learn to be better :)

Link to comment
Share on other sites

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

Don't blame me if i always ask for your help. I just want to learn to be better :)

Link to comment
Share on other sites

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 :(

Don't blame me if i always ask for your help. I just want to learn to be better :)

Link to comment
Share on other sites

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

Don't blame me if i always ask for your help. I just want to learn to be better :)

Link to comment
Share on other sites

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

Don't blame me if i always ask for your help. I just want to learn to be better :)

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

Announcements



×
×
  • Create New...

Important Information

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