在Runnable中使用计数器遇编译错误：局部变量需为final或有效final

嘿，这个问题我太熟了！咱们先搞懂为啥会报错，再一步步解决它～

首先，Java的匿名内部类（你这里的BukkitRunnable就是典型例子）在访问外部方法的局部变量时，要求变量必须是final或者有效final的。所谓“有效final”，就是变量在初始化后再也没有被修改过。你的counter在run()方法里做了counter++操作，这就打破了“有效final”的要求，所以编译器直接抛出了这个错误。

为啥Java要搞这个限制？其实是因为匿名内部类会在创建时把外部变量的副本保存一份，如果外部变量能被随意修改，内部类里的副本和外部的真实值就会出现不一致，很容易导致逻辑混乱，所以Java干脆从编译层面就禁止了这种情况。

这里有几种靠谱的解决办法，你可以根据自己的代码需求选：

方法1：把counter改成Bar类的成员变量 #

既然局部变量不行，那把它提升为内部类Bar的成员变量就好了，这样内部类可以直接访问，而且不受final限制，这也是最直观的方案：

public class Foo { 
    private class Bar { 
        // 把counter改成Bar的成员变量
        private int counter = 0;
        
        public void barMethod() { 
            Bar instance = this; 
            BukkitTask barTask = new BukkitRunnable() { 
                @Override 
                public void run() { 
                    if (counter == 5) { 
                        this.cancel(); 
                        return; 
                    } 
                    instance.doStuff(); 
                    counter++; 
                } 
            }.runTaskTimer(this.plugin, 0L, 2L); 
        } 
    } 
}

方法2：用可变容器包装counter #

如果你不想改动类的成员结构，也可以用一个“可变的盒子”来装counter——盒子本身是有效final的（你不会重新赋值这个盒子），但盒子里的内容可以随意修改：

用数组包装（简单直接）： #

public class Foo { 
    private class Bar { 
        public void barMethod() { 
            // 用数组包装counter，数组本身是final的
            final int[] counter = {0}; 
            Bar instance = this; 
            BukkitTask barTask = new BukkitRunnable() { 
                @Override 
                public void run() { 
                    if (counter[0] == 5) { 
                        this.cancel(); 
                        return; 
                    } 
                    instance.doStuff(); 
                    counter[0]++; 
                } 
            }.runTaskTimer(this.plugin, 0L, 2L); 
        } 
    } 
}

用AtomicInteger（适合多线程场景）： #

如果你的代码涉及多线程操作，用AtomicInteger会更安全，不过这里Bukkit的任务是单线程执行的，数组其实就够用了：

import java.util.concurrent.atomic.AtomicInteger;

public class Foo { 
    private class Bar { 
        public void barMethod() { 
            final AtomicInteger counter = new AtomicInteger(0); 
            Bar instance = this; 
            BukkitTask barTask = new BukkitRunnable() { 
                @Override 
                public void run() { 
                    if (counter.get() == 5) { 
                        this.cancel(); 
                        return; 
                    } 
                    instance.doStuff(); 
                    counter.incrementAndGet(); 
                } 
            }.runTaskTimer(this.plugin, 0L, 2L); 
        } 
    } 
}

方法3：把计数器放到Runnable内部 #

你也可以直接把counter放到匿名Runnable类里，作为它的成员变量，这样就完全不需要依赖外部的局部变量了，逻辑也更内聚：

public class Foo { 
    private class Bar { 
        public void barMethod() { 
            Bar instance = this; 
            BukkitTask barTask = new BukkitRunnable() { 
                // 把counter放到内部类里
                private int counter = 0;
                
                @Override 
                public void run() { 
                    if (counter == 5) { 
                        this.cancel(); 
                        return; 
                    } 
                    instance.doStuff(); 
                    counter++; 
                } 
            }.runTaskTimer(this.plugin, 0L, 2L); 
        } 
    } 
}

内容的提问来源于stack exchange，提问作者user3310334