如果我在这段代码中用 volatile 替换 AtomicBoolean 会有什么风险？

Question

I'm reading JCP 7.2.4, it has the following code:

    boolean checkMail(Set<String> hosts, long timeout, TimeUnit unit)
    throws InterruptedException
    {
        ExecutorService exec = Executors.newCachedThreadPool();
        final AtomicBoolean hasNewMail = new AtomicBoolean(false); // can we use volatile here?
        try{
            for(final String host : hosts){
                exec.execute(new Runnable() {
                    public void run(){
                        if(checkMain(host)){
                            hasNewMail.set(true);
                        }
                    }
                });
            }
        }finally{
            exec.shutdown();
            exec.awaitTermination(timeout, unit);
        }
        return hasNewMail.get();
    }

The book says: Atomic, not volatile, should be used, as the inner Runnable could visit hasNewMail .

Boolean value is set to true when there's mail, without considering if its previous value is true of false, regardless whether other threads are setting its value.

My doubt is, if I replace this AtomicBoolean with volatile, what will be the potential risk? Any working thread could set it to true(when find mail), so is there chance I couldn't get correct value?

Answer 1

Obviously not; local variables that are used in lambdas or inner classes must be effectively final . AtomicBoolean works because your variable is a reference to an object - and doesn't change.

Perhaps try it first before asking :)