Jump to content

Recommended Posts

Posted

So I've been learning Tile entities and adding gui to them. I've created a tile entity with a simple gui but when i right click the block. Nothing happens.. Its a fare bit of code. But I will include some. I can post more if needed.

 

Main class:

package com.duckcreeper.steelcraft;


import net.minecraft.init.Items;
import net.minecraft.item.ItemStack;
import net.minecraftforge.fml.common.Mod;
import net.minecraftforge.fml.common.Mod.EventHandler;
import net.minecraftforge.fml.common.Mod.Instance;
import net.minecraftforge.fml.common.SidedProxy;
import net.minecraftforge.fml.common.event.FMLInitializationEvent;
import net.minecraftforge.fml.common.event.FMLPostInitializationEvent;
import net.minecraftforge.fml.common.event.FMLPreInitializationEvent;
import net.minecraftforge.fml.common.network.NetworkRegistry;
import net.minecraftforge.fml.common.registry.GameRegistry;

import com.duckcreeper.steelcraft.gui.SCGuiHandler;
import com.duckcreeper.steelcraft.init.SteelAchievements;
import com.duckcreeper.steelcraft.init.SteelCraftBlocks;
import com.duckcreeper.steelcraft.init.SteelCraftItems;
import com.duckcreeper.steelcraft.init.SteelTileEntities;
import com.duckcreeper.steelcraft.proxy.CommonProxy;
import com.duckcreeper.steelcraft.tileentity.SCTileEntities;


@Mod(modid = Reference.MOD_ID, name = Reference.MOD_NAME, version = Reference.VERISION)
public class SteelMain {

@SidedProxy(clientSide = Reference.CLIENT_PROXY_CLASS, serverSide = Reference.SERVER_PROXY_CLASS)
public static CommonProxy proxy;

public static final SteelTab tabSteel = new SteelTab("tabSteel");

    @Instance
    public static SteelMain instance = new SteelMain();

@EventHandler
public void preInit(FMLPreInitializationEvent event)
{
	SteelCraftBlocks.init();
	SteelCraftBlocks.register();
	SteelCraftItems.init();
	SteelCraftItems.register();	

	/** Achievement Registering */
	SteelAchievements.loadAchievements();
	SteelAchievements.registerPage();

	/** Tile Entites*/
	SCTileEntities.init();

}

@EventHandler
public void init(FMLInitializationEvent event)
{
	proxy.registerRenders();
	GameRegistry.addRecipe(new ItemStack(SteelCraftBlocks.block_turbine, 4),
		"SIS", 
            "IBI",
            "SIS", 'I', Items.iron_ingot, 'S', SteelCraftItems.steel_ingot, 'B', new ItemStack(Items.dye, 1, 15) ); 

	GameRegistry.addRecipe(new ItemStack(SteelCraftItems.duck_bill_float),
		"SIR",
		"SIS",
		" S ", 'S', SteelCraftItems.steel_ingot, 'I', Items.iron_ingot, 'R', new ItemStack(Items.dye, 1, 1) );

	/** GUI Registering*/
	NetworkRegistry.INSTANCE.registerGuiHandler(this, new SCGuiHandler());

	/** TileEntity Registering */
	SteelTileEntities.register();

}
@EventHandler
public void postInit(FMLPostInitializationEvent event)
{

}

}

 

Tile Entity

package com.duckcreeper.steelcraft.tileentity;


import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.entity.player.InventoryPlayer;
import net.minecraft.inventory.Container;
import net.minecraft.inventory.ISidedInventory;
import net.minecraft.item.ItemFood;
import net.minecraft.item.ItemStack;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.nbt.NBTTagList;
import net.minecraft.network.NetworkManager;
import net.minecraft.network.Packet;
import net.minecraft.network.play.server.S35PacketUpdateTileEntity;
import net.minecraft.tileentity.TileEntityLockable;
import net.minecraft.util.ChatComponentText;
import net.minecraft.util.EnumFacing;
import net.minecraft.util.IChatComponent;

import com.duckcreeper.steelcraft.gui.containers.ContainerSCTileEntity;

