[1.10.2] Underground structure generation problem

[JimiIT92]

So i'm trying to spawn a structure underground but the problem is that is too common, despite i give it a really low chance of spawn. This is the code i'm using in the WorldGenerator class

if (world.getBiomeGenForCoords(new BlockPos(x, 30, z)).equals(Biomes.ICE_MOUNTAINS)
				|| world.getBiomeGenForCoords(new BlockPos(x, 30, z)).equals(Biomes.ICE_PLAINS)
				|| world.getBiomeGenForCoords(new BlockPos(x, 30, z)).equals(Biomes.MUTATED_ICE_FLATS)
				|| world.getBiomeGenForCoords(new BlockPos(x, 30, z)).equals(MWWorld.arctic)
				|| world.getBiomeGenForCoords(new BlockPos(x, 30, z)).equals(MWWorld.iceHills)
				&& random.nextInt(1000) == 1) {
			int randPosX = x + random.nextInt(35);
			int randPosZ = z + random.nextInt(35);
			int randPosY = 30 + random.nextInt(10);
			BlockPos position = new BlockPos(randPosX, randPosY, randPosZ);
			if (!(world.getBlockState(position).getBlock().equals(Blocks.WATER)))
				new WorldGenIceDungeon(position, world);
		}

 

And this is the amount of structures that are generated

As you can see there are too much of this. So how can i spawn the structure with the properly rarity?

[Draco18s]

Also wrap your booleans.

 

By having them unwrapped it does this:

if (world.getBiomeGenForCoords(new BlockPos(x, 30, z)).equals(Biomes.ICE_MOUNTAINS)...

"Hmm, is it ice mountains?  Yes, yes it is.  This whole statement evaluates to true."

 

And bypasses your random call.

 

https://en.wikipedia.org/wiki/Order_of_operations#Programming_languages

A || B && C ≡ A || (B && C)

[JimiIT92]

*facepalm* totally forgot the wrap! And that's why i should never program when i'm sick, that was a really dumb "error" Thanks for pointed it out, now it works