Only the first time you step on the block!

I created a PlayerMoveEvent for when the player stepping on the x, y, z defined happen one delay and teleports him. But how is a PlayerMoveEvent, every time he moves the same thing happens several times. Wanted only the first time he set foot on the block, this delay happens. I hope you can help me!

Thanks.

 

Code:java

public Map<String, Boolean> hasStepped = new HashMap<String, Boolean>();

Now when he steps on it:

Code:java

Player p = event.getPlayer();
if (!hasStepped.containskey(p.getName())) hasStepped.put(p.getName(), false);
if (hasStepped.get(p.getName()) == false){
//Do everything like normal
hasStepped.put(p.getName(), true);
else //Do stuff for when its NOT the first time.

 

zDubsCrazy You would have to keep track of the fact that he'd stepped on a block. Save an arraylist of players who stepped on it, and ignore them if they are already in the list.

 

zDubsCrazy
if (e.getTo().getX() != e.getFrom().getX() || e.getTo().getZ() != e.getFrom().getZ()) // moved a block.

 

A map with a boolean as a value can be simply changed to a Set. If the player is in the Set, they have stepped on the block, if they aren't, they haven't. It's simpler, more concise, and saves memory.

 

WeeziMonkey
Thanks . But all it in PlayerMoveEvent?

 

Everything except for the part defining the HashMap (or HashSet as the person above you said).

 

TryB4's example is the most efficient for this scenario (although it's incomplete). No need to store data or sift through lists of crap.

They key here is that the player is being teleported away when they step into this block. This doesnt need to fire 100 times from the player moving as it processes the teleport. Once the player is teleported away they are again no longer in the block, and no need to reset/modify any lists or sets for the player to become eligible for the teleport again.

Code:java

 
public void onPlayerMove(PlayerMoveEvent event) {
    if (event.getTo().getX() == event.getFrom().getX() &&
            event.getTo().getY() == event.getFrom().getY() &&
            event.getTo().getZ() == event.getFrom().getZ()) {
        return; // Stop. Player did not move between blocks (no need to check the location)
    }
    // The rest of this is just to finish the example of a hardcoded teleport location. Your existing code will go here instead.
    int x = event.getTo().getBlockX();
    int y = event.getTo().getBlockY();
    int z = event.getTo().getBlockZ();
    if (x == 10 && y == 64 && z == 10)
        event.getPlayer().teleport(new Location(someWorld, 20, 64, 10));
}
 

 

BillyGalbreath
Thanks.
But it will work with the delay? This delay is 3 seconds before the teleporter!