AutoBow - PlayerInteractEvent

Bukkit.getScheduler().scheduleSyncRepeatingTask(plugin, new BukkitRunnable() {
int count = 10;
public void run() {
player.launchProjectile(Arrow.class);
count--;
if (count >= 0 || player.getWalkSpeed() >= 0.2) {
cancel();
count = 10;
}
}
}, 30L, 5L);

replace with
new BukkitRunnable(){
int count = 10;
@Override
public void run(){
player.launchProjectile(Arrow.class);
count--;
if (count >= 0 || player.getWalkSpeed() >= 0.2) {
cancel();
count = 10;
return;
}
}
}.runTaskTimer(plugin, 30, 5);

 

ShadowLAX
Inside. Use this.cancel()

 

though shouldn't it also be if count <= 0 not >= ?

 

ShadowLAX
I think, there is no use in canceling the Runnable, the scheduler will start the next after intervall... You have to build an (inner) class extending Runnable, so you can store the id and implement a method to set this id after creating the schedule.
Here is an example, how that can look like: http://forums.bukkit.org/threads/cooldown.162006/#post-1767527

 

Code:java

    @EventHandler
    public void onPlayerInteract(final PlayerInteractEvent event) {
        final Player player = event.getPlayer();
        if (player.getItemInHand().getType() == Material.BOW && player.hasPermission("elitefun.autobow")) {
            int schedule1;
            class AutoBowTask implements Runnable {
                EliteFun plugin;
                int schedule;
               //Number of arrows it shoots
                int count = 20;
 
                AutoBowTask(EliteFun p_plugin) {
                    plugin = p_plugin;
              }
                @Override
                public void run(){
                    player.launchProjectile(Arrow.class);
                    count--;
                    if(count <= 0) {
                        plugin.getServer().getScheduler().cancelTask(schedule);
                    }
                }
 
                public void setSchedule(int p_schedule) {
                    schedule = p_schedule;
                }
            }
            AutoBowTask task = new AutoBowTask(plugin);
            schedule1 = plugin.getServer().getScheduler().scheduleSyncRepeatingTask(plugin, task, 1, 1);
            task.setSchedule(schedule1);
        }
    }

Enjoy,