[1.10.2]TileEntity not Working properly (page 2)

[Villfuk02]

How do you mean it saves for you?

[loordgek]

when i exit out the game and come back the items are still there

[Villfuk02]

try pressing the Mill button until it's done and then try to take it the item out of the inventory.

[loordgek]

how does that work. what do i need to put in

[Villfuk02]

1st recipe: leaves tallgrass  melon wheat

2nd recipe: feather string glowstone_dust paper

3rd recipe: 4x essence dust of the same kind

 

the items can be in any order

[loordgek]

	public void finishRecipe(){
	ItemStack out = getOutput(itemHandler.getStackInSlot(0),itemHandler.getStackInSlot(1),itemHandler.getStackInSlot(2),itemHandler.getStackInSlot(3));
	itemHandler.insertItem(4, out, false);

	for(int i = 0; i < 4; i++){
		itemHandler.extractItem(i, 1, false); // first extract and then look it up that will trow a npe. so move this down

	essence_stored += getEssenceOutput(itemHandler.getStackInSlot(0),itemHandler.getStackInSlot(1),itemHandler.getStackInSlot(2),itemHandler.getStackInSlot(3)).getAmount();
	essence_type = getEssenceOutput(itemHandler.getStackInSlot(0),itemHandler.getStackInSlot(1),itemHandler.getStackInSlot(2),itemHandler.getStackInSlot(3)).getType();
	//---Something to write to NBT here---

	}
}

[Villfuk02]

That was simple, but what about the saving?

[Villfuk02]

Just updated the TileEntityManualMill on GitHub. Now doesn't overwrite the  outputs[] when processed.

[Villfuk02]

Hey, is anyone here?

 

I just don thave any idea what to do with this, i tried everything ií could think of.

 

PLZ HALP

[Animefan8888]

PLZ HALP

Please dont do that. Did you override writeToNBT and readFromNBT in your TE?

[Villfuk02]

Yes

 

btw you can look here:

https://github.com/Villfuk02/Essence/blob/master/java/com/villfuk02/essence/tileentities/TileEntityManualMill.java

 

 

[Animefan8888]

Yes

 

btw you can look here:

https://github.com/Villfuk02/Essence/blob/master/java/com/villfuk02/essence/tileentities/TileEntityManualMill.java

You are greatly overusing super calls in your update methods ie getUpdateTag. I cant see anything that wont let it read and write, is writeToNBT and readFromNBT even getting called?

[jeffryfisher]

 

I just don' have any idea what to do with this, i tried everything i could think of.

 

 

Have you set breakpoints and then stepped through and into code using the debugger? If not, then you have more to think of.

 

The debugger is such an essential tool that almost all non-crash runtime weirdness reported in this forum should include at least some observations made in the debugger before we can help most effectively.

 

Also, have you looked beyond the console output and scanned your log files? The log files contain warnings that do not show on the console (but are more likely to be helpful with rendering and crash issues).

 

[Villfuk02]

Somebody please help me

 

[Draco18s]

Have you set breakpoints and then stepped through and into code using the debugger? If not, then you have more to think of.

[Villfuk02]

I have debugged this so many times and nothing seems suspicious.

Everything does what it is supposed to and then it just resets like wtf

[Villfuk02]

It look just lik the server is not syncing with the server, BUT WHY?????

[Draco18s]

It look just lik the server is not syncing with the server, BUT WHY?????

 

If its not syncing with the server, you need to make it sync with the server.

Usually involves packets.

[Villfuk02]

K, but i dont know how

[jeffryfisher]

I have debugged this so many times and nothing seems suspicious.

Everything does what it is supposed to and then it just resets like wtf

 

Something else to watch out for: The default behavior for tile entities is that updating the blockstate will cause the TE to be replaced. There's a "shouldRefresh" method somewhere (I think in the Block class) that needs to be overridden -- but it should not always return false. Instead, it needs to return true when the TE is really not needed anymore (like after its block type has been replaced with something else).

 

If you haven't dealt with "shouldRefresh" yet, then Google it. It has come up several times in this forum.

[Villfuk02]

thx, i'll look it up tomorrow

i dont have any more time today, but thanks