5

我正在使用与原子变量（java.util.concurrent.atomic 包）进行比较的方法同步来试验 Java 多线程。

以下是课程：

// Interface ICounter.java
        public interface ICounter {
            public void increment();
            public void decrement();
            public int value();
        }

// Class Counter.java
    public class Counter implements ICounter {
        private int c = 0;

        @Override
        public void increment() {
            c++;
        }

        @Override
        public void decrement() {
            c--;
        }

        @Override
        public int value() {
            return c;
        }
    }

// Class AtomicCounter.java
    import java.util.concurrent.atomic.AtomicInteger;

    public class AtomicCounter implements ICounter {
        private AtomicInteger c = new AtomicInteger(0);

        @Override
        public void increment() {
            c.incrementAndGet();
        }

        @Override
        public void decrement() {
            c.decrementAndGet();
        }

        @Override
        public int value() {
            return c.get();
        }

        public long getIncrement() {
            return c.incrementAndGet();
        }
    }

// Class MainProg.java
    public class MainProg {
        public static void main(String args[]) {
            ICounter counter = new AtomicCounter();
                    //ICounter counter = new SynchronizedCounter();
            Thread thread1 = new Thread(new CountRunner(counter));
            Thread thread2 = new Thread(new CountRunner(counter));

            thread1.start();
            thread2.start();
        }
    }

    class CountRunner implements Runnable {
        private ICounter counter;
        public CountRunner(ICounter counter) {
            this.counter = counter;
        }

        public void run() {
            while (true) {
                counter.increment();
                System.out.println(Thread.currentThread().getName() + " count=" + counter.value());
                System.out.println("-------------------");
                try {
                    Thread.sleep(2000L);
                } catch (InterruptedException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }
            }
        }
    }

运行 Atomic 或 Synchronized 的结果并不表明变量 integer 是线程安全的，例如：

Thread-0 count=1
-------------------
Thread-1 count=2
-------------------
Thread-0 count=3
-------------------
Thread-1 count=4
-------------------
Thread-0 count=5
-------------------
Thread-1 count=6
-------------------
Thread-0 count=7
-------------------
Thread-1 count=8
-------------------
Thread-0 count=10
-------------------
Thread-1 count=10
-------------------

从结果中，最后 2 行显示 2 个线程正在访问计数器类的整数变量的相同值。也许我在这里遗漏了什么？

谢谢！

4

3 回答 3

4

因为你的counter.increment()andSystem.out.println不是一个原子动作。

Thread 1               Thread2

increment

                       increment

                       System.out.println  // print 10

System.out.println  
// print 10 too
于 2010-08-15T19:58:50.760 回答
4

您在一步中增加值，然后在另一步中获取值。虽然这些单独的步骤中的每一个都被底层AtomicInteger类保证是原子的，但是您进行两个单独的操作的事实使您在 print 语句中看到的值受线程执行顺序的支配。

为了能够准确地显示给定线程更新的值，您需要在单个操作中更新和获取结果值，您的getIncrement()方法就是这样做的。可以为您提供预期结果的代码如下所示：

int changedValue = counter.getIncrement();
System.out.println(Thread.currentThread().getName() + " count=" + changedValue);
于 2010-08-15T20:00:03.013 回答
2

您错过的是您的AtomicCounter类确实可以正常工作，并且观察到的行为发生在线程在调用.increment()和之间切换.value()：

-------------------
Thread-0 increment -> value = 9
-------------------
Thread-1 increment -> value = 10
-------------------
Thread-0 print value <- 10
-------------------
Thread-1 print value <- 10
-------------------
于 2010-08-15T19:58:47.313 回答