[1.12.1] NBT Reading Problem

[MineDen]

*SORRY FOR MY ENGLISH*

Hello, I have problems with NBT reading. Log says, that my mod "throws an exception" during loading.

 

[Server thread/ERROR] [FML]: A TileEntity minecraft:energy_cell(com.mineden.mtest.tileentity.AutoTEEnergy) has thrown an exception during loading, its state cannot be restored.
java.lang.InstantiationException: com.mineden.mtest.tileentity.AutoTEEnergy
	at ...

 

I tried to use other solutions, but that isn't worked for me. Can anyone help me?

What files I need to provide?

 

I'm developing an small energy mod, I have problems with Forge Energy and capabilities, my GUI shows enegy level properly, but there is problems with receiving and extracting energy. So, I writed my own energy storage class, and it's working properly. But after world restart, NBT deleting.

[Draco18s]

>> No code posted

> No help available

 

[MineDen]

I'm working on other computer, but I can provide zip with code or code files.

[MineDen]

[/] TestMod (Main File):

package com.mineden.mtest;

import com.mineden.mtest.blocks.MBlocks;
import com.mineden.mtest.gui.MGuiHandler;
import com.mineden.mtest.items.MItems;
import com.mineden.mtest.proxy.CommonProxy;
import com.mineden.mtest.recipes.RegRecipes;
import com.mineden.mtest.tileentity.MTileEntities;
import net.minecraftforge.fml.common.Mod;
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;


@Mod(modid = ModInfo.MOD_ID, name = ModInfo.MOD_NAME, version = ModInfo.MOD_VERSION)
public class TestMod {

    @SidedProxy(clientSide = ModInfo.CPROXY, serverSide = ModInfo.SPROXY)
    public static CommonProxy proxy;

    @Mod.Instance
    public static TestMod instance;

    @Mod.EventHandler
    public void preInit(FMLPreInitializationEvent event){
        MTileEntities.init();
        MTileEntities.register();
        MBlocks.init();
        MBlocks.register();
        MItems.init();
        MItems.register();
        RegRecipes.register();
    }

    @Mod.EventHandler
    public void init(FMLInitializationEvent event){
        proxy.init();
        NetworkRegistry.INSTANCE.registerGuiHandler(instance, new MGuiHandler());
        MBlocks.lead_ore_generator.register();
    }

    @Mod.EventHandler
    public void postInit(FMLPostInitializationEvent event){
    }

}

[/tileentity/] MTileEntities (Registration):

package com.mineden.mtest.tileentity;

import com.mineden.mtest.blocks.AutoBlockEnergy;
import com.mineden.mtest.gui.MGuiHandler;
import net.minecraft.creativetab.CreativeTabs;


public class MTileEntities {

    public static AutoBlockEnergy energy_cell_te;

    public static void init(){
        energy_cell_te = new AutoBlockEnergy("energy_cell", CreativeTabs.DECORATIONS, 2.8F, MGuiHandler.ENERGY_CELL_GUI);

    }

    public static void register(){
        energy_cell_te.register();
    }

}

[/blocks/] AutoBlockEnergy (Energy Cell block):

package com.mineden.mtest.blocks;

import com.mineden.mtest.ModInfo;
import com.mineden.mtest.TestMod;
import com.mineden.mtest.tileentity.AutoTEEnergy;
import net.minecraft.block.Block;
import net.minecraft.block.ITileEntityProvider;
import net.minecraft.block.material.Material;
import net.minecraft.block.state.IBlockState;
import net.minecraft.client.renderer.block.model.ModelResourceLocation;
import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.item.Item;
import net.minecraft.item.ItemBlock;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.util.EnumFacing;
import net.minecraft.util.EnumHand;
import net.minecraft.util.math.BlockPos;
import net.minecraft.util.text.TextComponentString;
import net.minecraft.world.World;
import net.minecraftforge.client.model.ModelLoader;
import net.minecraftforge.fml.common.registry.ForgeRegistries;
import net.minecraftforge.fml.common.registry.GameRegistry;


public class AutoBlockEnergy extends Block implements ITileEntityProvider {

    private int guiId;

