Jump to content

Recommended Posts

Posted

i have created a Level Capability to store data to the level, allowing for blocks to be painted by the player. The basic implementation works, i can right click a block and it gets a tint as defined. However, when I close the game and reopen it, all painted areas are gone, that is, it seems to either not save or reload the capabilties to or from disk. I have made a similar cabability for blockentities, and those save fine, so it seems to be related to the level. Perhaps a server/client issue? Either way, I'm not sure where to go, so any help would be greatly appreciated.

package com.matnx.masonry.multiblock.capability;

import net.minecraft.core.BlockPos;
import net.minecraft.nbt.CompoundTag;
import net.minecraftforge.common.capabilities.AutoRegisterCapability;
import org.checkerframework.checker.units.qual.A;

import java.nio.charset.StandardCharsets;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;
import java.util.stream.Collectors;

@AutoRegisterCapability
public class ColorCapability {
    private List<Integer> color = new ArrayList<>();
    private List<BlockPos> pos = new ArrayList<>();

    public int getColor(int i) {
        if (color.size() > i) {
            return color.get(i);
        } else {
            return -1;
        }
    }

    public void setColor(int p_18944_, int i) {
        color.set(i, p_18944_);
    }

    public void addColor(int p_18944_) {
        color.add(p_18944_);
    }

    public BlockPos getPos(int i) {
        if (pos.size() > i) {
            return pos.get(i);
        } else {
            return new BlockPos(0, -500, 0);
        }
    }

    public int containsPos(BlockPos posit) {
        return pos.indexOf(posit);
    }

    public void addPos(BlockPos p_18944_) {
        pos.add(p_18944_);
    }

    public void setPos(BlockPos p_18944_, int i) {
        pos.set(i, p_18944_);
    }

    public void saveNBTData(CompoundTag nbt) {
        String[] stringArray = pos.stream()
                .map(BlockPos::toShortString)
                .toArray(String[]::new);
        String concatenatedString = String.join(";", stringArray);
        byte[] byteArray = concatenatedString.getBytes(StandardCharsets.UTF_8);
        nbt.putByteArray("pos", byteArray);
        nbt.putIntArray("color", color);
    }

    public void loadNBTData(CompoundTag nbt) {
        byte[] byteArray = nbt.getByteArray("pos");
        String restoredString = new String(byteArray, StandardCharsets.UTF_8);
        String[] restoredArray = restoredString.split(";");

        pos = Arrays.stream(restoredArray).map(s -> {
                    String[] parts = s.split(",");
                    return new BlockPos(Integer.valueOf(parts[0]), Integer.valueOf(parts[1]), Integer.valueOf(parts[2]));
                }).collect(Collectors.toList());
        color = Arrays.stream(nbt.getIntArray("color"))
                .boxed()
                .collect(Collectors.toList());
    }
}

 

Posted
20 minutes ago, MatNX said:

Either way, I'm not sure where to go, so any help would be greatly appreciated.

Can you post the code for your capability provider? Also, you can use NbtUtils.writeBlockPos()/readBlockPos() to more easily write/read BlockPos to/from NBT.

Posted

Provider is fairly straightforward, here you go. Thanks for the fast response and the tip!

package com.matnx.masonry.multiblock.capability;

import net.minecraft.core.Direction;
import net.minecraft.nbt.CompoundTag;
import net.minecraftforge.common.capabilities.Capability;
import net.minecraftforge.common.capabilities.CapabilityManager;
import net.minecraftforge.common.capabilities.CapabilityToken;
import net.minecraftforge.common.capabilities.ICapabilityProvider;
import net.minecraftforge.common.util.INBTSerializable;
import net.minecraftforge.common.util.LazyOptional;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;

public class ColorCapabilityProvider implements ICapabilityProvider, INBTSerializable<CompoundTag> {
    public static Capability<ColorCapability> COLOR_CAPABILITY = CapabilityManager.get(new CapabilityToken<>() {
    });

    private ColorCapability colorCapability = null;
    private final LazyOptional<ColorCapability> optional = LazyOptional.of(this::createColor);

    private @NotNull ColorCapability createColor() {
        if (this.colorCapability == null) {
            this.colorCapability = new ColorCapability();
        }
        return this.colorCapability;
    }

    @Override
    public @NotNull <T> LazyOptional<T> getCapability(@NotNull Capability<T> cap, @Nullable Direction side) {
        if (cap == COLOR_CAPABILITY) {
            return optional.cast();
        }
        return LazyOptional.empty();
    }

    @Override
    public CompoundTag serializeNBT() {
        CompoundTag nbt = new CompoundTag();
        createColor().saveNBTData(nbt);
        return nbt;
    }

    @Override
    public void deserializeNBT(CompoundTag nbt) {
        createColor().loadNBTData(nbt);
    }
}

 

Posted
6 hours ago, MatNX said:

Provider is fairly straightforward, here you go. Thanks for the fast response and the tip!

Hmm, I don't see any problems with the code. You could try debugging and put breakpoints in saveNBTData() and loadNBTData() to see make sure that the methods are called and that the code in the methods works as expected.

Posted
15 hours ago, vemerion said:

You could try debugging and put breakpoints in saveNBTData() and loadNBTData()

The saveNBTData and loadNBTData both fires, and, when loading a world, I can see the data in the field while debugging. However, here is where it gets interesting: Save and load both get called 3 times, and they only have the data one the first load/save. It seems that the subsequent 2 calls override my color and pos fields. It seems like a Client/Server issue.

 

Posted

Update: I got it! It was a really stupid problem: blockpos.toshortstring concatenates using , . I was checking for ,. I just missed the space. A stupid mistake, but thanks for your help anyway.

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



×
×
  • Create New...

Important Information

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