jameshyland Posted December 9, 2015 Share Posted December 9, 2015 So i have been making a mod for the past week or so and I have stumbled upon problems when generating ores. I have entered the code, but the ores dont show up in game. Here is the code for the ore in question package com.jameshyland.MoneyMod; import java.util.Random; import net.minecraft.block.Block; import net.minecraft.init.Blocks; import net.minecraft.world.World; import net.minecraft.world.chunk.IChunkProvider; import net.minecraft.world.gen.feature.WorldGenMinable; import cpw.mods.fml.common.IWorldGenerator; public class OreGeneration implements IWorldGenerator { @Override public void generate(Random random, int chunkX, int chunkZ, World world, IChunkProvider chunkGenerator, IChunkProvider chunkProvider) { switch (world.provider.dimensionId) { case 0: generateEnd(world, random, chunkX, chunkZ ); break; case 1: generateSurface(world, random, chunkX, chunkZ); break; case -1: generateNether(world, random, chunkX, chunkZ); break; } } public void generateEnd(World world, Random rand, int x, int z) { generateOre(MoneyMod.blockMoneyOre,world, rand, x, z, 5, 20, 65, 20, 250, Blocks.end_stone); } public void generateSurface(World world, Random rand, int x, int z) { generateOre(MoneyMod.blockMoneyOre,world, rand, x, z, 5, 20, 65, 20, 250, Blocks.stone); } public void generateNether(World world, Random rand, int x, int z) { generateOre(MoneyMod.blockMoneyOre,world, rand, x, z, 5, 20, 65, 20, 250, Blocks.netherrack); } public void generateOre(Block block, World world, Random random, int chunkX, int chunkZ, int minVeinSize, int maxVeinSize, int chance, int minY, int maxY, Block generateIn) { int vienSize = minVeinSize + random.nextInt(maxVeinSize - minVeinSize); int heightRange = maxY - minY; WorldGenMinable gen = new WorldGenMinable(block, vienSize, generateIn); for (int i = 0; i < chance; i++) { int xRand = chunkX * 16 + random.nextInt(16); int yRand = random.nextInt(heightRange + minY); int zRand = chunkZ * 16 + random.nextInt(16); gen.generate(world, random, xRand, yRand, zRand); } } } If you could help please do. Thanks. Quote Link to comment Share on other sites More sharing options...
Draco18s Posted December 9, 2015 Share Posted December 9, 2015 Well, it would help if you generated surface ores in the overworld, end ores in the end, and nether orex in the nether. The overworld is dimensipn 0. Quote Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable. If you think this is the case, JUST REPORT ME. Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice. Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked. DO NOT PM ME WITH PROBLEMS. No help will be given. Link to comment Share on other sites More sharing options...
jameshyland Posted December 9, 2015 Author Share Posted December 9, 2015 I dont know what the difference is. This is my first mod and i have little to no experience with Java programming so... Can you please elaborate? Quote Link to comment Share on other sites More sharing options...
Draco18s Posted December 9, 2015 Share Posted December 9, 2015 This code seeds end ores into the overworld. case 0: // dimension 0 is the overworld generateEnd(world, random, chunkX, chunkZ ); // add end ores break; Etc. Quote Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable. If you think this is the case, JUST REPORT ME. Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice. Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked. DO NOT PM ME WITH PROBLEMS. No help will be given. Link to comment Share on other sites More sharing options...
jameshyland Posted December 9, 2015 Author Share Posted December 9, 2015 I already have that in the code though. Quote Link to comment Share on other sites More sharing options...
Draco18s Posted December 9, 2015 Share Posted December 9, 2015 Oh my ducking god. YOUR CODE IS WRONG Quote Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable. If you think this is the case, JUST REPORT ME. Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice. Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked. DO NOT PM ME WITH PROBLEMS. No help will be given. Link to comment Share on other sites More sharing options...
Cerandior Posted December 9, 2015 Share Posted December 9, 2015 There is a switch in your world generation class that is checking for the dimension ID. Draco18 spoke very clearly. 0 is the ID number for the overword dimension, however you are using this ID, the id that belongs to the overworld dimension, to generate in the end. At case 0 in the switch you want to execute the generateSurface method not the generateEnd that you are currently using Quote Link to comment Share on other sites More sharing options...
jameshyland Posted December 10, 2015 Author Share Posted December 10, 2015 It now looks like this: case 1: generateEnd(world, random, chunkX, chunkZ ); break; case 0: generateSurface(world, random, chunkX, chunkZ); break; case -1: generateNether(world, random, chunkX, chunkZ); break; } } public void generateEnd(World world, Random rand, int x, int z) { generateOre(MoneyMod.blockMoneyOre,world, rand, x, z, 2, 10, 5, 0, 100, Blocks.end_stone); } public void generateSurface(World world, Random rand, int x, int z) { generateOre(MoneyMod.blockMoneyOre,world, rand, x, z, 2, 10, 5, 0, 100, Blocks.stone); } public void generateNether(World world, Random rand, int x, int z) { generateOre(MoneyMod.blockMoneyOre,world, rand, x, z, 2, 10, 5, 0, 100, Blocks.netherrack); } public void generateOre(Block block, World world, Random random, int chunkX, int chunkZ, int minVeinSize, int maxVeinSize, int chance, int minY, int maxY, Block generateIn) { int vienSize = minVeinSize + random.nextInt(maxVeinSize - minVeinSize); int heightRange = maxY - minY; WorldGenMinable gen = new WorldGenMinable(block, vienSize, generateIn); for (int i = 0; i < chance; i++) { int xRand = chunkX * 16 + random.nextInt(16); int yRand = random.nextInt(heightRange + minY); int zRand = chunkZ * 16 + random.nextInt(16); gen.generate(world, random, xRand, yRand, zRand); Is that better? Quote Link to comment Share on other sites More sharing options...
ovikk Posted December 10, 2015 Share Posted December 10, 2015 Yes Quote I might be terribly wrong.. Like really, really wrong. But I'm just trying to help. Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.