player.setExp(0.50F); working in one plugin but not another?

Hey guys Im working on a plugin that requires the Exp to go from full down to half then zero. I already have all the code but for some reason player.setExp(0.50F); isn't working. I tested it in another plugin and it works fine. Could someone help me figure out why its not working. Anyways here is my code

Code:java

 @EventHandler
public void OnPlayerRocket(PlayerInteractEvent event){
final Player player = event.getPlayer();
if (!plugin.RocketCooldown.contains(player.getName())){
if (!plugin.RocketCooldown2.contains(player.getName())){
if((event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() ==Action.RIGHT_CLICK_BLOCK) && player.getItemInHand().getType() == Material.FIREWORK){
player.setVelocity(player.getLocation().getDirection()
.multiply(1.05).setY(1.15));
player.setExp(0.50F);
plugin.RocketCooldown2.add(player.getName());
}
}
if (plugin.RocketCooldown2.contains(player.getName())){
if((event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() ==Action.RIGHT_CLICK_BLOCK) && player.getItemInHand().getType() == Material.FIREWORK){
player.setVelocity(player.getLocation().getDirection()
.multiply(1.05).setY(1.15));
plugin.RocketCooldown.add(player.getName());
player.setExp(0);
 
}
 
}
}
}
@EventHandler
public void OnGround(PlayerMoveEvent e){
Player p = e.getPlayer();
if(p.getLocation().getBlock().getRelative(BlockFace.DOWN).getType() != Material.AIR) {
plugin.RocketCooldown.remove(p.getName());
plugin.RocketCooldown2.remove(p.getName());
p.setExp(1);
}
}
 

Or Here http://pastebin.com/hCuBXyLd

 

Everything else is working and it's just the setExp?

 

22vortex22
Any stacktrace?

 

Is the other code working? If not, are your events registered?

 

All my code is working except the player.setExp(0.50F) And yes my events are registered

Bump please guys I would love some answers?

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

 

Is it possible that anything else is setting their exp afterwards?

 

22vortex22 Only bump every 24 hours

 

I don't think so. You can check the code.

I need to solve this D:

 

Sounds like you have 2 plugins running a setExp , which would cause errors. Remove all plugins except for that one and try it.

 

Nope. My plugin is by itself and it still doesn't go to the halfway mark. It goes to 1 and 0 perfectly but never 0.50

 

22vortex22
I found it:
You're checking if the player is contained in a second list after you add him to it, so the code is acting on both at the same time. You need to remove the player from all other lists before setting his Exp from .50f to 0, otherwise its going to act on both. You understand?

 

Im checking if hes not in the list. Thats why I have a ! infront of it.
And no I don't really understand :/

 

Code:

        player.setExp(0.50F);
        plugin.RocketCooldown2.add(player.getName());
        if (plugin.RocketCooldown2.contains(player.getName())){
            if((event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() ==Action.RIGHT_CLICK_BLOCK) && player.getItemInHand().getType() == Material.FIREWORK){
                player.setVelocity(player.getLocation().getDirection()
                        .multiply(1.05).setY(1.15));
                plugin.RocketCooldown.add(player.getName());
                player.setExp(0);

Look at the code: Your adding them to a list after you set their exp, all good, but then you check if that SAME list contains the player, and if it does, your setting his exp to 0, ergo why you are getting it to go from 1-0 and skipping the half.

 

How do I fix it? calebbfmv

 

22vortex22
Simple.
Check is exp and set it to 0 if its at .5f

 

I must be really stupid calebbfmv but how do I check the xp and turn it into an if statement?

 

22vortex22 if(player.getExp() == 0) {

 

sgavster
Just had to get in on this didn't you?

 

It works but its giving me 3 jumps now, I only want two

sgavster calebbfmv

Code:java

    @EventHandler
    public void OnPlayerRocket(PlayerInteractEvent event){
        Player player = event.getPlayer();
        if (player.getExp() == 1){
            if((event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() ==Action.RIGHT_CLICK_BLOCK) && player.getItemInHand().getType() == Material.FIREWORK){
                player.setVelocity(player.getLocation().getDirection()
                        .multiply(1.05).setY(1.15));
                player.setExp(0.50F);
            }
        }
        else if(player.getExp() == 0.50F) {
                if((event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() ==Action.RIGHT_CLICK_BLOCK) && player.getItemInHand().getType() == Material.FIREWORK){
                    player.setVelocity(player.getLocation().getDirection()
                            .multiply(1.05).setY(1.15));
                    player.setExp(0);
                }
        }
 
    }
    @EventHandler
    public void OnGround(PlayerMoveEvent e){
        Player p = e.getPlayer();   
        if(p.getLocation().getBlock().getRelative(BlockFace.DOWN).getType() != Material.AIR) {
            p.setExp(1);   
        }
    }
 

 

This is where some lists come in handy:

Code:

//where you set exp to 0.5f:
player.setExp(0.5f);
plugin.RocketCooldownList2.add(player.getName());
//where you set his exp to 0:
if(plugin.RocketCooldownList2.cotains(player.getName()) && player.getExp() == 0.5f){

 

Guys it still gives me 3 jumps. I think this is because before I fully get off the ground it resets it again. However I am uploading a video to let you guys better understand. Here is the updated code if you need it

Code:java

@EventHandler
    public void OnPlayerRocket(PlayerInteractEvent event){
        Player player = event.getPlayer();
        if(player.getExp() == 1f){
            if((event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() ==Action.RIGHT_CLICK_BLOCK) && player.getItemInHand().getType() == Material.FIREWORK){
                player.setVelocity(player.getLocation().getDirection()
                        .multiply(1.05).setY(1.15));
                player.setExp(0.50F);
                plugin.RocketCooldown2.add(player.getName());
            }
        }
        else if(plugin.RocketCooldown2.contains(player.getName()) && player.getExp() == 0.5f){
                if((event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() ==Action.RIGHT_CLICK_BLOCK) && player.getItemInHand().getType() == Material.FIREWORK){
                    player.setVelocity(player.getLocation().getDirection()
                            .multiply(1.05).setY(1.15));
                    player.setExp(0);
                }
        }
 
    }
    int time;
    @EventHandler
    public void OnGround(PlayerMoveEvent e){
        final Player p = e.getPlayer();
        Bukkit.getServer().getScheduler().runTaskLater(plugin, new Runnable()
        {
            @Override
            public void run(){
                if(p.getLocation().add(0,-1,0).getBlock().getType() != Material.AIR){           
                    p.setExp(1);
                }
            }
        }
                ,19L);
 
 
    }
 

Bump

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