Solved BukkitRunnable() I dont get this working ...

Code:java

    public void breakBlockList(final List<Block> BlockList, final Player player, final ItemStack tool, long speed) {
 
        Bukkit.getScheduler().runTaskTimer(plugin, new BukkitRunnable() {
            int counter = 0;
            @Override
            public void run() {
                logger.log("" + counter, 0);
 
                events.get(counter).breakNaturally(tool);
 
                counter++;
 
                if (events.size() <= counter) {
                    this.cancel;
                    Bukkit.getScheduler().cancelTask(getTaskId());
                    return;
                }
            }
        }, speed, speed);
    }

2 Problems ...
the task needs to cancel itself! due to multiple of these tasks are running, i cannot define a id for everyone and neither "this.getTaskId()" nor "this.cancel()" is working ... always telling me "Task not sheduled yet." dunno how to fix this ...

2nd Problem .... the counter it is always 0 and wont count up ... and i cannot define a global var due to the same problem as above ...

someone can tell me how to get the "this.getTaskId()" and "this.cancel()" working?

 

1. BukkitRunnable can schedule itself independently. For example:

Code:

new BukkitRunnable()
{
      public void run()
      {
          Bukkit.getServer().broadcastMessage("Test");
      }
}.runTask(yourplugin);

2. Your runnable never gets scheduled, so its run method never gets called.

 

First, is that your actual code ? Because: this.cancel -> this.cancel()

The cancel() method does work.

I never used getScheduler() with BukkitRunnable so I don't know if that's valid but this works and it's also cleaner (and shorter):

Code:

new BukkitRunnable()
{
    public void run()
    {
         // ...
    }
}.runTaskTimer(plugin, 0, ticks)

Have you actually printed any values or are you guessing that they're what you expect ?

EDIT:
Yeah apparently using BukkitRunnable with getScheduler() strips it from its ability of using cancel() inside its method... you forgot to mention that you have console errors

 

Technius Digi
oh yeah ... if u schedule the Runnable with the sheduler it will not work like it should ... but if i do it your way it does
ty ...

I will test the counter next ... maybe it will work this way ...