public class SCTileEntity extends TileEntityLockable implements ISidedInventory
{
private static final int[] slots = new int[] { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14 };

private ItemStack[] inventory = new ItemStack[15];
private String customName;

public int getSizeInventory()
{
	return 15;
}

@Override
public ItemStack getStackInSlot(int par1)
{
	return this.inventory[par1];
}

@Override
public ItemStack decrStackSize(int par1, int par2)
{
	if (this.inventory[par1] != null)
	{
		ItemStack var3;

		if (this.inventory[par1].stackSize <= par2)
		{
			var3 = this.inventory[par1];
			this.inventory[par1] = null;
			this.markDirty();
			return var3;
		}
		var3 = this.inventory[par1].splitStack(par2);

		if (this.inventory[par1].stackSize == 0)
		{
			this.inventory[par1] = null;
		}

		this.markDirty();
		return var3;
	}
	return null;
}

@Override
public ItemStack getStackInSlotOnClosing(int par1)
{
	if (this.inventory[par1] != null)
	{
		ItemStack var2 = this.inventory[par1];
		this.inventory[par1] = null;
		return var2;
	}
	return null;
}

@Override
public void setInventorySlotContents(int par1, ItemStack par2ItemStack)
{
	this.inventory[par1] = par2ItemStack;

	if (par2ItemStack != null && par2ItemStack.stackSize > this.getInventoryStackLimit())
	{
		par2ItemStack.stackSize = this.getInventoryStackLimit();
	}

	this.markDirty();
}

/**
 * Reads a tile entity from NBT.
 */
public void readFromNBT(NBTTagCompound tagCompound)
{
	super.readFromNBT(tagCompound);
	NBTTagList tagList = (NBTTagList) tagCompound.getTag("Items");
	this.inventory = new ItemStack[this.getSizeInventory()];

	for (int count = 0; count < tagList.tagCount(); ++count)
	{
		NBTTagCompound nbt = (NBTTagCompound) tagList.getCompoundTagAt(count);
		int slot = nbt.getByte("Slot") & 255;

		if (slot >= 0 && slot < this.inventory.length)
		{
			this.inventory[slot] = ItemStack.loadItemStackFromNBT(nbt);
		}
	}

	if (tagCompound.hasKey("CustomName", )
	{
		this.customName = tagCompound.getString("CustomName");
	}
}

@Override
public void writeToNBT(NBTTagCompound par1NBTTagCompound)
{
	super.writeToNBT(par1NBTTagCompound);
	NBTTagList var2 = new NBTTagList();

	for (int var3 = 0; var3 < this.inventory.length; ++var3)
	{
		if (this.inventory[var3] != null)
		{
			NBTTagCompound var4 = new NBTTagCompound();
			var4.setByte("Slot", (byte) var3);
			this.inventory[var3].writeToNBT(var4);
			var2.appendTag(var4);
		}
	}

	par1NBTTagCompound.setTag("Items", var2);
}

@Override
public void onDataPacket(NetworkManager net, S35PacketUpdateTileEntity pkt)
{
	NBTTagCompound tagCom = pkt.getNbtCompound();
	this.readFromNBT(tagCom);
}

@Override
public Packet getDescriptionPacket()
{
	NBTTagCompound tagCom = new NBTTagCompound();
	this.writeToNBT(tagCom);
	return new S35PacketUpdateTileEntity(pos, getBlockMetadata(), tagCom);
}

@Override
public int getInventoryStackLimit()
{
	return 64;
}

@Override
public boolean isUseableByPlayer(EntityPlayer par1EntityPlayer)
{
	return this.worldObj.getTileEntity(pos) != this ? false : par1EntityPlayer.getDistanceSq(this.pos.getX() + 0.5D, this.pos.getY() + 0.5D, this.pos.getZ() + 0.5D) <= 64.0D;
}

@Override
public void invalidate()
{
	this.updateContainingBlockInfo();
	super.invalidate();
}

@Override
public boolean isItemValidForSlot(int par1, ItemStack par2ItemStack)
{
	return true;
}

@Override
public String getName()
{
	return this.hasCustomName() ? this.customName : "container.fridge";
}

@Override
public boolean hasCustomName()
{
	return this.customName != null && this.customName.length() > 0;
}

@Override
public IChatComponent getDisplayName()
{
	return new ChatComponentText(getName());
}

@Override
public void openInventory(EntityPlayer playerIn)
{
}

@Override
public void closeInventory(EntityPlayer playerIn)
{
}

@Override
public int getField(int id)
{
	return 0;
}

@Override
public void setField(int id, int value)
{
}

@Override
public int getFieldCount()
{
	return 0;
}

@Override
public void clear()
{
	for (int i = 0; i < inventory.length; i++)
	{
		inventory[i] = null;
	}
}

@Override
public Container createContainer(InventoryPlayer playerInventory, EntityPlayer playerIn) 
{
	return new ContainerSCTileEntity(playerInventory, this);
}

@Override
public String getGuiID() 
{
	return "0";
}

@Override
public int[] getSlotsForFace(EnumFacing side) 
{
	return slots;
}

@Override
public boolean canInsertItem(int index, ItemStack stack, EnumFacing direction) 
{
	return stack.getItem() instanceof ItemFood && !isLocked();
}

@Override
public boolean canExtractItem(int index, ItemStack stack, EnumFacing direction) 
{
	return !isLocked();
}
}

 

Tile Entities Class

package com.duckcreeper.steelcraft.tileentity;

import net.minecraftforge.fml.common.registry.GameRegistry;

public class SCTileEntities {

public static void init(){
	GameRegistry.registerTileEntity(SCTileEntity.class, "test_tile_entity");
}

}

 

Container For Tile Entity

package com.duckcreeper.steelcraft.gui.containers;

import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.inventory.Container;
import net.minecraft.inventory.IInventory;
import net.minecraft.inventory.Slot;
import net.minecraft.item.ItemFood;
import net.minecraft.item.ItemStack;

import com.duckcreeper.steelcraft.gui.GuiSCTileEntity;
import com.duckcreeper.steelcraft.gui.slots.SlotSCTile;
import com.duckcreeper.steelcraft.tileentity.SCTileEntity;

public class ContainerSCTileEntity extends Container
{
private IInventory lowerChestInventory;
private int numRows;

public ContainerSCTileEntity(IInventory playerInventory, IInventory SCTileEntityInventory)
{
	this.lowerChestInventory = playerInventory;
	SCTileEntityInventory.openInventory(null);
	int var4, var5;

	int count = 0;
	for (int i = 0; i < 3; i++)
	{
		for (int j = 0; j < 5; j++)
		{
			this.addSlotToContainer(new SlotSCTile(SCTileEntityInventory, count, j * 18 + 44, i * 18 + 18));
			count++;
		}
	}

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

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

public boolean canInteractWith(EntityPlayer par1EntityPlayer)
{
	return this.lowerChestInventory.isUseableByPlayer(par1EntityPlayer);
}

public ItemStack transferStackInSlot(EntityPlayer par1EntityPlayer, int par2)
{
	ItemStack var3 = null;
	Slot var4 = (Slot) this.inventorySlots.get(par2);

	if (var4 != null && var4.getHasStack())
	{
		ItemStack var5 = var4.getStack();
		var3 = var5.copy();

		if (!(var5.getItem() instanceof ItemFood))
			return null;

		if (par2 < 16)
		{
			if (!this.mergeItemStack(var5, 16, this.inventorySlots.size(), true))
			{
				return null;
			}
		}
		else if (!this.mergeItemStack(var5, 0, 16, false))
		{
			return null;
		}

		if (var5.stackSize == 0)
		{
			var4.putStack((ItemStack) null);
		}
		else
		{
			var4.onSlotChanged();
		}
	}

	return var3;
}

/**
 * Callback for when the crafting gui is closed.
 */
public void onContainerClosed(EntityPlayer par1EntityPlayer)
{
	super.onContainerClosed(par1EntityPlayer);
	this.lowerChestInventory.closeInventory(par1EntityPlayer);
}

public IInventory func_85151_d()
{
	return this.lowerChestInventory;
}
}

Posted

You don't have to do this: (initializing I mean)

