Solved Not getting the player correctly?

Well, I'm back again, with another problem. This time it's that when a player enters a team's base it doesn't execute. Here's my code so far:

AttackManager:

Code:java

static ArrayList<Player> aManager = new ArrayList<Player>();
 
    public void pick() {
        aManager.clear();
 
        final Player randomRed = pPicker.randomRedPlayer();
        final Player randomBlue = pPicker.randomBluePlayer();
 
        if(OffenseState.isState(OffenseState.BLUE)) {
            OffenseState.setState(OffenseState.RED);
            aManager.add(randomRed);
            Bukkit.getServer().broadcastMessage(ChatColor.GREEN.toString() + ChatColor.BOLD + ">> " + ChatColor.WHITE + randomRed.getDisplayName() + ChatColor.YELLOW + " is now the attack bomber!");
        } else if(OffenseState.isState(OffenseState.RED)) {
            OffenseState.setState(OffenseState.BLUE);
            aManager.add(randomBlue);
            Bukkit.getServer().broadcastMessage(ChatColor.GREEN.toString() + ChatColor.BOLD + ">> " + ChatColor.WHITE + randomBlue.getDisplayName() + ChatColor.YELLOW + " is now the attack bomber!");
        } else {
            List<String> teams = Arrays.asList("RED", "BLUE");
 
            String team = teams.get(r.nextInt(teams.size()));
 
            if(team.equals("RED")) {
                OffenseState.setState(OffenseState.RED);
                aManager.add(randomRed);
                Bukkit.getServer().broadcastMessage(ChatColor.GREEN.toString() + ChatColor.BOLD + ">> " + ChatColor.WHITE + randomRed.getDisplayName() + ChatColor.YELLOW + " is now the attack bomber!");
            } else if(team.equals("BLUE")) {
                OffenseState.setState(OffenseState.BLUE);
                aManager.add(randomBlue);
                Bukkit.getServer().broadcastMessage(ChatColor.GREEN.toString() + ChatColor.BOLD + ">> " + ChatColor.WHITE + randomBlue.getDisplayName() + ChatColor.YELLOW + " is now the attack bomber!");
            }
        }
    }

RegionHandler:

Code:java

@EventHandler
    public void onPlayerMove(PlayerMoveEvent e) {
        Player p = e.getPlayer();
        if(GameState.isState(GameState.INGAME)) {
            if(e.getFrom().getBlockX() != e.getTo().getBlockX() || e.getFrom().getBlockY() != e.getTo().getBlockY() || e.getFrom().getBlockZ() != e.getTo().getBlockZ()) {
                Location pLoc = p.getLocation();
                World pWor = p.getWorld();
 
                double Y = pLoc.getY() -1;
 
                Location block = new Location(pWor, pLoc.getX(), Y, pLoc.getZ());
                Block b = block.getBlock();
 
                if(b.getType() == Material.WOOL) {
                    Wool wool = new Wool(b.getType(), b.getData());
                    if(wool.getColor() == DyeColor.RED) {
                        if(AttackManager.aManager.contains(p)) {
                            Bukkit.broadcastMessage(ChatColor.GREEN.toString() + ChatColor.BOLD + ">> " + ChatColor.YELLOW + "The attack bomber has made it into the base! Boom!");
                            ScoreboardTasks.redLives = ScoreboardTasks.redLives -1;
                            if(ScoreboardTasks.redLives == 0) {
                                gManager.end();
                            } else {
                                gManager.next();
                            }
                        }
                    } else if(wool.getColor() == DyeColor.BLUE) {
                        if(AttackManager.aManager.contains(p)) {
                            Bukkit.broadcastMessage(ChatColor.GREEN.toString() + ChatColor.BOLD + ">> " + ChatColor.YELLOW + "The attack bomber has made it into the base! Boom!");
                            ScoreboardTasks.blueLives = ScoreboardTasks.blueLives -1;
                            if(ScoreboardTasks.blueLives == 0) {
                                gManager.end();
                            } else {
                                gManager.next();
                            }
                        }
                    }
                }
            }
        }
    }

 

WesJD, I know this sounds dumb, but is the event registered?

 

Tecno_Wizard Yup.

 

WesJD

Add debug statements to see what code is and isn't being reached.

 

What is pPicker?

 

mine-care My PlayerPicker class, but I know it works because it broadcasts the message correctly.

mythbusterma Already did, it stops when it trys to check if the player is in the ArrayList.

 

I'd compare the two player names.
Attributes of the player class change over time - like location, and so on.
Unless the ArrayList updates, they will be two different versions of that player.

 

@Darkpicasa Player picker picks a player out of team lists. It shouldn't change.

 

@WesJD

Don't you MAYBE think that would've been useful information to put in your original post?

Make sure that there's players actually in the set you're checking.

 

@mythbusterma It had a player in it, and I should have. My bad. Anyways, this is solved, I figured out that I wasn't setting the GameState to ingame...