player2 is null?

I'm creating a 1v1 plugin with a GUI feature, but there seems to be a NullPointerException when InventoryClickEvent is called.

Code:java

 
final Player player1 = Bukkit.getPlayerExact(Arena.arenaList.get(1).getPlayer1());
            final Player player2 = Bukkit.getPlayerExact(Arena.arenaList.get(1).getPlayer2());
            if(player2 == null) {
           if(player1 == null) {
           Arena.arenaList.get(1).setPlayer(1, p.getName());
                    p.sendMessage(Main.pre + ChatColor.AQUA + "You have been added to the " + ChatColor.DARK_GREEN + "Arena One " + ChatColor.AQUA + "queue!");
                    p.sendMessage(Main.pre + ChatColor.AQUA + "Your 1v1 battle will begin once another player joins.");
           } else {
           Arena.arenaList.get(1).setPlayer(2, p.getName());
                    player2.sendMessage(Main.pre + ChatColor.AQUA + "You have been added to the " + ChatColor.DARK_GREEN + "Arena One " + ChatColor.AQUA + "queue!");
                    player2.sendMessage(Main.pre + ChatColor.AQUA + "Your 1v1 battle against " + ChatColor.DARK_AQUA + player1.getName() + ChatColor.AQUA + " will begin in 5 seconds...");
                    player1.sendMessage(Main.pre + ChatColor.AQUA + "Another player has joined the queue for " + ChatColor.DARK_GREEN + "Arena One!");
                    player1.sendMessage(Main.pre + ChatColor.AQUA + "Your 1v1 battle against " + ChatColor.DARK_AQUA + player2.getName() + ChatColor.AQUA + " will begin in 5 seconds...");
 

The Arena class:

Code:java

 
public class Arena implements Iterable<String> {
 
public static Map<Integer, Arena> arenaList = new HashMap<Integer, Arena>();
private String player1 = "", player2 = "";
@SuppressWarnings("unused")
private int arenaID;
 
public List<Player> getBukkitPlayers() {
return Arrays.asList(Bukkit.getPlayerExact(this.player1), Bukkit.getPlayerExact(this.player2));
}
 
public String getPlayer1() {
return this.player1;
}
 
public String getPlayer2() {
return this.player2;
}
 
public Arena(int id) {
this.arenaID = id;
}
 
public List<String> getPlayers() {
return Arrays.asList(this.player1, this.player2);
}
 
public void setPlayer(int index, String playerName) {
if(index == 1) this.player1 = playerName;
else if(index == 2) this.player2 = playerName;
else return;
}
 
@Override
public Iterator<String> iterator() {
return this.getPlayers().listIterator();
}
 
}
 

 

It would help if you posted us the stacktrace.

 

BungeeTheCookie
NullPointerException. From player2 in the first set of code I listed.

 

There isn't enough information here to tell. From what I can see, you should be getting an ArrayIndexOutOfBoundsException before any NPEs. What's the stack trace?

 

Skye
Let me get that real quick...

EDIT: Here it is:
http://gyazo.com/5be2e088d4fc3d9dcdaf83e6f788a927

 

CandyCranium What's line 241 of ArenaSelector.java?

 

Skye
Line 2 in the code I showed yall.

 

Code:java

final Player player1 = Bukkit.getPlayerExact(Arena.arenaList.get(1).getPlayer1());

1. arenaList is null
2. index 1 of arenaList is null
3. getPlayer1 returned null
4. You somehow messed up the spelling/case of the name