    @Instance
    public static SteelMain instance = new SteelMain();

This is useless: (tab[0, 1, ..., n-1] is literally a number from 0 to n-1.)

private static final int[] slots = new int[] { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14 };

 

As to your problem - "we" need Block, Gui and GuiHandler classes.

1st one is basically responsible for calling opening, which should be done with GuiHandler implementation and which should then open Container AND Gui accordingly (to side).

1.7.10 is no longer supported by forge, you are on your own.

Posted

You do not have an onBlockActivated method telling the game to open the gui

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted

GUI Handler

package com.duckcreeper.steelcraft.gui;


import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.util.BlockPos;
import net.minecraft.world.World;
import net.minecraftforge.fml.common.network.IGuiHandler;

import com.duckcreeper.steelcraft.gui.containers.ContainerSCTileEntity;
import com.duckcreeper.steelcraft.tileentity.SCTileEntity;

public class SCGuiHandler implements IGuiHandler
{
@Override
public Object getServerGuiElement(int id, EntityPlayer player, World world, int x, int y, int z)
{
	TileEntity tile_entity = world.getTileEntity(new BlockPos(x, y, z));
	if (tile_entity instanceof SCTileEntity)
	{
		return new ContainerSCTileEntity(player.inventory, (SCTileEntity) tile_entity);
	}
	return null;
}

@Override
public Object getClientGuiElement(int id, EntityPlayer player, World world, int x, int y, int z)
{
	TileEntity tile_entity = world.getTileEntity(new BlockPos(x, y, z));
	if (tile_entity instanceof SCTileEntity)
	{
		return new GuiSCTileEntity(player.inventory, (SCTileEntity) tile_entity);
	}

	return null;
}

}

 

GUI

package com.duckcreeper.steelcraft.gui;

import net.minecraft.client.gui.inventory.GuiContainer;
import net.minecraft.inventory.IInventory;
import net.minecraft.util.ResourceLocation;

import com.duckcreeper.steelcraft.gui.containers.ContainerSCTileEntity;
import com.duckcreeper.steelcraft.tileentity.SCTileEntity;

public class GuiSCTileEntity extends GuiContainer
{
private static final ResourceLocation gui = new ResourceLocation("sc:textures/gui/basic_machine.png");

public GuiSCTileEntity(IInventory playerInv, SCTileEntity te){
	super(new ContainerSCTileEntity(playerInv, te));

this.xSize = 176;
this.ySize = 166;
}

@Override
protected void drawGuiContainerBackgroundLayer(float partialTicks, int mouseX, int mouseY) {
    }
}

 

Block

package com.duckcreeper.steelcraft.blocks;

import com.duckcreeper.steelcraft.tileentity.SCTileEntity;

import net.minecraft.block.BlockContainer;
import net.minecraft.block.material.Material;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.util.EnumWorldBlockLayer;
import net.minecraft.world.World;

public class basic_rebar_machine extends BlockContainer{

public basic_rebar_machine(Material materialIn) {
	super(materialIn);
}

@Override
public int getRenderType() {
	return 3;
}

@Override
public boolean isOpaqueCube() {
	return false;
}

@Override
public EnumWorldBlockLayer getBlockLayer() {
	return EnumWorldBlockLayer.CUTOUT;
}

@Override
public TileEntity createNewTileEntity(World worldIn, int meta) {
	return new SCTileEntity();
}

}

 

Hope that helps? :P

Posted

Well, basically Draco is right (Was that a pure guess Draco?).

 

You need to override onBlockActivated method in your Block class and call player.openGui(id, Mod.instance, x, y, z);

Looking at GuiHandler and everything else - it should work.

1.7.10 is no longer supported by forge, you are on your own.

Posted

Well, basically Draco is right (Was that a pure guess Draco?).

 

You need to override onBlockActivated method in your Block class and call player.openGui(id, Mod.instance, x, y, z);

Looking at GuiHandler and everything else - it should work.

 

