stijn/HungerGames
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix null-check for loot items and adjust rarity handling in generateLoot method

Browse Source
This commit is contained in:
akastijn 2026-06-15 22:15:45 +02:00
parent 25c95dd46a
commit 8ba514c9c2
1 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/main/java/com/alttd/hunger_games/services/LootService.java
View File

@ -2,6 +2,7 @@ package com.alttd.hunger_games.services;
import com.alttd.hunger_games.config.LootItems; import com.alttd.hunger_games.config.LootItems;
import com.alttd.hunger_games.data_objects.RARITY; import com.alttd.hunger_games.data_objects.RARITY;
import lombok.extern.slf4j.Slf4j;
import net.kyori.adventure.text.Component; import net.kyori.adventure.text.Component;
import org.bukkit.Bukkit; import org.bukkit.Bukkit;
import org.bukkit.Location; import org.bukkit.Location;
@ -14,6 +15,7 @@ import java.util.List;
import java.util.Random; import java.util.Random;
import java.util.concurrent.ThreadLocalRandom; import java.util.concurrent.ThreadLocalRandom;
@Slf4j
public class LootService { public class LootService {
private final HashMap<Location, Inventory> chestInventories = new HashMap<>(); private final HashMap<Location, Inventory> chestInventories = new HashMap<>();
@ -31,16 +33,18 @@ public class LootService {
private Inventory generateLoot() { private Inventory generateLoot() {
Inventory inventory = Bukkit.createInventory(null, 27, Component.text("Chest")); Inventory inventory = Bukkit.createInventory(null, 27, Component.text("Chest"));
RARITY rarity = decideRarity();
List<Material> possibleItems = LootItems.ITEMS.get(rarity);
if (possibleItems.isEmpty()) {
return inventory;
}
int itemCount = random.nextInt(3, 7); int itemCount = random.nextInt(3, 7);
for (int i = 0; i < itemCount; i++) { for (int i = 0; i < itemCount; i++) {
int slot = random.nextInt(27); int slot = random.nextInt(27);
RARITY rarity = decideRarity();
List<Material> possibleItems = LootItems.ITEMS.get(rarity);
if (possibleItems == null || possibleItems.isEmpty()) {
log.warn("No items found for rarity: {}", rarity);
continue;
}
Material material = possibleItems.get(random.nextInt(possibleItems.size())); Material material = possibleItems.get(random.nextInt(possibleItems.size()));
inventory.setItem(slot, new ItemStack(material)); inventory.setItem(slot, new ItemStack(material));
} }