Multiple Actions?

Hey so im making a rpg plugin and i've made a spell which enables if the player right clocks on a block but i want him to enable it if for example he right clicks on a block and then left click on a block. Heres my code, maybe someone has a nice idea

Code:java

@EventHandler
    public void RageStrike(PlayerInteractEvent event){
        Player player = event.getPlayer();
        if(isWarrior.containsKey(player)){
        if(player.getItemInHand().getType() == Material.DIAMOND_SWORD || player.getItemInHand().getType() == Material.IRON_SWORD || player.getItemInHand().getType() == Material.STONE_SWORD || player.getItemInHand().getType() == Material.WOOD_SWORD) {
            if(player.getLevel() <= 9){ //lvl 1
                if(RageStrike1.containsKey(player)){
                if(event.getAction() == Action.RIGHT_CLICK_BLOCK) {
                    if(event.getAction() == Action.LEFT_CLICK_BLOCK){
                    long last = LastUsed.containsKey(player.getName()) ? LastUsed.get(player.getName()) : 0;
                    long ctm = System.currentTimeMillis();
                    long timeleft = RageStrikeCooldown + last - ctm / 1000;
                    if (ctm - last < RageStrikeCooldown) {
                      event.getPlayer().sendMessage(ChatColor.DARK_RED + "You still have " + timeleft + " seconds cooldown for Rage Strike.");
                      event.setCancelled(true);
                    } else {
                      LastUsed.put(player.getName(), now);
                      Location Location = player.getLocation();
                      player.playEffect(Location, Effect.SMOKE, 10);
                    for (Entity entity : player.getNearbyEntities(3, 3, 3))
                    {
                        if (entity instanceof LivingEntity)
                        {
                            ((LivingEntity) entity).damage(4, player);
                        }
                    }
                }
                }

 

Lines 8 and 9: you're checking if it's Action.RIGHT_CLICK_BLOCK, then immediately after if it's Action.LEFT_CLICK_BLOCK - they can't both be true

 

Chinwe so how will i be able to make a countdown to check if the player has left clicked in that amount of time? (im a nooby developer :$ ) i tried this but it doesnt work:

Code:java

@EventHandler
    public void RageStrike(PlayerInteractEvent event){
        Player player = event.getPlayer();
        if(isWarrior.containsKey(player)){
        if(player.getItemInHand().getType() == Material.DIAMOND_SWORD || player.getItemInHand().getType() == Material.IRON_SWORD || player.getItemInHand().getType() == Material.STONE_SWORD || player.getItemInHand().getType() == Material.WOOD_SWORD) {
            if(player.getLevel() <= 9){ //lvl 1
                if(RageStrike1.containsKey(player)){
                if(event.getAction() == Action.RIGHT_CLICK_BLOCK) {
                    long l = lastUsage.containsKey(player.getName()) ? lastUsage.get(player.getName()) : 0;
    long n = System.currentTimeMillis();
    if(n - l < ActionTime){
                    if(event.getAction() == Action.LEFT_CLICK_BLOCK){
                    long last = LastUsed.containsKey(player.getName()) ? LastUsed.get(player.getName()) : 0;
                    long ctm = System.currentTimeMillis();
                    long timeleft = RageStrikeCooldown + last - ctm / 1000;
                    if (ctm - last < RageStrikeCooldown) {
                      event.getPlayer().sendMessage(ChatColor.DARK_RED + "You still have " + timeleft + " seconds cooldown for Rage Strike.");
                      event.setCancelled(true);
                    } else {
                      LastUsed.put(player.getName(), now);
                      Location Location = player.getLocation();
                      player.playEffect(Location, Effect.SMOKE, 10);
                    for (Entity entity : player.getNearbyEntities(3, 3, 3))
                    {
                        if (entity instanceof LivingEntity)
                        {
                            ((LivingEntity) entity).damage(4, player);
                        }
                    }
                }
                }

 

spiroulis You want the player to right click on a block and then left click, so the spell happens?
I would recommend you to create a boolean and then set the it to true when right clicking a block and if left clicking a block check if the boolean is true.
Somewhere before the Event Handler do

Code:java

boolean blockEnabled = false;

and then change the code to something like:

Code:java

        @EventHandler
        public void RageStrike(PlayerInteractEvent event){
            Player player = event.getPlayer();
            if(isWarrior.containsKey(player)){
                if(player.getItemInHand().getType() == Material.DIAMOND_SWORD || player.getItemInHand().getType() == Material.IRON_SWORD || player.getItemInHand().getType() == Material.STONE_SWORD || player.getItemInHand().getType() == Material.WOOD_SWORD) {
                    if(player.getLevel() <= 9){ //lvl 1
                        if(RageStrike1.containsKey(player)){
                            if(event.getAction() == Action.RIGHT_CLICK_BLOCK) {
                                blockEnabled = true;
                            }
                            if(event.getAction() == Action.LEFT_CLICK_BLOCK && blockEnabled == true){
                                long last = LastUsed.containsKey(player.getName()) ? LastUsed.get(player.getName()) : 0;
                                long ctm = System.currentTimeMillis();
                                long timeleft = RageStrikeCooldown + last - ctm / 1000;
                                if (ctm - last < RageStrikeCooldown) {
                                    event.getPlayer().sendMessage(ChatColor.DARK_RED + "You still have " + timeleft + " seconds cooldown for Rage Strike.");
                                    event.setCancelled(true);
                                } else {
                                    LastUsed.put(player.getName(), now);
                                    Location Location = player.getLocation();
                                    player.playEffect(Location, Effect.SMOKE, 10);
                                    for (Entity entity : player.getNearbyEntities(3, 3, 3)) {
                                        if (entity instanceof LivingEntity){
                                            ((LivingEntity) entity).damage(4, player);
                                        }
                                    }
                                }
                                blockEnabled = false;
                            }
                        }
                    }
                }
            }
        }

This one should hopefully work, if not, just tahg me and I will try to help!

 

Lionhard Thanks for the help it works fine now but i still have one question. What if i wanted to make the player so when he right clicks if he doesnt left click in a specific amount of time the spell would be cancelled?

 

spiroulis Hmm, I have no experience with this, I just got an Idea how you could get this to work. In the main class,
in the onEnable() Method do following:

Code:java

// This will run the MyMethod() each amount of ticks you have specified.
this.getServer().getScheduler().scheduleSyncRepeatingTask(this, new Runnable() {
public void run() {
myMethod();
}
}, 0, 1200); // Replace 1200 with your time. (In Ticks, 20 ticks = 1 second.)
//This will run the myMethod() each amount of ticks you have specified.

If the third parameter is 0, when the server starts it runs the method without looking at the 'timer'. After that, the method will run each 1200 ticks. If you want it to start 1200 ticks after the plugin enables, replace 0 with 1200.
Hope you understood that, I'm not that good at explaining things.

and then create the MyMethod() somewhere in the code like:

Code:java

public void myMethod(){
if(blockEnabled){
blockEnabled = false;
}
}

This should work, although if you right click a block 1 tick before the method will run, it will be set again to false so you would have to right click again and then left click.

 

Lionhard Thank you so much, i changed the code a bit and it works as i want now (even though there are some tiny bugs) here is what i did just if you are interested to see xd

Code:java

public void RageStrike(PlayerInteractEvent event){
        final Player player = event.getPlayer();
        if(isWarrior.containsKey(player)){
        if(player.getItemInHand().getType() == Material.DIAMOND_SWORD || player.getItemInHand().getType() == Material.IRON_SWORD || player.getItemInHand().getType() == Material.STONE_SWORD || player.getItemInHand().getType() == Material.WOOD_SWORD) {
            if(player.getLevel() <= 9){ //lvl 1
                if(RageStrike1.containsKey(player)){
                if(event.getAction() == Action.RIGHT_CLICK_BLOCK) {
                blockEnabled = true;
                RightClicked = true;
                this.getServer().getScheduler().scheduleAsyncRepeatingTask(this, new Runnable() {
                    public void run() {
                    if(RightClicked == true){
                        player.sendMessage("test");
                        RightClicked = false;
                    }
                    }
                    }, 100L, 100L); //tick time 20 = 1 second
                player.playSound(player.getLocation(), Sound.CLICK, 1, 1);
                }
                    if(event.getAction() == Action.LEFT_CLICK_BLOCK && blockEnabled == true && RightClicked == true){
                    RightClicked = false;
 

and then the rest of the code

 

spiroulis Glad that I helped you. Although, why do you have 2 variables blockEnabled and RightClicked? Isn't 1 enough?