Animefan8888

Where did you declare the variable.

Where is the ItemStack stored?

How much Java experience do you have?

Yes that method of registry is outdated. That is correct if your changed the Block in Register<Block> to Item

Could you post your DragonNest class? I'm pretty sure you are going to need that BlockPos.

Anywhere in your project. It doesn't really matter where.

It is really simple and if I get anything wrong I really want someone to correct me. Make a class that has the @Mod.EventBusSubscriber and in that class put the method in the documentation. @SubscribeEvent public void registerBlocks(RegistryEvent.Register<Block> event) { event.getRegistry().registerAll(block1, block2, ...); } And where it says Register<Block> you can change Block to any class that extends IForgeRegistryEntry.Impl. This includes Item. Then event.getRegistry().register... is how you register your Items, Blocks, etc.

That piece of code is entirely useless for your means. You are checking if a 9x6x11 space is AIR. You don't want to do that.

This is documentation on the new registration system.

This part checks to see if a 9x11xHeight space is AIR. Which will almost never happen underground.

Random#nextInt(78 - 65) + 65 Will give you a random number between 65 and 78.

Pretty much. At most your structure of 30x30 could belong in 4 chunks. Which means 3 extra chunks are possible when the world is generating, but the problem with that is when it loads those chunks it is possible that, that chunk will also want to generate it there to. There is almost nothing you can do about it taking up four chunks and those chunks generating.

Yes, now it will spawn 78 blocks below the top of the ground, which could be negative, which is out of bounds so you will want to check for that.

To make it generate underground you just need to make the y variable smaller.

Then problem solved, right?

What you will want to do is make a List of ChunkPos that it has generated in. And if the position is to close to one in the list don't generate.

You also need to override getMaxItemUseDuration

You will have to make a method to do it, I would not necessarily do it before because then you will have to call World#setBlockState more times than necessary. Instead place your building in the world and then clear the area above it. Also do take into account what jeffryfisher said because what he said can and will happen which causes world gen lag because it can spiral out of control. It is called Runaway Chunk Generation.

If I am not mistaken you do not need a ChunkProvider for Structure Generation. I think the biggest problem you have is that you check if the block is replaceable, why? Instead you should get the first ground block at the x, z position you get, and then tell the game to generate your structure. If there are air blocks or plants underneath then simply replace them with a foundation type of block. Kinda like what Minecraft does with it's village buildings, and then clear above your structure by replacing with air. Not all the way up to build height mind you, maybe a couple blocks if the structure is huge and if not clear until there is nothing connected above it.

No problem.

If it returns true it believes you did something and the onItemRighClick/onItemUse shouldn't be called. Edit: It also tells the client to play the animation of the player moving their hand when true is returned.

Ok....so you registered a world generator that possibly generates a house 10 times per every chunk that gets loaded. In the exact same place as it was the first time it generated in that chunk. Also i think your house is a 9 * 9 * 6 prism of stone brick. Try Not generating it 10 times every chunk and instead make it only appear in 1/50 chunks by removing the for loop in NeemWorldGenerator#generateOverWorld.

Sorry I miss the small details a lot, have you tried stepping through it with the debugger because it is not popping out at me.

I was taking a shot in the dark...where are you appending that "X" at?

Change this to return itemStack.getTagCompound();