		player.openGui(1, SteelMain.instance, world, x, y, z);

 

I tried this but its says that

SteelMain.instance

"Instance" should be changed to int but i dont know what to do lol :P

Posted

My mistake for giving bad example, your bad for not looking at method.

openGui(instance, id, world, x, y, z);

1.7.10 is no longer supported by forge, you are on your own.

Posted

Well, basically Draco is right (Was that a pure guess Draco?).

 

Not a pure guess. I guessed, looked at his block (labeled container) and it wasn't there.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted

Hello DuckCreeper,

 

I am fairly new to modding, so feel free to beheckle me if my observation is born of ignorance, but I noticed in your GUI handler, you have overridden the drawGuiContainerBackgroundLayer, but you are not doing anything in the body. Which, if I understand things correctly, would mean that the UI would be effectively invisible.

 

If it is any help at all, I have recently been learning in this area too, and my source repository has code that does not much more than enable a block inventory / gui... so it may be of some help as cross-reference in resolving your issues.

 

(Though, as stated, I am very new, so there maybe many glaring errors and bad practices in play in my code.)

 

https://github.com/lynchiem/ClockworkAdventure/blob/master/src/main/java/net/clockworkrocketry/clockworkadventure/gui/GUIPortableChest.java

 

I also noticed that when you register your guiHandler, you are passing in "this" where all the examples I have seen pass in "instance"... though I am not sure this will make any difference, as assuming the instantiation is done without voodoo they are probably equivalent.

 

Sorry I could not be of more help, and best of luck resolving your issue.

 

 

 

 

Posted

If you are still having trouble and can post your updated code (is it in github or similar where it can easily be browsed by chance?), I would be happy to help you debug.

Posted

Sorry, I haven't had a huge amount of time today, but had a quick look over now and my first thoughts are:

 

Your proxy and guihandler are separate. When I implemented mine, based on examples I had seen, my proxy is also the guihandler.

 

I am not experienced enough with modding to know if this would cause a problem, but figured I would mention it, because perhaps there is a conflict happening between server code and client code.

 

If you look at my code (linked in a post above) you will note that my CommonProxy, used by the server, returns null for the getClientGuiElement, but my ClientProxy overrides that behaviour.

 

I will have more time tomorrow after work, so if you haven't resolved by then, I will take a more thorough look at your code.

Posted

So I've been playing around. Now I get this in my console and my game crashes. I mean now i know its trying to make one. But I dont know what to do.

 

Invalid attempt to create a GUI during mod construction. Use an EventHandler instead
at net.minecraftforge.fml.common.network.NetworkRegistry.registerGuiHandler(NetworkRegistry.java:219)

 

Here is my main class

