Java多线程查询

Question

您好，我是并发编程领域的新手，我正在测试下面的代码，似乎 while 循环没有终止线程。谁能帮忙解释一下这里发生了什么。

public class PrimePrinter{
    public long counter = 0;
    public synchronized long getCounter() {
        return counter++;
    }
    public Thread makeThread() {
        Runnable rn = new Runnable() {          
            /* (non-Javadoc)
             * @see java.lang.Runnable#run()
             */
            @Override
            public void run() {
                while (counter < 100) {
                    try {
                        Thread.sleep(1000);
                        System.out.println(Thread.currentThread().getName() + " : " +getCounter());
                    } catch (InterruptedException e) {
                        e.printStackTrace();
                    }                   
                }
            }
        };
        return new Thread(rn);
    }
    public static void main(String[] args) {
        int n = 10;
        Thread[] threads = new Thread[10];
        PrimePrinter pp = new PrimePrinter();
        for(int i = 1; i < n; i++) {            
            threads[i] = pp.makeThread();
            threads[i].start();
        }
    }
}

最后几行输出

Thread-4 : 81
Thread-5 : 87
Thread-7 : 91
Thread-5 : 97
Thread-2 : 95
Thread-4 : 98
Thread-6 : 96
Thread-8 : 90
Thread-1 : 93
Thread-3 : 92
Thread-0 : 94
Thread-2 : 99
Thread-6 : 107
Thread-3 : 103
Thread-0 : 104
Thread-1 : 105
Thread-8 : 106
Thread-5 : 102
Thread-4 : 101
Thread-7 : 100

Answer 1

这是行不通的一个重要原因，不容错过。考虑您的代码实际运行的方式。它检查计数器值，然后 hibernate ，然后打印然后递增。

您有 9 个线程同时运行。然后，其中 8 个线程检查该值，此时，该值小于 100。因此它们通过了测试并继续前进。然后所有 8 个 sleep 1000 毫秒。同时，您的另一个线程刚刚增加了值...还有 8 个线程即将从 sleep 中醒来。

因此，如果您的值是 99(这是不可避免的)，您现在将从刚刚递增的线程中获得值 100。然后，那些已经通过测试的 8 也将递增，使计数器的结果值为 108。

但是，最后运行的线程将显示值 107，因为您打印 THEN 所有运行的增量。

如果您以相反的方式执行此操作(如在下面的代码中)，则不会遇到此问题。

以下代码可以正常工作:

public class PrimePrinter {
    public long counter = 0;

    public synchronized long getCounter() {
        return counter;
    }

    public synchronized void incrementCounter() {
        counter++;
    }

    public Thread makeThread() {
        Runnable rn = new Runnable() {
            /*
             * (non-Javadoc)
             * 
             * @see java.lang.Runnable#run()
             */
            @Override
            public void run() {
                while (getCounter() < 100) {
                    try {
                        incrementCounter();
                        System.out.println(Thread.currentThread().getName()
                                + " : " + getCounter());
                        Thread.sleep(1000);
                    } catch (InterruptedException e) {
                        e.printStackTrace();
                    }
                }
            }
        };
        return new Thread(rn);
    }

    public static void main(String[] args) {
        int n = 10;
        Thread[] threads = new Thread[10];
        PrimePrinter pp = new PrimePrinter();
        for (int i = 1; i < n; i++) {
            threads[i] = pp.makeThread();
            threads[i].start();
        }
    }
}