Odd problem w/ array lists?

This is supposed to stop a player from tping whenever they move:

The event that checks if their moving:

Code:java

@EventHandler
public void onPMove(PlayerMoveEvent e) {
Player p = e.getPlayer();
if (e.getFrom().getBlockX() == e.getTo().getBlockX() && e.getFrom().getBlockY() == e.getTo().getBlockY() && e.getFrom().getBlockZ() == e.getTo().getBlockZ()) return;
d.remove(p.getName());
}
}

The teleport event that is supposed to happen only if they are in the array list.

Code:java

new BukkitRunnable() {
@Override
public void run() {
if(d.contains(p.getName())){
p.teleport(new Location(w, x, y, z));
}
}
}.runTaskLater(plugin, 60);

The problem is, even when I move, I still get tped. Does anyone know why this might be happening?

Well, I'm off to bed, if anyone could figure this out that would be great.

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

 

You have to cancel the task, as it already started.

 

No, that is only required if he runs the task timer.

You are activating the task multiple times,which is odd.
Also, why are you listening for the player to be teleported? If you execute Entity.teleport(<>), It is safe to assume the teleport was successful.

EDIT: By the way, use a Set, not a List. People seem to not know the difference.

 

Instead of listening for them to move to cancel the event, why not just do this:

* player initiates teleport, store their location, andcountdown begins
* When countdown ends, check current locations against the stored location. If same, teleport. If different, don't teleport.

 

Only downside is if the cooldown is long, players won't get notified until it is about to teleport them.

 

what happends if you print the values from the hashmap or arraylist?
I'm very sure it is empty, a common mistake is when you use a ArrayList or HashMap inside a other class without making it static or using getters in the class the ArrayList object is.
this means if you would instance it like in your event class with 'new' you create also a new ArrayList instance which is empty.

also why are you using a ArrayList?
an ArrayList can have duplicates thats very dangerous for memory leaks but if you use a HashMap or a HashSet, you can't have duplicates its alot safer.