How make custom wall?

[MairwunNx]

Hi guys! I fixed the fence myself, but why can't I make a wall like fence? The method super offers me a model Block, but I do not understand what to write?

[MairwunNx]

I need help

[MairwunNx]

  On 8/7/2018 at 10:06 AM, diesieben07 said:

Show what you tried.

Expand  

package com.mairwunnx.mnxemeralds.blocks;

import net.minecraft.block.Block;
import net.minecraft.block.BlockWall;

public class emeraldwall extends BlockWall
{
    public emeraldwall(String name, float resistance, float hardness, Block modelBlock)
    {
        super(modelBlock);
        setUnlocalizedName(name);
        setRegistryName(name);
        setResistance(resistance);
        setHardness(hardness);
        this.useNeighborBrightness = true;
        setLightLevel(20.0F);
    }
}

 

it's Emerald Wall class.

 

But, i don't know as call emeraldwall method.

 

public static Block KEY4 = new emeraldwall("emerald_wall", 13.5F, 20.0F, ??? ).setCreativeTab(defaultModTab);

 

 ? ? ? - I do not know what argument to put, i try other blocks, e.g KEY1 (is emerald_hardness_block), But then in the tab with the mod, I have two blocks.

[MairwunNx]

  On 8/7/2018 at 10:18 AM, diesieben07 said:

• Problematic code, issue 10 and 14.
• modelBlock tells BlockWall it's properties (like hardness, resistance, etc.), that's all.

• Show more of your code.

Expand  

yes, but what put to modelBlock? BlockWall not work, and other blocks not work. 

 

in BlockWall class i find it:

 

But, what is it blockMaterial? It's don't work.

 

More code:

 

emeraldwall class:

 

package com.mairwunnx.mnxemeralds.blocks;

import net.minecraft.block.Block;
import net.minecraft.block.BlockWall;

public class emeraldwall extends BlockWall
{
    public emeraldwall(String name, float resistance, float hardness, Block modelBlock)
    {
        super(modelBlock);
        setUnlocalizedName(name);
        setRegistryName(name);
        setResistance(resistance);
        setHardness(hardness);
        this.useNeighborBrightness = true;
        setLightLevel(20.0F);
    }
}

 

blockregister class:

 

package com.mairwunnx.mnxemeralds.registers;

import com.mairwunnx.mnxemeralds.blocks.*;
import net.minecraft.block.Block;
import net.minecraft.block.BlockStairs;
import net.minecraft.block.BlockWall;
import net.minecraft.client.Minecraft;
import net.minecraft.client.renderer.block.model.ModelResourceLocation;
import net.minecraft.item.Item;
import net.minecraft.item.ItemBlock;
import net.minecraftforge.fml.common.registry.ForgeRegistries;
import net.minecraftforge.fml.relauncher.Side;
import net.minecraftforge.fml.relauncher.SideOnly;

import static com.mairwunnx.mnxemeralds.mnxemeralds.defaultModTab;

public class blockregister
{
    public static Block KEY = new emeraldhardblock("emerald_hard_block").setCreativeTab(defaultModTab);
    public static Block KEY1 = new emeraldfence("emerald_fence", 13.5F, 20.0F).setCreativeTab(defaultModTab);
    public static Block KEY2 = new emeraldhalfslab("emerald_slab", 13.5F, 20.0F).setCreativeTab(defaultModTab);
    public static Block KEY3 = new emeraldstairs("emerald_stairs", 13.5F, 20.0F, KEY.getDefaultState()).setCreativeTab(defaultModTab);
    public static Block KEY4 = new emeraldwall("emerald_wall", 13.5F, 20.0F, ???).setCreativeTab(defaultModTab);

    public static void register()
    {
        setRegister(KEY);
        setRegister(KEY1);
        setRegister(KEY2);
        setRegister(KEY3);
        setRegister(KEY4);
    }

    @SideOnly(Side.CLIENT)
    public static void registerRender()
    {
        setRender(KEY);
        setRender(KEY1);
        setRender(KEY2);
        setRender(KEY3);
        setRender(KEY4);
    }

    private static void setRegister(Block block)
    {
        ForgeRegistries.BLOCKS.register(block);
        ForgeRegistries.ITEMS.register(new ItemBlock(block).setRegistryName(block.getRegistryName()));
    }

    @SideOnly(Side.CLIENT)
    private static void setRender(Block block)
    {
        Minecraft.getMinecraft().getRenderItem().getItemModelMesher().register(Item.getItemFromBlock(block), 0, new ModelResourceLocation(block.getRegistryName(), "inventory"));
    }
}

 

 

[MairwunNx]

  On 8/7/2018 at 10:42 AM, diesieben07 said:

You need to pass in the Block you want the wall to resemble. In your case: Emerald.

Expand  

Ok, i make:

 

 

 

in hand: 

 

no normal \ normal  https://imgur.com/a/qBKx8Bn

 

but i see two blocks (left normal, right no normal) 

 

Edited August 7, 2018 by MairwunNx

[MairwunNx]

  On 8/7/2018 at 12:36 PM, diesieben07 said:

I have no idea what you are saying.

Two blocks? What are they called? Post logs.

Expand  

Yes, two blocks, called equally, 1 block - tile.emerald_wall.name, 2 block tile.emerald_wall.name

 

Logs: debug.logFetching info...

[MairwunNx]

  On 8/7/2018 at 1:00 PM, diesieben07 said:

From the code you posted it is not possible to know why.

Expand  

https://github.com/MairwunNx/mnxemeralds-1.12.2 see, but But i already given you the code for that one. 

 

Updated log: debug.log

 

 

