Hello guys,

 

In the past i have created mobs with random textures, it uses this function inside getentitytexture:
            int id = (int) ((par1Entity.getUniqueID().getLeastSignificantBits() ^ par1Entity.getUniqueID().getMostSignificantBits()) % 3);

so it will basicly be able to use 3 textures at random.

 

this works just fine.

but, i recently created a mob wich has a total of 10 textures but when i change the modules to % 10 it works but isnt completely random anymore i spawned 100 mobs got 40 of texture wich corrosponds to id value 9 but none with value 1 and the rest just dont spawn that often.


is this becouse of the way the uniqueid are made?

What function is better to use?



example:
https://gyazo.com/c4c33d934749112dc726e65cdfc1aecb

blue white shirts corrospond to value 9.
 

 

 

 

	@SideOnly(Side.CLIENT)
	public void registerRenderers() {
		RenderLiving customRender = new RenderLiving(Minecraft.getMinecraft().getRenderManager(), new Civilian.ModelNew(), 0) {
			protected ResourceLocation getEntityTexture(Entity par1Entity) {
			int id = (int) ((par1Entity.getUniqueID().getLeastSignificantBits() ^ par1Entity.getUniqueID().getMostSignificantBits()) % 10);
			
							if(id == 0) {
								return new ResourceLocation("Civilian.png");	
							}
							else if(id == 1) {
								return new ResourceLocation("Civilian1.png");	
							}
							else if(id == 2) {
								return new ResourceLocation("Civilian2.png");
							}
							else if(id == 3) {
								return new ResourceLocation("Civilian3.png");
							}
							else if(id == 4) {
								return new ResourceLocation("Civilian4.png");
							}
							else if(id == 5) {
								return new ResourceLocation("Civilian5.png");
							}
							else if(id == 6) {
								return new ResourceLocation("Civilian6.png");
							}
							else if(id == 7) {
								return new ResourceLocation("Civilian7.png");
							}
							else if(id == 8) {
								return new ResourceLocation("Civilian8.png");
							}
							else {
								return new ResourceLocation("Civilian9.png");
							}
						}
					};
		RenderingRegistry.registerEntityRenderingHandler(Civilian.EntityCivilian.class, customRender);

	}

 

Ew. That if-else block. Yuck.

return new ResourceLocation("Civilian" + id + ".png"); //magic

Also, the reason it isn't random is because the XOR operation isn't random. You're much more likely to get particular values than others.

 

When I did something like this, I used a Mersenne twister. Nothing else got me close.

https://github.com/Draco18s/HarderStuff/blob/master/src/main/java/com/draco18s/hazards/block/BlockUnstableStone.java#L175-L180