Non-Blockstate values in IBlockColor [1.15.1]

[Jipthechip]

My BlockState became too complicated for the values I wanted to feed into my IBlockColor#getColor() implementation.

 

Is there any alternative to getting values from the BlockState, or will I have to use a TER/TESR?

Edited April 20, 2020 by Jipthechip

[Animefan8888]

  On 4/20/2020 at 12:52 AM, Jipthechip said:

My BlockState became too complicated for the values I wanted to feed into my IBlockColor#getColor() implementation.

 

Is there any alternative to getting values from the BlockState, or will I have to use a TER/TESR?

Expand  

I'm not sure if this still works, but you might be able to cast the ILightReader to a World(do an instanceof check it can be null). Then if it is use World::getTileEntity and store your colors in the TileEntity.

[Animefan8888]

  On 4/20/2020 at 7:47 AM, diesieben07 said:

No need for the cast, ILightReader extends IBlockReader, which has getTileEntity.

Expand  

Ah ok I didn't get that from the name, and didn't check either. Either way they still have to check for null.

[Jipthechip]

  On 4/20/2020 at 1:24 AM, Animefan8888 said:

I'm not sure if this still works, but you might be able to cast the ILightReader to a World(do an instanceof check it can be null). Then if it is use World::getTileEntity and store your colors in the TileEntity.

Expand  

  On 4/20/2020 at 7:47 AM, diesieben07 said:

No need for the cast, ILightReader extends IBlockReader, which has getTileEntity.

Expand  

 

Not casting it worked, but I tried casting it anyway just to see what would happen and it crashed. It gave the error:

java.lang.ClassCastException: net.minecraft.client.renderer.chunk.ChunkRenderCache cannot be cast to net.minecraft.world.World

 

[Animefan8888]

  On 4/20/2020 at 2:50 PM, Jipthechip said:

but I tried casting it anyway just to see what would happen and it crashed. It gave the error:

Expand  

  On 4/20/2020 at 1:24 AM, Animefan8888 said:

(do an instanceof check

Expand  

Also it can be null make sure it is not null before you use it.

[Jipthechip]

  On 4/20/2020 at 2:53 PM, Animefan8888 said:

Also it can be null make sure it is not null before you use it.

Expand  

  On 4/20/2020 at 1:24 AM, Animefan8888 said:

(do an instanceof check it can be null).

Expand  

I did that, and it just ended up never getting to the code I wanted it to.

 

[Animefan8888]

  On 4/20/2020 at 2:56 PM, Jipthechip said:

I did that, and it just ended up never getting to the code I wanted it to.

Expand  

Which means I was wrong about it being a World instance when you need it to be.

  On 4/20/2020 at 1:24 AM, Animefan8888 said:

I'm not sure if this still works, but you might be able to cast the ILightReader to a World

Expand  

 

[Jipthechip]

  On 4/20/2020 at 2:59 PM, diesieben07 said:

Show your code.

Expand  

My code was just this

assert lightReader != null;
assert pos != null;
if(lightReader instanceof World){
    // misc code never reached
}else{
    return MaterialColor.WATER.colorValue;
}

Like I said, not casting the ILightReader worked just fine. I was just curious if it could be cast to a World, and it doesn't seem like it can.

Edited April 20, 2020 by Jipthechip