Cancelling bukkitrunnable does not cancel nested bukkitrunnable

Hello!
I have a problem with trying to cancel a bukkitrunnable, which I store in a hashmap, as I have another bucketrunnable that runs inside it.

Code:java

public static HashMap<String, BukkitTask> taskIDMap = new HashMap<String, BukkitTask>();
        public void countdown(final Player sender, final Player target, int playerDuration) {
        taskIDMap.put(sender.getName(), new BukkitRunnable() {
 
            @Override
            public void run() {
                new BukkitRunnable() {
                    int timer = 10;
 
                    @Override
                    public void run() {
 
                        if (timer < 1) {
                            swapPlayers(sender, target);
                            timer = 10;
                            this.cancel();
                        } else {
 
                            sender.sendMessage("Time remaining: " + timer);
                            target.sendMessage("Time remaining: " + timer);
                            --timer;
                        }
 
                    }
                }.runTaskTimer(plugin, 0L, 20L);
 
            }
 
        }.runTaskTimer(plugin, playerDuration, playerDuration));
    }
 

I cancel the runnable in, among other places, using something like this:

Code:java

if (taskIDMap.get(sender.getName()) != null) {
                    taskIDMap.get(sender.getName()).cancel();
                    taskIDMap.remove(sender.getName());
                }

The only solution I could come up with is storing the nested bukkitrunnable separately, cancelling it whenever I cancel the outer one (as long as it isn't null) and cleaning it up. This would, however, require a lot of cluttering of my code, by adding something like

Code:java

if (taskIDMap2.get(sender.getName()) != null) {
                    taskIDMap2.get(sender.getName()).cancel();
                    taskIDMap2.remove(sender.getName());
                }

in if-cases and events.

Does anyone know of a smoother way to do this? Thanks

 

Could you not have something like this...?

Code:java

new BukkitRunnable() {
    BukkitRunnable subRunnable;
 
    @Override
    public void run() {
        subRunnable = new BukkitRunnable() {
            int timer = 10;
 
            @Override
            public void run() {
                ...Your Code...
            }
        }.runTaskTimer(plugin, 0L, 20L);
    }
 
    @Override
    public void cancel(){
        subRunnable.cancel();
        super.cancel();
    }
}.runTaskTimer(plugin, delay, period);

Not sure how advisable it is though

 

Thank you for the reply!
I tried implementing exactly this, with no other changes except:

Code:java

BukkitTask subRunnable;

instead of BukkitRunnable due to type mismatch, but the inner runnable still doesn't seem to cancel while counting down for some reason.
The cancel() seems to work exactly the same as before.