 @EventHandler
 public void load(FMLInitializationEvent event)
    {
    	NetworkRegistry.INSTANCE.registerGuiHandler(instance, proxy);
            }

 

 

NOTE

 

I changed the part that says

   

NetworkRegistry.INSTANCE.registerGuiHandler(instance, proxy);

 

to

   

NetworkRegistry.INSTANCE.registerGuiHandler(this, proxy);

 

and now my game loads.... and when i right click it... nothing pops up but my hand moves... just like when you open a chest or something. So basically my GUI is not showing ingame now...

 

What do i do!?!?!!?

Posted
Your proxy and guihandler are separate. When I implemented mine, based on examples I had seen, my proxy is also the guihandler.

 

Ew.

There is no reason to combine them.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted

So in the onBlockActivated Method. I'm using

player.openGui(SteelMain.instance, 1, world, pos.getX(), pos.getY(), pos.getZ());

But When looking at the openGui its self. The second parameter is the GuiID for the gui. Here is what its set to in my code. :

    @Override
    public String getGuiID()
    {
        return Reference.SteelCraftBlocks.BASIC_REBAR_MACHINE.getRegistryName() + ":Basic";
    }

 

But what do I do? I'm a bit confused on this. An explanation would be helpful :P

Posted

So I thought it would be helpful if i posted another update :P.... Playing around more I finally understand that. But still the GUI wont open. I get that right arm punching animation like when you open a furnace etc. But nothing happens when i do so.... rip

Posted

So in the onBlockActivated Method. I'm using

player.openGui(SteelMain.instance, 1, world, pos.getX(), pos.getY(), pos.getZ());

But When looking at the openGui its self. The second parameter is the GuiID for the gui. Here is what its set to in my code. :

    @Override
    public String getGuiID()
    {
        return Reference.SteelCraftBlocks.BASIC_REBAR_MACHINE.getRegistryName() + ":Basic";
    }

 

But what do I do? I'm a bit confused on this. An explanation would be helpful :P

 

The gui id is the integer 1 in the opengui call.that other method is badly named as does not do what it is called, nor have you shown what you use it for.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted

So in the onBlockActivated Method. I'm using

player.openGui(SteelMain.instance, 1, world, pos.getX(), pos.getY(), pos.getZ());

But When looking at the openGui its self. The second parameter is the GuiID for the gui. Here is what its set to in my code. :

