[1.10.2] Loading custom gui for custom block

[Animefan8888]

Lets try something instead of re creating the variables

public final int fuel_slots = 1;
public final int input_slots = 1;
public final int output_slots = 2;

just initialize them in the constructor.

[Draco18s]

Has it occurred to anyone in this thread to use Capabilities instead?

[Animefan8888]

Has it occurred to anyone in this thread to use Capabilities instead?

Why would he want to use capabilities, he is using a TileEntity, not saving data to the player.

Note to self capabilities can be tied to TileEntities.

 

[Messorix]

Like this?

	public ContainerFluxGrinder(InventoryPlayer player, TileEntityFluxGrinder tileentity)
{
	super(player, tileentity);

	super.fuel_slots = 1;
	super.input_slots = 1;
	super.output_slots = 2;

	tileEntityFluxGrinder = tileentity;
}

[Animefan8888]

Yes, did that work.

[Messorix]

It did not

Meanwhile I'm reading up on what Capabilities actually is

[Animefan8888]

Like this?

	public ContainerFluxGrinder(InventoryPlayer player, TileEntityFluxGrinder tileentity)
{
	super(player, tileentity);

	super.fuel_slots = 1;
	super.input_slots = 1;
	super.output_slots = 2;

	tileEntityFluxGrinder = tileentity;
}

 

Instead change the variables to not final and call super after changing them.

[Messorix]

super(player, tileentity); has to be the first line in the constructor... (finals were already removed)

 

[Animefan8888]

Why would super have to be first?

[Messorix]

Eclipse says "Constructor call must be the first statement in a constructor" so....

[Animefan8888]

Well then I'm going to say what I think happened since you are using for loops and setting variables that are initially set to 0. The constructor for the previous class gets called before the variables could get set in your ContainerFluxGrinder.

 

Solution add slots in your new Containers atleast the ones that are not required.

[Messorix]

As far as my knowledge of Java-programming goes, child-constructors get called first and then go up the parent-tree.

I'm also not entirely sure how you mean this:

Solution add slots in your new Containers atleast the ones that are not required.

[Animefan8888]

Sadly it is not so http://stackoverflow.com/questions/1168345/why-does-this-and-super-have-to-be-the-first-statement-in-a-constructor

 

What I mean is if you have something like upgrade slots that will be in all of the containers, you can add them in the main container. And all new ones for that specific TileEntity you add in that TileEntities container.

[Messorix]

That still doesn't explain how I'm supposed to handle this problem though...

[Animefan8888]

Ok you are going to have to add your Slots that are for you TileEntityFluxGrinder, you need to add them in your ContainerFluxGrinder.

[Messorix]

I never make any custom slots though...

Only in the container classes are Slots (as classes/entities) mentioned.

 

So I'm still unsure what you exactlye mean by that

[Animefan8888]

addSlotToContainer is being called in your main class.

When you call it the number of slots is still = 0 for all of those, and since you stored the number of slots in your TileEntity I don't understand why you don't use those? Instead of creating new variables in your container.

 

*Note make them not final.

[Messorix]

I got the slots to show up properly and if I shift-click a copper ingot it doesn't crash anymore.

However, it doesn't get placed in the input slot nor does fuel end up in the fuel slot...

Have I missed some functionality?

 

git is updated

[Animefan8888]

What does this line print out now?

System.out.println(first_input_index + ", " + input_slots + ", " + (first_input_index + input_slots));

[Messorix]

[10:20:05] [Client thread/INFO] [sTDOUT]: [com.messorix.moleculecraft.base.containers.ContainerFluxGrinder:transferStackInSlot:45]: 37, 1, 38

[10:20:05] [server thread/INFO] [sTDOUT]: [com.messorix.moleculecraft.base.containers.ContainerFluxGrinder:transferStackInSlot:45]: 37, 1, 38

 

So that works I guess

[Animefan8888]

That should be working....but change

//This
if (!mergeItemStack(sourceStack, first_fuel_index, first_fuel_index + fuel_slots, true))
// To this
if (!mergeItemStack(sourceStack, first_fuel_index, first_fuel_index + fuel_slots, false))

[Messorix]

Not sure what should've changed, but it didn't do anything.

Everything is exactly the same, even error log

[Animefan8888]

There was an error log?

[Messorix]

I meant the console...

My bad

[Animefan8888]

Did you check if you could shift click fuel into the inventory?

 

*Ignore what code question was here.