    public AutoBlockEnergy(String unlocName, CreativeTabs creativeTab, float hardness, int guiId) {
        super(Material.ROCK);
        this.setUnlocalizedName(unlocName);
        this.setRegistryName(this.getUnlocalizedName().substring(5));
        this.setCreativeTab(creativeTab);
        this.setHardness(hardness);
        this.setHarvestLevel("pickaxe", 0);
        this.guiId = guiId;
    }

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

    public void register() {
        ForgeRegistries.BLOCKS.register(this);
        Item iBlock = new ItemBlock(this);
        iBlock.setRegistryName(this.getRegistryName());
        ForgeRegistries.ITEMS.register(iBlock);
        GameRegistry.registerTileEntity(AutoTEEnergy.class, this.getUnlocalizedName().substring(5));
        ModelResourceLocation MRL = new ModelResourceLocation(ModInfo.MOD_ID + ":" + iBlock.getUnlocalizedName().substring(5), "invertory");
        ModelLoader.setCustomModelResourceLocation(iBlock, 0, MRL);
    }

    public boolean onBlockActivated(World world, BlockPos pos, IBlockState state, EntityPlayer player,
                                    EnumHand hand, EnumFacing side,
                                    float hitX, float hitY, float hitZ){
        if(!world.isRemote && pos != null) {
            player.sendMessage(new TextComponentString("AutoBlockEnergy activated"));
            player.openGui(TestMod.instance, this.guiId, world, pos.getX(), pos.getY(), pos.getZ());
        }
        else{
            if(pos != null) {
                player.openGui(TestMod.instance, this.guiId, world, pos.getX(), pos.getY(), pos.getZ());
            }
        }
        return true;
    }

}

[/blocks/] AutoBlock (Addditional methods):

package com.mineden.mtest.blocks;

import com.mineden.mtest.ModInfo;
import net.minecraft.block.Block;
import net.minecraft.block.material.MapColor;
import net.minecraft.block.material.Material;
import net.minecraft.client.renderer.block.model.ModelResourceLocation;
import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.item.Item;
import net.minecraft.item.ItemBlock;
import net.minecraft.item.ItemStack;
import net.minecraft.item.crafting.Ingredient;
import net.minecraft.util.ResourceLocation;
import net.minecraftforge.client.model.ModelLoader;
import net.minecraftforge.fml.common.registry.ForgeRegistries;
import net.minecraftforge.fml.common.registry.GameRegistry;


public class AutoBlock extends Block{

    public AutoBlock(MapColor blockMapColorIn, String unlocName, CreativeTabs creativeTab, float hardness) {
        super(Material.ROCK, blockMapColorIn);
        this.setUnlocalizedName(unlocName);
        this.setRegistryName(this.getUnlocalizedName().substring(5));
        this.setCreativeTab(creativeTab);
        this.setHardness(hardness);
        this.setHarvestLevel("pickaxe", 0);
    }

    public void register(){
        ForgeRegistries.BLOCKS.register(this);
        Item iBlock = new ItemBlock(this);
        iBlock.setRegistryName(this.getRegistryName());
        ForgeRegistries.ITEMS.register(iBlock);
        ModelResourceLocation MRL = new ModelResourceLocation(ModInfo.MOD_ID + ":" + iBlock.getUnlocalizedName().substring(5), "invertory");
        ModelLoader.setCustomModelResourceLocation(iBlock, 0, MRL);
    }

    public void addShapedCraft(String craftName, Object... input){
        ResourceLocation name = new ResourceLocation(ModInfo.MOD_ID + ":" + craftName);
        ItemStack recipeIS = new ItemStack(Item.getItemFromBlock(this), 1);
        GameRegistry.addShapedRecipe(name, null, recipeIS, input);
    }

    public void addShapelessCraft(String craftName, Ingredient... ingredients){
        ResourceLocation name = new ResourceLocation(ModInfo.MOD_ID + ":" + craftName);
        ItemStack recipeIS = new ItemStack(Item.getItemFromBlock(this), 1);
        GameRegistry.addShapelessRecipe(name, null, recipeIS, ingredients);
    }

