[1.7.2]Best way to add an entity spawn

[burnner]

Hello,

what's the proper way to add a spawn?

Is that the right way:

 

EntityRegistry.addSpawn(EntityPig2.class, 1, 1, 1, EnumCreatureType.creature, BiomeGenBase.desert,

                    BiomeGenBase.desertHills,

                    BiomeGenBase.extremeHills,

                    BiomeGenBase.extremeHillsEdge,

                    BiomeGenBase.forest,

                    BiomeGenBase.forestHills,

                    BiomeGenBase.frozenOcean,

                    BiomeGenBase.frozenRiver,

                    BiomeGenBase.hell,

                    BiomeGenBase.iceMountains,

                    BiomeGenBase.icePlains,

                    BiomeGenBase.jungle,

                    BiomeGenBase.jungleHills,

                    BiomeGenBase.mushroomIsland,

                    BiomeGenBase.ocean,

                    BiomeGenBase.plains,

                    BiomeGenBase.river,

                    BiomeGenBase.swampland,

                    BiomeGenBase.taiga);

 

 

 

burnner

[f1rSt1kChannel]

EntityRegistry.addSpawn(EntityPig2.class, 1, 1, 1, EnumCreatureType.creature, BiomeGenBase.biomeList);

[SanAndreaP]

  On 3/23/2014 at 1:16 PM, f1rSt1kChannel said:

EntityRegistry.addSpawn(EntityPig2.class, 1, 1, 1, EnumCreatureType.creature, BiomeGenBase.biomeList);

 

No, this will crash, since biomeList contains null values!

Look here for a solution:

http://www.minecraftforge.net/forum/index.php/topic,17455.msg88353.html#msg88353

[burnner]

Thanks for reply, but biomList isn't visible?

[SanAndreaP]

there's no "biomList"

Also since when is biomeList not visible?

You checked the BiomeGenBase class for it and made sure it's really not visible? If it isn't, is there a getter method?

[coolboy4531]

I think he means that its not biomeList, he can't find it.

 

@OP:

Probably as field/func names.

[f1rSt1kChannel]

  On 3/23/2014 at 2:47 PM, SanAndreasP said:

  Quote

EntityRegistry.addSpawn(EntityPig2.class, 1, 1, 1, EnumCreatureType.creature, BiomeGenBase.biomeList);

 

No, this will crash, since biomeList contains null values!

Look here for a solution:

http://www.minecraftforge.net/forum/index.php/topic,17455.msg88353.html#msg88353

I am sorry to have confused.

[sequituri]

Try BiomeGenBase$getBiomeGenArray()

 

It returns the biomeList

[burnner]

  On 3/23/2014 at 4:47 PM, SanAndreasP said:

there's no "biomList"

Also since when is biomeList not visible?

You checked the BiomeGenBase class for it and made sure it's really not visible? If it isn't, is there a getter method?

Eclipse says it's not visibile

[SanAndreaP]

  On 3/24/2014 at 6:29 AM, burnner said:

  Quote

there's no "biomList"

Also since when is biomeList not visible?

You checked the BiomeGenBase class for it and made sure it's really not visible? If it isn't, is there a getter method?

Eclipse says it's not visibile

 

Well, then it's changed in 1.7. Use sequituri's suggestion then.

[f1rSt1kChannel]

Use this:

public static BiomeGenBase[] biomesWithout(BiomeGenBase... biomesWithout){
        ArrayList<BiomeGenBase> biomes = new ArrayList<BiomeGenBase>();
        for(BiomeGenBase biome : BiomeGenBase.biomeList){
            for(int i = 0; i < biomesWithout.length; i++){
                if(biome != null && biome != biomesWithout[i]){
                    biomes.add(biome);
                }
            }
        }
        return biomes.toArray(new BiomeGenBase[biomes.size()]);
    }

And:

EntityRegistry.addSpawn(EntityPig2.class, 1, 1, 1, EnumCreatureType.creature, biomesWithout(/** here write biomes in which the mob will not spawn */));

[burnner]

I think BiomeGenBase.getBiomeGenArray() will do it 

[sequituri]

I'm pretty sure I said that.