Jump Pads not working.

Im trying to add a feature to my plugin that will make pressure plates work as jump pads. I don't get why this code isn't working. I think it should be. Any idea why its not?

Code:java

    public void onPlayerStep(PlayerMoveEvent e) {
        Player p = e.getPlayer();
        Block standingOn = p.getWorld().getBlockAt(0, -1, 0);
        if(standingOn.getType().equals(Material.STONE_PLATE)) {
            e.getPlayer().sendMessage(ChatColor.GREEN + "Woosh!");
            Player player = e.getPlayer();
            Vector dir = player.getLocation().getDirection();
            Vector vec = new Vector(dir.getX() * 0.8D, 0.8D, dir.getZ() * 0.8D);
            player.setVelocity(vec);
            player.setFallDistance(-100.0F);
        }
    }

 

Here's one that works
try this:

Code:java

@SuppressWarnings("deprecation")
    @EventHandler
    public void onLaunchMove(PlayerMoveEvent event) {
        Player player = event.getPlayer();
        Location playerLoc = player.getLocation();
        int ID = playerLoc.getWorld().getBlockAt(playerLoc).getRelative(0, -1, 0).getTypeId();
        int plate = playerLoc.getWorld().getBlockAt(playerLoc).getTypeId();
        if (((player instanceof Player) && player.hasPermission("perm.perm"))) {
            if (ID == 133) {
                if (plate == 70) {
                    player.setVelocity(player.getLocation().getDirection().multiply(3));
                      player.setVelocity(new Vector(player.getVelocity().getX(), 1.0D, player.getVelocity().getZ()));
                      player.playSound(player.getLocation(), Sound.AMBIENCE_THUNDER, 1, 1);
                }
            }
        }
    }

 

Show the whole class.
Do you get any errors? Remember to register the events.

 

ksbdude The problem with your code is that you are getting the block under the player, but don't forget that the pressure plate is not part of that block; it occupies the block you are standing in. So instead of getting the block under the player, get the block at the player's location.

 

I made a plugin called PlayerLaunchPads
http://dev.bukkit.org/bukkit-plugins/playerboostpads/
If you have any questions about it , add me on skype: ShearsSheep

 

ksbdude
You have no idea how many times PlayerMoveEvent is fired, but let me tell you: An INSANE amount of times. use PlayerInteractEvent instead, at least it doesnt fire every other tick (pretty much) like PlayerMoveEvent.

You can use the same implementation, but instead of getting the block by location, use PlayerInteractEvent#getClickedBlock(). Make sure to check that the Action is Action.PHYSICAL first though. Then you can launch the player.

Note to self: The description to Action.PHYSICAL is surprisingly hilarious:

 

@xTrollxDudex and @ShearsSheep
Im still getting some problems
This does not work

Code:java

 public void onLaunchMove(PlayerMoveEvent event) {
Player player = event.getPlayer();
Location playerLoc = player.getLocation();
int ID = playerLoc.getWorld().getBlockAt(playerLoc).getRelative(0, -1, 0).getTypeId();
int plate = playerLoc.getWorld().getBlockAt(playerLoc).getTypeId();
if (ID == 133) {
if (plate == 70) {
player.setVelocity(player.getLocation().getDirection().multiply(3));
player.setVelocity(new Vector(player.getVelocity().getX(), 1.0D, player.getVelocity().getZ()));
player.playSound(player.getLocation(), Sound.AMBIENCE_THUNDER, 1, 1);
}
}
}

I then tried to use something like this as Troll said which also isnt working

Code:java

 @EventHandler
public static final Action PHYSICAL(){
return null;
 
}

 

ksb, it was already explained not to get the block below the player, but the block the player is in.

There is also no need to get the id, since they are going to be removed soon.
Get the Material.

 

Okay here is my code for the jumpPads!

Code:java

public class Main extends JavaPlugin implements Listener {
 
        private ArrayList<Player> jumpers = new ArrayList<Player>();
 
        public void onEnable() {
                Bukkit.getServer().getPluginManager().registerEvents(this, this);
        }
 
        @EventHandler
        public void onPlayerMove(PlayerMoveEvent e) {
                if (e.getTo().getBlock().getRelative(BlockFace.DOWN).getType() == Material.STONE_PLATE) {
                        e.getPlayer().setVelocity(e.getPlayer().getLocation().getDirection().multiply(4));
                        e.getPlayer().setVelocity(new Vector(e.getPlayer().getVelocity().getX(), 1.0D, e.getPlayer().getVelocity().getZ()));
                        jumpers.add(e.getPlayer());
                }
        }
 

Mhm...I will not spoon feed you , if you really need help msg me on skype.

 

ShearsSheep

Thank goodness you're not spoon-feeding, because there's two things wrong with that code that I can see off the bat.

1) Why are you using PlayerMoveEvent? That's called over 40 times with one little bit of movement. (From Bukkit code comment) Just use PlayerInteract and Action.PHYSICAL

2) You're storing Player objects, which, if you do it right, is fine. But you've made an arraylist and are repeatedly storing people, possibly more than once every time they jump, and you never clear it

If someone was in need of 'spoon-feeding', you assume that they do not understand code very well. Your code, in the hands of someone new, could crash a server over time given enough players

 

PlayerMoveEvent shouldn't be called more than 20 times per second(unless there's lag or the player is using timer hacks), but whatever.

 

Garris0n
idk. I remember going through the CraftBukkit section of Bukkit to check out NMS, and that's what I read in a comment.

 

Quote from xTrollxDudex

@ksbdude
You have no idea how many times PlayerMoveEvent is fired, but let me tell you: An INSANE amount of times. use PlayerInteractEvent instead, at least it doesnt fire every other tick (pretty much) like PlayerMoveEvent.

You can use the same implementation, but instead of getting the block by location, usePlayerInteractEvent#getClickedBlock(). Make sure to check that the Action is Action.PHYSICAL first though. Then you can launch the player.

//end quote:

Something like this

Code:java

@EventHandler
public void onPlayerPush(PlayerInteractEvent event) {
 
if (event.getAction() == Action.PHYSICAL) {
 
           //get location and if there is a pressureplate ON THAT LOCATION, dont look beneath it, because player will be "In the pressureplate"
          //If so, then launch them
          }
}