    public void addSmelting(Item input){
        ItemStack outputIS = new ItemStack(Item.getItemFromBlock(this), 1);
        ItemStack inputIS = new ItemStack(input, 1);
        GameRegistry.addSmelting(inputIS, outputIS, 0.8F);
    }

}

[/tileentity/] AutoTEEnergy (Energy Cell TileEntity):

package com.mineden.mtest.tileentity;

import com.mineden.mtest.MEnergyStorage;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.util.ITickable;


public class AutoTEEnergy extends TileEntity implements ITickable{

    public MEnergyStorage storage;
    private int capacity = 10000;
    private int transferSpeed = 100;

    public AutoTEEnergy(int capacity, int transferSpeed){
        storage = new MEnergyStorage(capacity, transferSpeed);
        this.capacity = capacity;
        this.transferSpeed = transferSpeed;
    }

    @Override
    public NBTTagCompound writeToNBT(NBTTagCompound nbt){
        nbt.setInteger("Energy", this.storage.getEnergy());
        super.writeToNBT(nbt);
        return nbt;
    }

    @Override
    public void readFromNBT(NBTTagCompound nbt){
        int energy = nbt.getInteger("Energy");
        this.storage.setEnergy(energy);
        super.readFromNBT(nbt);
    }

    @Override
    public void update(){
        if(this.world != null){
            this.storage.addEnergy(1);
            this.markDirty();
        }
    }

}

[/] ModInfo (Reference):

package com.mineden.mtest;


public class ModInfo {
    public static final String MOD_ID = "mtest";
    public static final String MOD_NAME = "Energy Mod Test";
    public static final String MOD_VERSION = "1.1.25";
    public static final String CPROXY = "com.mineden.mtest.proxy.ClientProxy";
    public static final String SPROXY = "com.mineden.mtest.proxy.CommonProxy";
}

 

[Draco18s]

Tile Entities, when reconstructed from the save file on disk, are created using a zero-argument constructor (which yours does not have). This means that it is skipping the storage = new MEnergyStorage(capacity, transferSpeed); line, thus storage is null.

[MineDen]

Now I understand. Also I see "no method" errors in log. Are I need to create this constructor?

public AutoTEEnergy(){
	storage = new MEnergyStorage(this.capacity, this.transferSpeed);
}

I think, that in this case  capacity and transferSpeed don't saved, and when I add some other energy cells with other capacity and transfer speed, this can cause problems.

[MineDen]

Now I have side problems. On client side NBT saving properly, but on server side energy always 10000. When I adding energy on server side, energy shows on gui, but don't saves on client side before world reloading, after reloading it's always 10000. When I adding energy on client side, energy saves properly, but don't shows on gui and don't sync with server, after world reload on server always 10000. How I can properly sync energy?

Edited October 29, 2017 by MineDen

[Draco18s]

  On 10/29/2017 at 11:36 AM, MineDen said:

Now I understand. Also I see "no method" errors in log. Are I need to create this constructor?

public AutoTEEnergy(){
	storage = new MEnergyStorage(this.capacity, this.transferSpeed);
}

I think, that in this case  capacity and transferSpeed don't saved, and when I add some other energy cells with other capacity and transfer speed, this can cause problems.

Expand  

You need to create the storage in the readFromNBT method, once you know what the capacity and transfer speeds  are.

[MineDen]

  On 10/29/2017 at 2:39 PM, Draco18s said:

You need to create the storage in the readFromNBT method, once you know what the capacity and transfer speeds  are.

Expand  

Thanks!

OFFTOP: Can I use non 16x16 textures, ex. 32x32?

[Draco18s]

  On 10/29/2017 at 2:41 PM, MineDen said:

OFFTOP: Can I use non 16x16 textures, ex. 32x32?

Expand  

That's what new topics are for. That are "trying it and seeing."

But yes.

[MineDen]

  On 10/29/2017 at 2:43 PM, Draco18s said:

That's what new topics are for. That are "trying it and seeing."

But yes.

Expand  

Another question: on what side I need to add/set/get/remove energy?

[Draco18s]

  On 10/29/2017 at 2:45 PM, MineDen said:

Another question: on what side I need to add/set/get/remove energy?

