Question

我试图弄清楚java中这个线程程序有什么问题。任何人都能解释一下吗？这是代码：

public class Z {
    private Account account = new Account();
    private Thread[] thread = new Thread[100];

    public static void main(String[] args) {
        Z test = new Z();
        System.out.println("What is balance ? " + test.account.getBalance());
    }

    public Z() {
        ThreadGroup g = new ThreadGroup("group");
        boolean done = false;

        // Create and launch 100 threads
        for (int i = 0; i < 100; i++) {
            thread[i] = new Thread(g, new AddAPennyThread(), "t" + i);
            thread[i].start();
            System.out.println("depositor: " + thread[i].getName());
        }
        // Check if all the threads are finished
        while (!done)
            if (g.activeCount() == 0)
                done = true;
    }

    // A thread for adding a penny to the account
    class AddAPennyThread extends Thread {
        public void run() {
            account.deposit(1);
        }
    }

    // An inner class for account
    class Account {
        private int balance = 0;

        public int getBalance() {
            return balance;
        }

        public void deposit(int amount) {
            int newBalance = balance + amount;
            balance = newBalance;
        }
    }
}

它编译并运行良好。这是一个我错过的测试问题，并且知道它究竟出了什么问题。谢谢！

Answer 1

没有一个专用于同步100个线程的位，所有线程都处理一个（1 !!!）数据。

任何都可能发生。 “任何东西”包括由于某些“巧合”而导致的代码大部分时间都在起作用：

手头的任务很小。（只是一个补充）
创建并立即启动任务。
两个“创建+开始”对之间有一个小延迟：System.out.println。

这加起来：可能在大多数测试运行中工作。但这是一个错误的，不确定的程序。

Answer 2

[Tread1] int newBalance = balance + amount;
[Tread2] int newBalance = balance + amount;
[Tread1] balance = newBalance;
[Tread2] balance = newBalance;

public synchronized void deposit(int amount)

Answer 3

balance应该是private volatile int（因此Java知道永远不会缓存它 - 它的值在不同的线程访问之间可能会发生变化，而下一个线程不知道）并使Account#deposit(int amount)成为public synchronized void（因此Java使方法体成为一个关键区域，并阻止同时访问它触及的任何对象，确保balance的值的完整性。）

此外，使用new自己实例化100个线程会带来很多开销 - 虽然我很欣赏这只是一个玩具示例，但更有效的方法是使用Java thread pool。< / p>

这个java线程程序有什么问题

3 个答案: