See what i did wrong?

This is my code for tpa/tpaccept and when i send the request it sends properly, but when i try to to /tpaccept it says no requests.

Code:java

 
    HashMap<Player, Player> req = new HashMap<Player, Player>();
 
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        if (!(sender instanceof Player)) {
            return true;
        }
        if (cmd.getName().equalsIgnoreCase("tpa")) {
            Player player = (Player) sender;
            if (args.length == 0) {
                player.sendMessage(ChatColor.RED + "/tpa <player>");
                return true;
            }
            Player target = Bukkit.getServer().getPlayer(args[0]);
            if (target == null) {
                player.sendMessage(ChatColor.RED + "Player not found");
                return true;
            }
            req.put(target, player);
            player.sendMessage(ChatColor.GOLD + "Teleport request sent to " + ChatColor.AQUA + target.getName());
            target.sendMessage(ChatColor.GOLD + "You recived a teleport request from " + ChatColor.AQUA + player.getName());
            target.sendMessage(ChatColor.GOLD + "Use " + ChatColor.AQUA + "/tpaccept" + ChatColor.AQUA + " to accept the teleport request!");
            return true;
        }
        if (cmd.getName().equalsIgnoreCase("tpaccept")) {
            Player target = (Player) sender;
            if (!req.containsKey(target)) {
                target.sendMessage(ChatColor.RED + "You do not have any teleport requests");
                return true;
            }
            Player player = Bukkit.getServer().getPlayerExact(req.get(target).getName());
            player.teleport(target);
            return true;
        }
        return true;
    }

Thanks for the help!

 

MineDoubleSpace

What you are doing here is teleporting the sender to himself. As you can see, Player target = (Player) sender, creates a variable (target) and you make a variable player, which gets the name of the target from your HashMap. You teleport the target from the hashmap to himself in the next thing. I think this is why it seems like nothing is happening.

I know you pretty well, and I know that you are able to fix this yourself .
BTW. You might want to check if the sender is an instanceof player under your if statement for the tpaccept command.

 

MineDoubleSpace
*Ehm* player objects in collections?...

 

xTrollxDudex
Yeah.. done it before,

 

Not bad if garbage is collected.

 

Question is... Is it being GC'd?

Back to the topic: