I've been working on my mod for awhile and i'm trying to add a custom slab but I can't get the MaterialColor  and Material to work because it says they're private. Here's my code:

 

Java Class:

public class CoolSlab extends SlabBlock {
    public CoolSlab() {
        super(new Properties(Material.IRON, MaterialColor.AIR).hardnessAndResistance(12, 40).harvestTool(ToolType.PICKAXE).sound(SoundType.GLASS));
    }
}

 

Deferred Registry:

public static final RegistryObject<Block> COOL_SLAB = BLOCKS.register("cool_slab", CoolSlab::new);

Edited October 10, 20205 yr by Ron_Parker

1 hour ago, Ron_Parker said:

super(new Properties(Material.IRON, MaterialColor.AIR).hardnessAndResistance(12, 40).harvestTool(ToolType.PICKAXE).sound(SoundType.GLASS));

Don't hardcode a parameter, just call the superclass directly. This is a waste of an object and resources.

1 hour ago, Ron_Parker said:

MaterialColor  and Material to work because it says they're private.

They are definitely not private. They are static final fields accessible to any who calls it. You might want to try setting up your workspace again.

Edited October 10, 20205 yr by ChampionAsh5357

It underlines Properties and says 'Properties(net.minecraft.block.material.Material, net.minecraft.block.material.MaterialColor)' has private access in 'net.minecraft.block.AbstractBlock.Properties'

I read what you said and not what the code written was. Use the static constructors like create of from instead of using the new instance ones.

What do those look like?

1 minute ago, Ron_Parker said:

What do those look like?

Properties::create and Properties::from?

Examine existing usages using your IDE? Right click -> Find all References

Ok I used Properties::from but it doesn't seem to like it, There are red underlines around the parentheses around CoolGlass (Java Class):

public static final RegistryObject<Block> COOL_GLASS = BLOCKS.register("cool_glass", AbstractBlock.Properties::from(CoolGlass));

Because :: is not literal Java syntax.

https://stackoverflow.com/questions/27015495/meaning-of-in-java-syntax

Should I use a lambda instead?

 

You are messing up in many fundamental java principals. 

Guys, please don't give him copy paste code as there are many out there he could find if he cared.

The example in DeferredRegister shows what you should be passing in.

Which is a Supplier<Block>

That supplier is typically created via a lambda, but it doesn't have to be, this is a implementation detail that's completely up to you.

That supplier has to return a NEW instance of a Block, or subclass of Block.

To create a Block instance you need the Properties instance.

Which has a private constructor. So you need to figure out if there are any functions you can access that create a new instance of that class.

This is where your IDE comes in, just like Draco said, go to the constructor, right click, find references.

The method you should all should be really obvious.