[1.12.2] Player Capability null pointer exception

[SaltStation]

Hello everyone.

The problem I have is that every time I try accessing my capability I get a null pointer exception.

Do I have to send packets to the client at the start of the game to prevent this?

Because I thought you only need packets to update information.

 

Default Implementation:

  Reveal hidden contents

public class Fatigue implements IFatigue {

    private double fatigue = 0;
    private double limit = 20;
    private double regen = 0;

    @Override
    public void setFatigue(double points) {
        this.fatigue=points;

    }

    public void setLimit(double points) {
        this.limit=points;

    }

    public void setRegen(double points) {
        this.regen=points;

    }

    @Override
    public void add(double points) {
        this.fatigue+=points;

    }

    @Override
    public void remove(double points) {
        this.fatigue-=points;

    }

    @Override
    public double getFatigue() {
        return this.fatigue ;
    }

    @Override
    public double getLimit() {
        return this.limit ;
    }

    @Override
    public double getRegen() {
        return this.regen ;
    }
}

 

Storage:

  Reveal hidden contents

public class FatigueStorage implements Capability.IStorage<IFatigue> {

    @Override
    public NBTBase writeNBT(Capability<IFatigue> capability, IFatigue instance, EnumFacing side) {

        NBTTagCompound tag = new NBTTagCompound();
        tag.setDouble("fatigue", instance.getFatigue());
        tag.setDouble("fatigue_limit", instance.getLimit());
        tag.setDouble("cosmo_regen", instance.getRegen());

        System.out.println("written to NBT(Fatigue): "+instance.getFatigue());

        return tag;
    }

    @Override
    public void readNBT(Capability<IFatigue> capability, IFatigue instance, EnumFacing side, NBTBase nbt) {

        NBTTagCompound tag = (NBTTagCompound) nbt;
        instance.setFatigue(tag.getDouble("fatigue"));
        instance.setLimit(tag.getDouble("fatigue_limit"));
        instance.setRegen(tag.getDouble("fatigue_regen"));

        System.out.println("read from NBT(Fatigue): "+instance.getFatigue());

    }
}

 

Provider:

  Reveal hidden contents

public class FatigueProvider implements ICapabilitySerializable<NBTBase> {
    @CapabilityInject(IFatigue.class)
    public static final Capability<IFatigue> FATIGUE_CAPABILITY = null;

    private IFatigue instance = FATIGUE_CAPABILITY.getDefaultInstance();

    @Override
    public boolean hasCapability(Capability<?> capability, EnumFacing facing){
        return capability == FATIGUE_CAPABILITY;
    }

    @Override
    public <T> T getCapability(Capability<T> capability, EnumFacing facing)
    {
        return capability == FATIGUE_CAPABILITY ? FATIGUE_CAPABILITY.<T> cast(this.instance) : null;
    }

    @Override
    public NBTBase serializeNBT(){
        return FATIGUE_CAPABILITY.getStorage().writeNBT(FATIGUE_CAPABILITY, this.instance, null);
    }

    @Override
    public void deserializeNBT(NBTBase nbt) {
        FATIGUE_CAPABILITY.getStorage().readNBT(FATIGUE_CAPABILITY, this.instance,null,nbt);
    }

}

 

Capability Handler:

  Reveal hidden contents

public class CapabilityHandeler {
    public static final ResourceLocation FATIGUE_CAPABILITY = new ResourceLocation(Reference.MOD_ID,"fatigue");

    @SubscribeEvent
    public void attachCapability(AttachCapabilitiesEvent<Entity> event){
        if (!(event.getObject() instanceof EntityPlayer)) return;
        event.addCapability(FATIGUE_CAPABILITY, new FatigueProvider());
    }
}

 

 

Other Handler:

  Reveal hidden contents

public class FatiguePlayerHandeler {

    @SubscribeEvent
    public void onPlayerSleep(PlayerSleepInBedEvent event)
    {
        EntityPlayer player = event.getEntityPlayer();
        IFatigue fatigue = player.getCapability(FatigueProvider.FATIGUE_CAPABILITY, null);

        System.out.println(fatigue.getLimit());
        System.out.println(fatigue.getFatigue());
        System.out.println(fatigue.getFatigue());


    }
    @SubscribeEvent
    public void onPlayerClone(PlayerEvent.Clone event){
        EntityPlayer player = event.getEntityPlayer();
        IFatigue fatigue = player.getCapability(FatigueProvider.FATIGUE_CAPABILITY, null);
        IFatigue oldFatigue = event.getOriginal().getCapability(FatigueProvider.FATIGUE_CAPABILITY, null);

        fatigue.setFatigue(oldFatigue.getFatigue());
        fatigue.setLimit(oldFatigue.getLimit());
        fatigue.setRegen(oldFatigue.getRegen());
    }
}

 

Common Proxy:

  Reveal hidden contents

public class CommonProxy {

    public void preInit() {
        CapabilityManager.INSTANCE.register(IFatigue.class,new FatigueStorage(),Fatigue.class);
        MinecraftForge.EVENT_BUS.register(new CapabilityHandeler());
    }

    public void init() {}

    public void postInit() {}


    public void registerItemRenderer(Item item, int meta, String id) {
    }

}

 

 

 

 

 

 

 

 

 

Edited May 8, 2019 by SaltStation
typo

[V0idWa1k3r]

  On 5/8/2019 at 6:16 PM, SaltStation said:

Common Proxy:

  Reveal hidden contents

Expand  

Don't use a CommonProxy, it makes no sense. Proxies exist to separate sided only code. If your code is common it goes anywhere else but your proxy. This might be the cause of your issue, but

 

  On 5/8/2019 at 6:16 PM, SaltStation said:

CapabilityManager.INSTANCE.register(IFatigue.class,new FatigueStorage(),Fatigue.class);

Expand  

Don't use this overload, it's deprecated. Use the one that takes in a Callable as it's last parameter. I think that this is the case of your issue since you don't provide a public parameterless constructor in your implementation.

 

  On 5/8/2019 at 6:16 PM, SaltStation said:

public void registerItemRenderer(Item item, int meta, String id) {

Expand  

Don't use IHasModel?