Well java is annoying. Help me out

[Ghast]

This is my method to show rewards,

 public void showRewards(Player p, Bossraid raid, boolean regular) {
        Inventory inv;
        RewardList list;
        if (regular) {
            inv = Bukkit.createInventory(null, 54, ChatColor.RED + "Regular rewards for " + raid.getName());
            list = HandlerManager.getRewardHandler().getRewardList(raid.getRegularRewardList());

        } else {
            inv = Bukkit.createInventory(null, 54, ChatColor.RED + "Epic rewards for " + raid.getName());
            list = HandlerManager.getRewardHandler().getRewardList(raid.getEpicRewardList());
        }
        if (!list.isReady()) {
            MessageManager.message(p, ChatColor.RED + "List is not yet finalized.");
        } else {
            list.getItems().forEach((i, c) -> {
                inv.addItem(ItemFactory.createItem(i, "", ChatColor.GREEN + "Chance: " + ChatColor.YELLOW + i + "%"));
            });
            p.openInventory(inv);
        }
    }

Reward handler:

public RewardList getRewardList(String name) {
        Optional<RewardList> o = rewards.stream().filter(rewardList -> rewardList.getName().equalsIgnoreCase(name)).findAny();
        return o.isPresent() ? o.get() : null;
    }

This is my command to show rewards:

if (args[1].equalsIgnoreCase("rewards")) {
                if (args[2].equalsIgnoreCase("regular")) {
                    BossraidManager.getInstance().showRewards(p, raid, true);
                    return true;
                }
                if (args[2].equalsIgnoreCase("epic")) {
                    BossraidManager.getInstance().showRewards(p, raid, false);
                    return true;
                }
                MessageManager.message(p, Message.NO_SUCH_LIST.toString());
                return true;
            }

And this is the wall I keep hitting:

 

[Ghast]

Cole

 

[Cole]

What's line 81 in the boss raid manager class? It's returning null

 

[Ghast]

What's line 81 in the boss raid manager class? It's returning null

if (!list.isReady()) {

 

[Ghast]

Well, it's pretty clear that list is null. Since getRewardList can return null you should add in a null check before you attempt to see if it's ready.

So you can try to change

if (!list.isReady()) {

}

to

if (list == null || !list.isReady()) {

}

Since the list is null, you should go over getRewardList and follow wherever it's retrieving its information from and try to see if you can spot why it might be returning null.

I appreciate the spoon feed, I guess I learnt more about nulls :shrug:
Thank you Carlos.