Solved Hashmap returning null

I have a for-statement I want to do something for everyone in a hasmap. It has worked just fine until now. I did not change anything.

Class with the for-statement:

Code:java

public class TimerR {
 
    public static boolean tp;
    public MainC pl;
    public TimerR(MainC plugin) {
        pl = plugin;
    }
 
    private A A;
    private OS OS;
    private V SG = new V(pl);
 
    public void Timer() {
        int a = SG.getTimer() -1;
        Bukkit.broadcastMessage(ChatColor.BLUE + "Timer:" + a);
        SG.setTimer(a);
        for (String name : pl.Player.keySet()) {
            Player p = Bukkit.getPlayerExact(name);
            p.setLevel(SG.getTimer());
            if (SG.getTimer() <= 10 && tp) {
                p.sendMessage(SG.getPrefix() + SG.getTimer() + " seconds left!");
            }
        }
        if (SG.getTimer() == 0 && !tp && pl.Player.size() >= 2) {
            int q = 1;
            tp = true;
            SG.setTimer(11);
            List<String> list = A.getArenaConfig().getStringList("list");
            int one = pl.votes.get(list.get(0));
            int two = pl.votes.get(list.get(1));
            int three = pl.votes.get(list.get(2));
            int four = pl.votes.get(list.get(3));
            int five = pl.votes.get(list.get(4));
            int i = Math.max(one, Math.max(two, Math.max(three, Math.max(four, five))));
            for (String name : pl.votes.keySet()) {
                if(pl.votes.get(name) == i) {
                    SG.setArena(name);
                }
            }   
            for (String name : pl.Player.keySet()) {
                Player p = Bukkit.getPlayerExact(name);
                World w = Bukkit.getWorld(A.getArenaConfig().getString("spawn." + SG.getArena() + "." + q + ".world"));
                double x = A.getArenaConfig().getDouble("spawn." + SG.getArena() + "."  + q + ".x");
                double y = A.getArenaConfig().getDouble("spawn." + SG.getArena() + "."  + q + ".y");
                double z = A.getArenaConfig().getDouble("spawn." + SG.getArena() + "."  + q + ".z");
                p.teleport(new Location(w, x, y, z));
                p.sendMessage(SG.getPrefix() + "Arena " + ChatColor.GREEN + SG.getArena() + ChatColor.GOLD + " has won!");
                p.sendMessage(SG.getPrefix() + "Teleporting you to arena...");
                q = q +1;
            }
        }
        else if (pl.Player.size() <= 2 && SG.getTimer() == 0){
            OS.cancel();
        }
        if (SG.getTimer() == 0 && tp && SG.getAmountOfPlayers() >= 2 && SG.getArena() != null) {
            OS.start();
            OS.chests(SG.getArena());
            SG.setCountDown(false);
        }
        if (SG.getArena() == null && SG.getTimer() == 0) {
            for (String name : pl.Player.keySet()) {
                Player p = Bukkit.getPlayerExact(name);
                p.sendMessage(SG.getPrefix() + "Arena does not exist!");
                p.sendMessage(SG.getPrefix() + ChatColor.BOLD + ChatColor.DARK_RED + "Teleporting everybody to Survival Games hub...");
                p.teleport(SG.getHub());
            }
        }
        else if (SG.getTimer() == 0) {
            for (String name : pl.Player.keySet()) {
                Player p = Bukkit.getPlayerExact(name);
                p.sendMessage(SG.getPrefix() + "Something is wrong!");
                p.sendMessage(SG.getPrefix() + ChatColor.BOLD + ChatColor.DARK_RED + "Teleporting everybody to Survival Games hub...");
                pl.Player.clear();
                pl.Spectator.clear();
                p.teleport(SG.getHub());
            }
        }
        if (SG.getCountDown()) {
            Timer();
        }
    }
}

It is this line which returns null:

Code:java

 for (String name : pl.Player.keySet()) {

 

pl is null, because you never initialized it to anything.
On a side note, dear god please name your variables sensible things, it makes reading your code painful.

 

Ok, I am not done at all, I will rename the variables too. But what do you mean by not initialized? I am not good with this stuff, so how would I initialize it? I thought it was initialized with

Code:java

 public TimerR(MainC plugin) {
        pl = plugin;
    }

 

whoops, must have missed that bit. In that case, pretty sure the issue is that you are calling pl.Player, which would be an invalid name. (I dont know what you have Player set as in your main class, but Bukkit already has Player)

 

Oh, I didn't think it would matter. It is a Hashmap with <String, String>, I will change the name to something else.
This really helped me alot, thank you!