TPA system - not working properly

I'm trying to develop a TPA system similar to the essentials /tpa for a plugin that I'm building. The only problem is (and I tested this because I thought this would happen); Lets say I /tpa to person A, and person B /tpa's to me, when person A does /tpaccept, person B teleports to person A, instead of me being teleported to person A.
Is there a fix for this kind of thing?

Code for /tpa:

Code:java

public class CommandTpa implements CommandExecutor{
 
public static boolean enabled = false;
public static Player tpaSender;
 
 
private BDMain plugin;
 
public CommandTpa(BDMain plugin) {
this.plugin = plugin;
}
 
@Override
public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
// TODO Auto-generated method stub
 
if (sender instanceof Player){
 
Player player = (Player) sender;
Player target = (Bukkit.getServer().getPlayer(args[0]));
tpaSender = player;
if(cmd.getName().equalsIgnoreCase("tpa")){
player.sendMessage(ChatColor.GREEN + "Your request has been sent!");
target.sendMessage(ChatColor.YELLOW + "A player has requested to teleport to you: " + player);
target.sendMessage(ChatColor.YELLOW + "Type /tpaccept to accept teleport request");
}
}
 
return false;
}
}

Code for /tpaccept:

Code:java

public class CommandTpaccept implements CommandExecutor {
 
public static boolean enabled = false;
 
 
private BDMain plugin;
 
public CommandTpaccept(BDMain plugin) {
this.plugin = plugin;
}
 
@Override
public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
// TODO Auto-generated method stub
 
if (sender instanceof Player){
 
Player player = (Player) sender;
Player requester = CommandTpa.tpaSender;
if(cmd.getName().equalsIgnoreCase("tpaccept")){
requester.teleport(player);
player.sendMessage(ChatColor.GREEN + "Request accepted!");
requester.sendMessage(ChatColor.GREEN + "Teleport request accepted!");
Bukkit.broadcastMessage(ChatColor.BLUE + "Player " + requester + " has teleported to " + player + "!");
}
}
return false;
}
}
 

I actually just figured it out. For the reference of others, you'd need to use a Java Hashmap. Here's what I did and how I used it:

CommandTpa:

Code:java

package me.BattleDome.main;
 
public class CommandTpa implements CommandExecutor{
 
public static boolean enabled = false;
public static Player tpaSender;
public static Map<Player, Player> tpamap = new HashMap<Player, Player>();
 
 
    private BDMain plugin;
 
    public CommandTpa(BDMain plugin) {
        this.plugin = plugin;
    }
 
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        // TODO Auto-generated method stub
 
        if (sender instanceof Player){
 
            Player player = (Player) sender;
            Player target = (Bukkit.getServer().getPlayer(args[0]));
            tpaSender = player;
            if(cmd.getName().equalsIgnoreCase("tpa")){
                tpamap.put(target, player);
                String playername = player.getName();
                String targetname = target.getName();
                  player.sendMessage(ChatColor.GOLD + "Your request has been sent!");
                  target.sendMessage(ChatColor.GOLD + "A player has requested to teleport to you: " + ChatColor.RED + playername);
                  target.sendMessage(ChatColor.GOLD + "Type /tpaccept to accept teleport request");
            }
        }
 
        return false;
    }
}
 

CommandTpaccept:

Code:java

public class CommandTpaccept implements CommandExecutor {
 
public static boolean enabled = false;
 
 
    private BDMain plugin;
 
    public CommandTpaccept(BDMain plugin) {
        this.plugin = plugin;
    }
 
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        // TODO Auto-generated method stub
 
        if (sender instanceof Player){
 
            Player player = (Player) sender;
            Player requester = CommandTpa.tpamap.get(player);
            if(cmd.getName().equalsIgnoreCase("tpaccept")){
                requester.teleport(player);
                CommandTpa.tpamap.remove(player);
                String playername = player.getName();
                String requestername = requester.getName();
                player.sendMessage(ChatColor.GREEN + "Request accepted!");
                requester.sendMessage(ChatColor.GREEN + "Teleport request accepted!");
                Bukkit.broadcastMessage(ChatColor.BLUE + "Player " + requestername + " has teleported to " + playername + "!");
            }
        }
        return false;
    }
}

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

@MRGhorm1379
I know this is a tad old, and its probably wont be answered.
But what if multiple people send the same player a request, wouldnt this code teleport you to all of them.
Or do hashmaps only store one of each key?
so lets say you have

Code:java

hashmap.put(1,"hello");
hashmap.put(1,"bye");
hashmap.put(1,"test");

Would "1" be overwrited every time and the final value be test?

 

In short, yes.

Code:java

Map<int, String> hashmap = new HashMap<Key, Value>;

In hashmaps, the 'Key' is always unique. So when you say:

Code:java

hashmap.put(1, "foo");
hashmap.put(1, "bar");

You're writing 'foo' to key '1'.
Then, you're overwriting the key '1', afterwords, with 'bar'.

 

FerusGrim thanks for that, just clarifying.