Edited August 7, 2018 by MairwunNx

[MairwunNx]

UPD: i fixed naming, i make, all norm, but ... tow block still is..... 

 

left - normal, right also normal, but with out texture and still is.

 

two block caused:  100%

Block modelBlock

 

 

Logs: debug.log

 

Edited August 7, 2018 by MairwunNx

[MairwunNx]

  On 8/7/2018 at 2:03 PM, diesieben07 said:

• Problematic code, issue 1, 2, 10. You need to fix these.
• BlockWall has an implementation of getSubBlocks that is not suitable for your block. Extending BlockWall is not the way to go.

Expand  

Ok :(, but what is fix with out BlockWall, and with my bad block register.

[MairwunNx]

  On 8/7/2018 at 2:57 PM, diesieben07 said:

I have no idea what your question is.

Expand  

Sorry, i use google translate. Then how do I make a wall with what I have?

It is fully working and visible, but I interfere with the extra block.

 

I also try replace code: 

 

    public void getSubBlocks(CreativeTabs itemIn, NonNullList<ItemStack> items)
    {
        for (BlockWall.EnumType blockwall$enumtype : BlockWall.EnumType.values())
        {
            items.add(new ItemStack(this, 1, blockwall$enumtype.getMetadata()));
        }
    }

 

on:

 

    public void getSubBlocks(CreativeTabs itemIn, NonNullList<ItemStack> items)
    {
        for (BlockWall.EnumType blockwall$enumtype : BlockWall.EnumType.values())
        {
            items.add(new ItemStack(this, 0, blockwall$enumtype.getMetadata()));
        }
    }

 

But it remove all my wall blocks of tab. But walls available on game with help recipes or command give.

[Draco18s]

  On 8/7/2018 at 3:03 PM, MairwunNx said:

items.add(new ItemStack(this, 0, blockwall$enumtype.getMetadata()));

Expand  

You told the game that your subblocks are stack size 0 (and thus, air). I don't think that's what you wanted.

[MairwunNx]

  On 8/7/2018 at 3:42 PM, Draco18s said:

You told the game that your subblocks are stack size 0 (and thus, air). I don't think that's what you wanted.

Expand  

yes, i already it understand when i put 2

[Draco18s]

You shouldn't have even needed to try.

For on, the method parameters are well labled and for two, the third parameter already being passed has a getMetadata() call. GOSH I WONDER WHAT THAT IS FOR.

[MairwunNx]

  On 8/7/2018 at 3:47 PM, Draco18s said:

You shouldn't have even needed to try.

For on, the method parameters are well labled and for two, the third parameter already being passed has a getMetadata() call. GOSH I WONDER WHAT THAT IS FOR.

Expand  

Ok, sorry i don't understand it's ?

 

For on, the method parameters are well labled and for two, the third parameter already being passed has a getMetadata() call.

[MairwunNx]

I need help please  

[Cadiboo]

  On 8/7/2018 at 4:59 PM, MairwunNx said:

I need help please  

Expand  

Do you know what metadata is? I assume that google translate is mangling it so.

Metadata = https://minecraft.gamepedia.com/Java_Edition_data_values

[Cadiboo]

  On 8/8/2018 at 4:46 AM, Cadiboo said:

Do you know what metadata is? I assume that google translate is mangling it so.

Metadata = https://minecraft.gamepedia.com/Java_Edition_data_values

Expand  

setMetadata(<value>) sets the data

[MairwunNx]

  On 8/8/2018 at 4:48 AM, Cadiboo said:

setMetadata(<value>) sets the data

Expand  

How it use? please

[Cadiboo]

  On 8/8/2018 at 9:07 AM, MairwunNx said:

How it use? please

Expand  

 

  On 8/7/2018 at 3:47 PM, Draco18s said:

You shouldn't have even needed to try.

For on, the method parameters are well labled and for two, the third parameter already being passed has a getMetadata() call. GOSH I WONDER WHAT THAT IS FOR.

Expand  

 

[MairwunNx]

  On 8/8/2018 at 9:10 AM, Cadiboo said:

 

 

Expand  

 

I try call getMetadata() and return null or 0 or 1, it's not work or i stupid? 

[Cadiboo]

  On 8/8/2018 at 9:27 AM, MairwunNx said:

 

I try call getMetadata() and return null or 0 or 1, it's not work or i stupid? 

Expand  

did you read the article and/or anything you were told? As a general rule, if your item doesn't have subtypes its metadata is 0. Not 1 and definitely not null (Are integers even allowed to be null?)

[MairwunNx]

  On 8/8/2018 at 9:30 AM, Cadiboo said:

did you read the article and/or anything you were told? As a general rule, if your item doesn't have subtypes its metadata is 0. Not 1 and definitely not null (Are integers even allowed to be null?)

Expand  

I feel a little stupid. :), If it doesn't have subtypes and I return 0, then I shouldn't have the second block?

 

 

[Cadiboo]

  On 8/8/2018 at 9:35 AM, MairwunNx said:

I feel a little stupid. :), If it doesn't have subtypes and I return 0, then I shouldn't have the second block?

 

 

Expand  

this code has me screaming for at least 7 reasons

1) naming conventions

2) someone said not to extend block wall

3) I assume resistance doesn't change? and that your not creating multiple instances of the block? then hardcode it

4) same as resistance but for hardness

5) why is unlocalised name before registry name,

6) not that it matters, they should be in a different class (your base mod block class)

7) why are you calling getMetadata?

why are you overriding getMetadata?

9) assuming getMetadata actually did something useful, why aren't you doing anything with the returned value?

10) wheres your @Override annotation? its important

Edited August 8, 2018 by Cadiboo