From 4ddd416e60a5c42ef0ef6a3af6af0547cc7009fa Mon Sep 17 00:00:00 2001
From: IThundxr <me@ithundxr.dev>
Date: Fri, 1 Nov 2024 23:03:38 -0400
Subject: [PATCH] Improve code a bit

---
 .../BlockStateBlockItemGroupRecipeList.java   | 20 +++++++++----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/common/src/main/java/com/railwayteam/railways/base/data/recipe/BlockStateBlockItemGroupRecipeList.java b/common/src/main/java/com/railwayteam/railways/base/data/recipe/BlockStateBlockItemGroupRecipeList.java
index b4ae7eaf2..e803abb53 100644
--- a/common/src/main/java/com/railwayteam/railways/base/data/recipe/BlockStateBlockItemGroupRecipeList.java
+++ b/common/src/main/java/com/railwayteam/railways/base/data/recipe/BlockStateBlockItemGroupRecipeList.java
@@ -18,27 +18,25 @@
 
 package com.railwayteam.railways.base.data.recipe;
 
+import com.google.common.collect.Streams;
 import com.railwayteam.railways.content.buffer.BlockStateBlockItem;
 import com.railwayteam.railways.content.buffer.BlockStateBlockItemGroup;
 import com.tterrag.registrate.util.entry.ItemEntry;
-import org.apache.commons.compress.utils.Lists;
 import org.jetbrains.annotations.NotNull;
 
 import java.util.Iterator;
+import java.util.List;
 import java.util.NoSuchElementException;
 import java.util.function.Function;
 
 public class BlockStateBlockItemGroupRecipeList implements Iterable<RailwaysRecipeProvider.GeneratedRecipe> {
-    protected final RailwaysRecipeProvider.GeneratedRecipe[] values;
+    private final List<RailwaysRecipeProvider.GeneratedRecipe> values;
 
     public BlockStateBlockItemGroupRecipeList(BlockStateBlockItemGroup<?, ?> group, Function<@NotNull ItemEntry<? extends BlockStateBlockItem<?>>, RailwaysRecipeProvider.GeneratedRecipe> filler) {
-        values = new RailwaysRecipeProvider.GeneratedRecipe[Lists.newArrayList(group.getItems().iterator()).size()];
-        
-        int i = 0;
-        
-        for (ItemEntry<? extends BlockStateBlockItem<?>> entry : group.getItems()) {
-            values[i++] = filler.apply(entry);
-        }
+        this.values = Streams
+                .stream(group.getItems())
+                .map(filler)
+                .toList();
     }
     
     @Override
@@ -48,14 +46,14 @@ public BlockStateBlockItemGroupRecipeList(BlockStateBlockItemGroup<?, ?> group,
 
             @Override
             public boolean hasNext() {
-                return index < values.length;
+                return index < values.size();
             }
 
             @Override
             public RailwaysRecipeProvider.GeneratedRecipe next() {
                 if (!hasNext())
                     throw new NoSuchElementException();
-                return values[index++];
+                return values.get(index++);
             }
         };
     }