Jump to content

Is there a better way to do this?


colejagdtiger

Recommended Posts

I am currently working on a chunk based energy system, however every chunk will have its own different energy value and max value, in addition I am making splitting the energy into different types, meaning that each chunk would have a energy value and a max energy value. I plan on using a Hashmap for each energy type which stores arraylists containing the current energy value and the max energy value of the chunk it is tied to. My concern is that this could cause a lot of lag, so my question is if there is a better way to do this, and even if there is a better way, is it just better to simplify the energy system and leave out some of what I am planing(I am planing on have 5 to 7 different energy types).

Link to comment
Share on other sites

"How to do it" depends on other design criteria, such as needing data when chunks are not loaded, and how do chunks affect their neighbors.

 

If each chunk is independent, and if its values are only needed when loaded, then things stay simple. Otherwise things get messy.

The debugger is a powerful and necessary tool in any IDE, so learn how to use it. You'll be able to tell us more and get better help here if you investigate your runtime problems in the debugger before posting.

Link to comment
Share on other sites

I'd use encapsulation. Make a class for each energy type that contains within it the data. The following code is an example from mind and I have not tested it. It is just to show the idea. It's been a while since I have written Java code so forgive if I forget a keyword in the following example.

 

Interface CustomEnergy.java

 

public interface CustomEnergy {
    public int getCurrentEnergyValue();
    public int getMaxEnergyValue();
}

 

Class FireEnergy.java

 

public class FireEnergy implements CustomEnergy {
    private int currentEnergyValue;
    private int maxEnergyValue;

    public FireEnergy(int currentEnergyValue, int maxEnergyValue) {
        this.currentEnergyValue = currentEnergyValue;
        this.maxEnergyValue = maxEnergyValue;
    }

    //Getters
    @Override
    public int getCurrentEnergyValue(){
        return this.currentEnergyValue;
    }
    @Override
    public int getMaxEnergyValue(){
        return this.maxEnergyValue;
    }
}

 

Class WaterEnergy.java

 

public class WaterEnergy implements CustomEnergy {
    private int currentEnergyValue;
    private int maxEnergyValue;

    public WaterEnergy(int currentEnergyValue, int maxEnergyValue) {
        this.currentEnergyValue = currentEnergyValue;
        this.maxEnergyValue = maxEnergyValue;
    }

    //Getters
    @Override
    public int getCurrentEnergyValue(){
        return this.currentEnergyValue;
    }
    @Override
    public int getMaxEnergyValue(){
        return this.maxEnergyValue;
    }
}

 

Then you would do anywhere else in your code the following

 

Map<Chunk, CustomEnergy> hashMap = new HashMap<Chunk, CustomEnergy>();
BlockPos somePos = new BlockPos(30, 20, 10);
BlockPos anotherPos = new BlockPos(100, 64, 100);
Chunk someChunk = world.getChunkFromBlockCoords(pos);
Chunk anotherChunk = world.getChunkFromBlockCoords(anotherPos);
FireEnergy someFireEnergy = new FireEnergy(10, 100);
WaterEnergy someWaterEnergy = new WaterEnergy(5, 100);
hashMap.put(someChunk, someFireEnergy);
hashMap.put(anotherChunk, someWaterEnergy);

 

And handle it that way. One thing I am not sure though is if each Chunk will get the same hash if you for example query it once from the world and then query it again to pass it to the get() function of the HashMap. If it is the case, then the HashMap would allow for an efficient find. Otherwise it won't work at all. Also you cannot use CustomEnergy as a key, it always has to be a value in the HashMap. If you want to use it as key, then each energy class would need to implement the hashCode() and equals() methods.

I hope this helps.

 

As to how you would save that data between game sessions I have no idea. 

Edited by ctbe
Add semicolon in code
Link to comment
Share on other sites

Similar questions have been asked (and answered) here and here.

 

I implemented a solution the to the latter thread myself, you can see the API here and the implementation here. This uses a World capability to store the Map<ChunkPos, IChunkEnergy> at runtime, but loads the IChunkEnergy and saves it to the chunk's data using ChunkDataEvent.Load/Save.

Please don't PM me to ask for help. Asking your question in a public thread preserves it for people who are having the same problem in the future.

Link to comment
Share on other sites

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

    • As the title says i keep on crashing on forge 1.20.1 even without any mods downloaded, i have the latest drivers (nvidia) and vanilla minecraft works perfectly fine for me logs: https://pastebin.com/5UR01yG9
    • Hello everyone, I'm making this post to seek help for my modded block, It's a special block called FrozenBlock supposed to take the place of an old block, then after a set amount of ticks, it's supposed to revert its Block State, Entity, data... to the old block like this :  The problem I have is that the system breaks when handling multi blocks (I tried some fix but none of them worked) :  The bug I have identified is that the function "setOldBlockFields" in the item's "setFrozenBlock" function gets called once for the 1st block of multiblock getting frozen (as it should), but gets called a second time BEFORE creating the first FrozenBlock with the data of the 1st block, hence giving the same data to the two FrozenBlock :   Old Block Fields set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=head] BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@73681674 BlockEntityData : id:"minecraft:bed",x:3,y:-60,z:-6} Old Block Fields set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} Frozen Block Entity set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockPos{x=3, y=-60, z=-6} BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} Frozen Block Entity set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockPos{x=2, y=-60, z=-6} BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} here is the code inside my custom "freeze" item :    @Override     public @NotNull InteractionResult useOn(@NotNull UseOnContext pContext) {         if (!pContext.getLevel().isClientSide() && pContext.getHand() == InteractionHand.MAIN_HAND) {             BlockPos blockPos = pContext.getClickedPos();             BlockPos secondBlockPos = getMultiblockPos(blockPos, pContext.getLevel().getBlockState(blockPos));             if (secondBlockPos != null) {                 createFrozenBlock(pContext, secondBlockPos);             }             createFrozenBlock(pContext, blockPos);             return InteractionResult.SUCCESS;         }         return super.useOn(pContext);     }     public static void createFrozenBlock(UseOnContext pContext, BlockPos blockPos) {         BlockState oldState = pContext.getLevel().getBlockState(blockPos);         BlockEntity oldBlockEntity = oldState.hasBlockEntity() ? pContext.getLevel().getBlockEntity(blockPos) : null;         CompoundTag oldBlockEntityData = oldState.hasBlockEntity() ? oldBlockEntity.serializeNBT() : null;         if (oldBlockEntity != null) {             pContext.getLevel().removeBlockEntity(blockPos);         }         BlockState FrozenBlock = setFrozenBlock(oldState, oldBlockEntity, oldBlockEntityData);         pContext.getLevel().setBlockAndUpdate(blockPos, FrozenBlock);     }     public static BlockState setFrozenBlock(BlockState blockState, @Nullable BlockEntity blockEntity, @Nullable CompoundTag blockEntityData) {         BlockState FrozenBlock = BlockRegister.FROZEN_BLOCK.get().defaultBlockState();         ((FrozenBlock) FrozenBlock.getBlock()).setOldBlockFields(blockState, blockEntity, blockEntityData);         return FrozenBlock;     }  
    • It is an issue with quark - update it to this build: https://www.curseforge.com/minecraft/mc-mods/quark/files/3642325
    • Remove Instant Massive Structures Mod from your server     Add new crash-reports with sites like https://paste.ee/  
  • Topics

×
×
  • Create New...

Important Information

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