Want to modify every MONSTER loot table

[BurstSword]

Hi, Im so noob modding and I want to know if I can add my custom items to only monster loot tables with lootLoad() without writing infinite cases or else ifs

 

Thanks

[BurstSword]

@SubscribeEvent
    public void lootLoad(LootTableLoadEvent evt) {
        if (evt.getName().toString().equals("minecraft:chests/buried_treasure")) {
            evt.getTable().addPool(LootPool.builder().addEntry(TableLootEntry.builder(new ResourceLocation(Constants.MODID, "inject/buried_treasure"))).build());
        } else if (evt.getName().toString().equals("minecraft:gameplay/fishing")) {
            evt.getTable().addPool(LootPool.builder().addEntry(TableLootEntry.builder(new ResourceLocation(Constants.MODID, "inject/fishing"))).build());
        } else if (evt.getName().toString().equals("minecraft:entity/phantom")) {
            evt.getTable().addPool(LootPool.builder().addEntry(TableLootEntry.builder(new ResourceLocation(Constants.MODID, "inject/phantom"))).build());
        }
    }

This is my code for injections in the loot tables, but now if I want to change monsters loot tables i have to write one else if ( or switch cases) for each monster, I want to know if there is an easier way to do it.

 

Thanks you for the fast reply

[BurstSword]

Just now, diesieben07 said:

Again, please use the global loot modifier system instead of using code to modify the tables.

Could you explain me what it is or send me a link to see it?

 

Thank you

[Draco18s]

https://github.com/MinecraftForge/MinecraftForge/pull/6401

[BurstSword]

Can I see an implementation of this? Only with that link I don´t know how to make it work

[Ugdhar]

Here's another thread about it with some links for more information, have you checked it out? Found it by searching the forums for "global loot modifiers"

https://www.minecraftforge.net/forum/topic/83204-1152-globallootmodifier-not-working/?tab=comments#comment-393325

 

[BaconBombing]

This isn't quite global variables, but the way I cut down on code was writing a function with a switch case that uses a prefix to cut down on string length for every string that was applicable. 

 

The "modifyLoot" functions probably won't be useful to you in this case because you're not modifying original Loot Pools but you can modify it to your liking.

 

So your prefix in this case would use "entities" instead, and you would just have to write every monster name once.

 

https://github.com/EricHedengren/StandardMaterials/blob/master/com/baconbombing/standardmaterials/loot/ModLoot.java

 

Hope this helps!

Edited April 11, 2020 by BaconBombing

[Turtledove]

^^^^^^ this. It's more work the other way anyways.