Hi there.

 

I've been trying to create a basic Gui for my Mod. The Gui has a background, a text input, buttons and has strings drawn on. My Gui displays and functions correctly but every now and then it will cause a fatal crash. The crash happens randomly. Sometimes when I try and display it, it will instantly crash. Other times I can open and close the screen multiple times with no issue.

 

The Gui is called by right clicking on a block using:

 

	Minecraft.getMinecraft().displayGuiScreen(new GuiAdditionBlock());

 

My Gui class can be found at : GuiMathsBlock

 

The Crash report for it can be seen at: Crash Report

 

I think the error is being created with drawing strings as fontRendererObj is being set to null somehow (You can see the print statements in the crash report).

 

To test this I created an even simpler GUI screen that had just a background and a button. This also crashes but threw a different error. Can anyone see what i'm doing wrong here?

Here is the other Gui made : The Even simpler Gui

Crash report from that one : crash report

 

I've attached images of the working Gui but the upload has failed. Please let me know if you have any suggestions or need more details.

 

Heres a quick video if my bug. (Video is a little laggy) : GUI Bug

 

Thanks.

Edited April 17, 20178 yr by Sparky3295
added video link

@diesieben07

I open the GUI from the BlockMathsAdditionBlock

https://gist.github.com/MarkWatt3295/103b6a5b8b1976a8f18ceef48ae7cf48

Hi, GUI is strictly client-sided.

 In your GUIMathsBlock.java your block activation method should check whether the its on the client side.

 

if (player.world.isRemote){
	Minecraft.getMinecraft().displayGuiScreen(new GuiAdditionBlock());
}

Should fix it.

world.isRemote refers to client side. If its false, its server side.

Edited April 17, 20178 yr by kirinnee97

@kirinnee97 I do already do that if you look at my code.

@diesieben07 Thanks Ill have a read now. I've been stuck with this for a while. It works fine but randomly crashes. Am I just calling it wrong?

1 minute ago, Sparky3295 said:

@kirinnee97 I do already do that if you look at my code.

You used (!player.world.isRemote) instead of (player.world.isRemote).

If you want it to work on dedicated server, replace the openGUI statement with one that sends a packet to client side. Then on the client packet handler, open the gui.

@kirinnee97 Do you have any good examples for sending packets that deal with the Gui? I have only used packets to give items to the player. Thanks.

@diesieben07 I see. And I guess in this instance that behaviour is the random crash.

@diesieben07 All I want is the player to answer the question and if its right they are given an item as a reward. This is for a singleplayer mod.

4 minutes ago, Sparky3295 said:

@diesieben07 All I want is the player to answer the question and if its right they are given an item as a reward. This is for a singleplayer mod.

Then using  (player.world.isRemote) instead of (!player.world.isRemote)  for your check would work.

6 minutes ago, diesieben07 said:

This is not needed. The GUI can be opened on the client directly through the client proxy if no inventory interaction is needed.

Oops looks like i was mistaken. So by registering the open gui statement under clientproxy that overrides commonproxy, then call the common proxy would work?

Edited April 17, 20178 yr by kirinnee97

3 minutes ago, diesieben07 said:

Only when the displayGuiScreen call is moved to the client proxy.

Sorry, but i just did a test

	@Override
		public boolean onBlockActivated(World worldIn, BlockPos pos, IBlockState state, EntityPlayer playerIn, EnumHand hand, EnumFacing facing, float hitX, float hitY, float hitZ){
			if(worldIn.isRemote){
				Minecraft.getMinecraft().displayGuiScreen(new HUDEditorGUI(playerIn));
			}
			return true;
		}

Works fine in single player, but as you predicted, fails in dedi servs, so since he is using for SP,  it assume it should work fine?

@kirinnee97 @diesieben07

Im testing using :
 

player.world.isRemote

So far no crashes. Ill let you know when I've finished testing. Thanks for the help!

@kirinnee97 @diesieben07 Seem to have done the trick Thanks for the help.

Try running your game in Eclipse as the dedicated server*, then run a client and connect to it (server IP: 127.0.0.1) and interacting with your block.

 

*Next to the green "run" button is a little down-pointing triangle. Click that and pick "Run Server" then again for "Run Client"