How do I create a LootPool with multiple entries

[skeddles]

I originally created my LootPool by making my LootEntry a LootEntryTable that gets the items from a loot table json, but now I want to do it with code. I figured out how to create a loot entry for a single item by using LootEntryItem instead of LootEntryTable, but how can I make a list of multiple items?

 

 

//loot tables
@SubscribeEvent
public static void lootLoad(LootTableLoadEvent evt) {
   String name = evt.getName().toString();
   
   if (name.contains("chest")) {
      LootEntry entry = new LootEntryItem(Items.COOKIE, 15, 60, new LootFunction[0], new LootCondition[0], "modpoolcustomtest");

      LootPool pool = new LootPool(new LootEntry[] {entry}, new LootCondition[0], new RandomValueRange(1), new RandomValueRange(1), "modpool"); 
      
      evt.getTable().addPool(pool);
   }
}

Edited September 9, 2018 by skeddles

[Draco18s]

Take a look at this:

https://github.com/Draco18s/ReasonableRealism/blob/1.12.1/src/main/java/com/draco18s/hardlib/util/LootUtils.java

[skeddles]

I didn't realize that it could be an array.  This code seems to work now.

 

I haven't used arrays before, how does this look? Would there be any performance loss doing it this way, even if there was hundreds or thousands of items?

 

//loot tables
@SubscribeEvent
public static void lootLoad(LootTableLoadEvent evt) {
   String name = evt.getName().toString();
   
   if (name.contains("chest")) {

      LootEntry[] entry = {
              new LootEntryItem(Items.COOKIE, 1, 60, new LootFunction[0], new LootCondition[0], "test:cookie"),
              new LootEntryItem(Items.SKULL, 1, 60, new LootFunction[0], new LootCondition[0], "test:skull")
      };

      LootPool pool = new LootPool(entry, new LootCondition[0], new RandomValueRange(1), new RandomValueRange(1), "testpool");
      
      evt.getTable().addPool(pool);
   }
}

 

edit: just realized i need to be able to add to the array at will so probably need an arraylist, workin on that

 

edit:

here's what I got now, uses an arraylist and then converts it at the end:

 

ArrayList<LootEntryItem> entries = new ArrayList<LootEntryItem>();

entries.add(new LootEntryItem(Items.COOKIE, 1, 60, new LootFunction[0], new LootCondition[0], "examplemod:cookie"));
entries.add(new LootEntryItem(Items.SKULL, 1, 60, new LootFunction[0], new LootCondition[0], "examplemod:skull"));

LootEntry [] entriesArray = entries.toArray(new LootEntry[entries.size()]);

LootPool pool = new LootPool(entriesArray, new LootCondition[0], new RandomValueRange(1), new RandomValueRange(1), "heroicarmorypool");

evt.getTable().addPool(pool);

Edited September 9, 2018 by skeddles

[Draco18s]

Yeah. Gotta be an array, but lists are easier to work with.

Its why my utils class got so convoluted so quickly.