Expand  

Either:

A) I don't understand the question

or

B) That's only a question you can answer. What side do you WANT to add/set/get/remove energy?

[MineDen]

  On 10/29/2017 at 2:46 PM, Draco18s said:

Either:

A) I don't understand the question

or

B) That's only a question you can answer. What side do you WANT to add/set/get/remove energy?

Expand  

I'm try to use both server and client side.

For additional info: I'm working in IntelliJ IDEA 2016.3, using Hot Swap feature, but it not works when deleting methods. So, I need to restart Minecraft to apply changes.

[Draco18s]

Let me break this down for you:

 

The server is responsible for EVERYTHING except rendering and user input.

The client is responsible for rendering and user input.

 

Which side would it make sense to do things on?

[MineDen]

I tried to add energy on server side (when tileentity updates) and energy doesn't saves. No errors in log. On client side NBT is always 0.

[MineDen]

  On 10/29/2017 at 3:06 PM, diesieben07 said:

There is no NBT on the client side, that does not make sense.

NBT is a data format, it is used for saving to disk and (occasionally) for synchronizing data to the client.

Expand  

  On 10/29/2017 at 12:02 PM, MineDen said:

How I can properly sync NBT?

Expand  

 

[Draco18s]

https://github.com/Draco18s/ReasonableRealism/blob/master/src/main/java/com/draco18s/farming/entities/TileEntityTanner.java#L174-L189

[Draco18s]

  On 10/29/2017 at 3:27 PM, diesieben07 said:

but it's an inefficient way to do it.

Expand  

By "inefficient" you mean "built into the TE class all you have to do is override three methods" right? One of which is literally called getUpdateTag and returns an NBTTagCompound

[MineDen]

I override this methods, but energy still don't saving.

Another problem: my tileentity don't updates before I open GUI. Is this problem can cause other problems?

[MineDen]

  On 10/29/2017 at 3:23 PM, Draco18s said:

https://github.com/Draco18s/ReasonableRealism/blob/master/src/main/java/com/draco18s/farming/entities/TileEntityTanner.java#L174-L189

Expand  

This don't worked for me.

[MineDen]

  On 10/30/2017 at 5:44 AM, diesieben07 said:

Please post updated code, preferably just a Git repository.

Expand  

Fortunately, I already have a GitHub account. What files I need to upload?

1. All workspace [/]
2. Src folder [/src/]
3. Only code [/src/main/]
4. My package [/src/main/com/mineden/mtest/]

[MineDen]

  On 10/30/2017 at 1:34 PM, diesieben07 said:

The repository should be in your project root.

You need to include the following:

- Your src folder

- Your gradle build script

- Preferably: The gradle wrapper.

Expand  

Good. Wait a minute. After downloading, if you use IDEA, open build.gradle of project and in gradle panel run needed commands. If you working in Eclipse, run gradlew setupDecompWorkspace eclipse.

[MineDen]

  On 10/30/2017 at 1:43 PM, diesieben07 said:

You don't have to explain the setup process to me...

Expand  

Okay. I explained it, because most mod developers using Eclipse. But I using IDEA, because it has some extra features like hot swap, gradle commands running etc.

[MineDen]

  On 10/30/2017 at 5:44 AM, diesieben07 said:

Please post updated code, preferably just a Git repository.

Expand  

Published. See here.

[MineDen]

  On 10/30/2017 at 2:38 PM, diesieben07 said:

• TileEntity classes must have a no-argument constructor. I usually recommend not adding any additional constructors, as things can get quite confusing that way (when reading from NBT only the parameterless constructor will be called).
• Why are you adding energy every tick on the client?

Expand  

1. I want to optimize code, like this:

energy_cell = new AutoBlockEnergy("energy_cell", CreativeTabs.DECORATIONS, 10000, 100);
adv_energy_cell = new AutoBlockEnergy("adv_energy_cell", CreativeTabs.DECORATIONS, 20000, 150);
ultimate_energy_cell = new AutoBlockEnergy("ultimate_energy_cell", CreativeTabs.DECORATIONS, 40000, 300);

2. I just testing. I don't want to implement energy generator/consumer right now.