    @Override
    public String getGuiID()
    {
        return Reference.SteelCraftBlocks.BASIC_REBAR_MACHINE.getRegistryName() + ":Basic";
    }

 

But what do I do? I'm a bit confused on this. An explanation would be helpful :P

 

The gui id is the integer 1 in the opengui call.that other method is badly named as does not do what it is called, nor have you shown what you use it for.

 

Draco I have figured that out now. I now need help on figureing out why the GUI wont show ingame... Tell me if code is needed.

Posted

Try replacing:

 

@Override
public Object getServerGuiElement(int id, EntityPlayer player, World world, int x, int y, int z)
{
	TileEntity tile_entity = world.getTileEntity(new BlockPos(x, y, z));
	if (tile_entity instanceof SCTileEntity)
	{
		return new ContainerSCTileEntity(player.inventory, (SCTileEntity) tile_entity, x, y);
	}
	return null;
}

@Override
public Object getClientGuiElement(int id, EntityPlayer player, World world, int x, int y, int z)
{
	TileEntity tile_entity = world.getTileEntity(new BlockPos(x, y, z));
	if (tile_entity instanceof SCTileEntity)
	{
		return new GuiSCTileEntity(player.inventory, (SCTileEntity) tile_entity);
	}

	return null;
}

 

with:

 

@Override
public Object getServerGuiElement(int id, EntityPlayer player, World world, int x, int y, int z)
{
	TileEntity tile_entity = world.getTileEntity(new BlockPos(x, y, z));
	switch(id){
                     case YOURCONTAINERANDGUIID:
                             return new ContainerSCTileEntity(player.inventory, (SCTileEntity) tile_entity, x, y);
                             break;
                }
	return null;
}

@Override
public Object getClientGuiElement(int id, EntityPlayer player, World world, int x, int y, int z)
{
	TileEntity tile_entity = world.getTileEntity(new BlockPos(x, y, z));
	switch(id){
                     case YOURCONTAINERANDGUIID:
                             return new GuiSCTileEntity(player.inventory, (SCTileEntity) tile_entity);
                             break;
                }

	return null;
}

 

PS: sorry for my bad english :D

Posted

Yeah. YOURCONTAINERANDGUIID is the number you used when opening the gui

 

player.openGui(modInstance, ID....);

 

additional info: gui id's should be unique for your mod, don't define them multiple

 

Create a field in your gui handler class:

 

public static final YOUR_GUIID=0;

public static final ANOTHER_GUIID=1;

 

and then access this ids instead of typing a number, so you can't get confused if you need them :D

 

EDIT: please place the onBlockActivated in your block class and not in your tileentity (maybe you already done that, i've seen it in your sources you linked)

Posted

Your proxy and guihandler are separate. When I implemented mine, based on examples I had seen, my proxy is also the guihandler.

 

Ew.

There is no reason to combine them.

 

Thank you.

 

As I have mentioned, I am very new to modding, and was simply sharing thoughts based on the difference I could identify between my working code and the problematic code.

 

Combined is probably an overstatement, given the GUI related functions are the only ones I am currently handling over riding in the "proxy".

 

I am still struggling to get my head around how the proxy system works. I need to explicitly tell the mod which class to use as the client and server proxies.

 

@SidedProxy(clientSide = "net.clockworkrocketry.clockworkadventure.network.ClientProxy", serverSide = "net.clockworkrocketry.clockworkadventure.network.CommonProxy")

 

If I get rid of this, my GUI stops working.

 

And my proxy is nothing more than in implementation of IGuiHandler at this stage, containing pretty much only the getClientGuiElement and getServerGuiElement functions.

 

So I thought it was worth exploring as a possible breakage point, as I thought maybe the getClientGuiElement and getServerGuiElement functions needed to be "wired" up to the server/client workflow.

 

But obviously this is incorrect.

Posted

Take a look at my code sometime.

I don't so things the best, objectively. But the class layout is more standard.

 

https://github.com/Draco18s/Artifacts/tree/master/main/java/com/draco18s/artifacts/

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

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



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • Version 1.19 - Forge 41.0.63 I want to create a wolf entity that I can ride, so far it seems to be working, but the problem is that when I get on the wolf, I can’t control it. I then discovered that the issue is that the server doesn’t detect that I’m riding the wolf, so I’m struggling with synchronization. However, it seems to not be working properly. As I understand it, the server receives the packet but doesn’t register it correctly. I’m a bit new to Java, and I’ll try to provide all the relevant code and prints *The comments and prints are translated by chatgpt since they were originally in Spanish* Thank you very much in advance No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. MountableWolfEntity package com.vals.valscraft.entity; import com.vals.valscraft.network.MountSyncPacket; import com.vals.valscraft.network.NetworkHandler; import net.minecraft.client.Minecraft; import net.minecraft.network.syncher.EntityDataAccessor; import net.minecraft.network.syncher.EntityDataSerializers; import net.minecraft.network.syncher.SynchedEntityData; import net.minecraft.server.MinecraftServer; import net.minecraft.server.level.ServerPlayer; import net.minecraft.world.entity.EntityType; import net.minecraft.world.entity.Mob; import net.minecraft.world.entity.ai.attributes.AttributeSupplier; import net.minecraft.world.entity.ai.attributes.Attributes; import net.minecraft.world.entity.animal.Wolf; import net.minecraft.world.entity.player.Player; import net.minecraft.world.entity.Entity; import net.minecraft.world.InteractionHand; import net.minecraft.world.InteractionResult; import net.minecraft.world.item.ItemStack; import net.minecraft.world.item.Items; import net.minecraft.world.level.Level; import net.minecraft.world.phys.Vec3; import net.minecraftforge.event.TickEvent; import net.minecraftforge.eventbus.api.SubscribeEvent; import net.minecraftforge.network.PacketDistributor; public class MountableWolfEntity extends Wolf { private boolean hasSaddle; private static final EntityDataAccessor<Byte> DATA_ID_FLAGS = SynchedEntityData.defineId(MountableWolfEntity.class, EntityDataSerializers.BYTE); public MountableWolfEntity(EntityType<? extends Wolf> type, Level level) { super(type, level); this.hasSaddle = false; } @Override protected void defineSynchedData() { super.defineSynchedData(); this.entityData.define(DATA_ID_FLAGS, (byte)0); } public static AttributeSupplier.Builder createAttributes() { return Wolf.createAttributes() .add(Attributes.MAX_HEALTH, 20.0) .add(Attributes.MOVEMENT_SPEED, 0.3); } @Override public InteractionResult mobInteract(Player player, InteractionHand hand) { ItemStack itemstack = player.getItemInHand(hand); if (itemstack.getItem() == Items.SADDLE && !this.hasSaddle()) { if (!player.isCreative()) { itemstack.shrink(1); } this.setSaddle(true); return InteractionResult.SUCCESS; } else if (!level.isClientSide && this.hasSaddle()) { player.startRiding(this); MountSyncPacket packet = new MountSyncPacket(true); // 'true' means the player is mounted NetworkHandler.CHANNEL.sendToServer(packet); // Ensure the server handles the packet return InteractionResult.SUCCESS; } return InteractionResult.PASS; } @Override public void travel(Vec3 travelVector) { if (this.isVehicle() && this.getControllingPassenger() instanceof Player) { System.out.println("The wolf has a passenger."); System.out.println("The passenger is a player."); Player player = (Player) this.getControllingPassenger(); // Ensure the player is the controller this.setYRot(player.getYRot()); this.yRotO = this.getYRot(); this.setXRot(player.getXRot() * 0.5F); this.setRot(this.getYRot(), this.getXRot()); this.yBodyRot = this.getYRot(); this.yHeadRot = this.yBodyRot; float forward = player.zza; float strafe = player.xxa; if (forward <= 0.0F) { forward *= 0.25F; } this.flyingSpeed = this.getSpeed() * 0.1F; this.setSpeed((float) this.getAttributeValue(Attributes.MOVEMENT_SPEED) * 1.5F); this.setDeltaMovement(new Vec3(strafe, travelVector.y, forward).scale(this.getSpeed())); this.calculateEntityAnimation(this, false); } else { // The wolf does not have a passenger or the passenger is not a player System.out.println("No player is mounted, or the passenger is not a player."); super.travel(travelVector); } } public boolean hasSaddle() { return this.hasSaddle; } public void setSaddle(boolean hasSaddle) { this.hasSaddle = hasSaddle; } @Override protected void dropEquipment() { super.dropEquipment(); if (this.hasSaddle()) { this.spawnAtLocation(Items.SADDLE); this.setSaddle(false); } } @SubscribeEvent public static void onServerTick(TickEvent.ServerTickEvent event) { if (event.phase == TickEvent.Phase.START) { MinecraftServer server = net.minecraftforge.server.ServerLifecycleHooks.getCurrentServer(); if (server != null) { for (ServerPlayer player : server.getPlayerList().getPlayers()) { if (player.isPassenger() && player.getVehicle() instanceof MountableWolfEntity) { MountableWolfEntity wolf = (MountableWolfEntity) player.getVehicle(); System.out.println("Tick: " + player.getName().getString() + " is correctly mounted on " + wolf); } } } } } private boolean lastMountedState = false; @Override public void tick() { super.tick(); if (!this.level.isClientSide) { // Only on the server boolean isMounted = this.isVehicle() && this.getControllingPassenger() instanceof Player; // Only print if the state changed if (isMounted != lastMountedState) { if (isMounted) { Player player = (Player) this.getControllingPassenger(); // Verify the passenger is a player System.out.println("Server: Player " + player.getName().getString() + " is now mounted."); } else { System.out.println("Server: The wolf no longer has a passenger."); } lastMountedState = isMounted; } } } @Override public void addPassenger(Entity passenger) { super.addPassenger(passenger); if (passenger instanceof Player) { Player player = (Player) passenger; if (!this.level.isClientSide && player instanceof ServerPlayer) { // Send the packet to the server to indicate the player is mounted NetworkHandler.CHANNEL.send(PacketDistributor.PLAYER.with(() -> (ServerPlayer) player), new MountSyncPacket(true)); } } } @Override public void removePassenger(Entity passenger) { super.removePassenger(passenger); if (passenger instanceof Player) { Player player = (Player) passenger; if (!this.level.isClientSide && player instanceof ServerPlayer) { // Send the packet to the server to indicate the player is no longer mounted NetworkHandler.CHANNEL.send(PacketDistributor.PLAYER.with(() -> (ServerPlayer) player), new MountSyncPacket(false)); } } } @Override public boolean isControlledByLocalInstance() { Entity entity = this.getControllingPassenger(); return entity instanceof Player; } @Override public void positionRider(Entity passenger) { if (this.hasPassenger(passenger)) { double xOffset = Math.cos(Math.toRadians(this.getYRot() + 90)) * 0.4; double zOffset = Math.sin(Math.toRadians(this.getYRot() + 90)) * 0.4; passenger.setPos(this.getX() + xOffset, this.getY() + this.getPassengersRidingOffset() + passenger.getMyRidingOffset(), this.getZ() + zOffset); } } } MountSyncPacket package com.vals.valscraft.network; import com.vals.valscraft.entity.MountableWolfEntity; import net.minecraft.network.FriendlyByteBuf; import net.minecraft.server.level.ServerLevel; import net.minecraft.server.level.ServerPlayer; import net.minecraft.world.entity.Entity; import net.minecraft.world.entity.player.Player; import net.minecraftforge.network.NetworkEvent; import java.util.function.Supplier; public class MountSyncPacket { private final boolean isMounted; public MountSyncPacket(boolean isMounted) { this.isMounted = isMounted; } public void encode(FriendlyByteBuf buffer) { buffer.writeBoolean(isMounted); } public static MountSyncPacket decode(FriendlyByteBuf buffer) { return new MountSyncPacket(buffer.readBoolean()); } public void handle(NetworkEvent.Context context) { context.enqueueWork(() -> { ServerPlayer player = context.getSender(); // Get the player from the context if (player != null) { // Verifies if the player has dismounted if (!isMounted) { Entity vehicle = player.getVehicle(); if (vehicle instanceof MountableWolfEntity wolf) { // Logic to remove the player as a passenger wolf.removePassenger(player); System.out.println("Server: Player " + player.getName().getString() + " is no longer mounted."); } } } }); context.setPacketHandled(true); // Marks the packet as handled } } networkHandler package com.vals.valscraft.network; import com.vals.valscraft.valscraft; import net.minecraft.resources.ResourceLocation; import net.minecraftforge.network.NetworkRegistry; import net.minecraftforge.network.simple.SimpleChannel; import net.minecraftforge.network.NetworkEvent; import java.util.function.Supplier; public class NetworkHandler { private static final String PROTOCOL_VERSION = "1"; public static final SimpleChannel CHANNEL = NetworkRegistry.newSimpleChannel( new ResourceLocation(valscraft.MODID, "main"), () -> PROTOCOL_VERSION, PROTOCOL_VERSION::equals, PROTOCOL_VERSION::equals ); public static void init() { int packetId = 0; // Register the mount synchronization packet CHANNEL.registerMessage( packetId++, MountSyncPacket.class, MountSyncPacket::encode, MountSyncPacket::decode, (msg, context) -> msg.handle(context.get()) // Get the context with context.get() ); } }  
    • Do you use features of inventory profiles next (ipnext) or is there a change without it?
    • Remove rubidium - you are already using embeddium, which is a fork of rubidium
  • Topics

×
×
  • Create New...

Important Information

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