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

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 !!!）数据。

什么都可能发生。 “任何内容”都包括由于某些“巧合”而使代码大部分时间都可以使用的情况：

• 手头的任务很小。 （仅添加）
• 任务已创建并立即启动。
• 在两个“ create + start”对之间有一个小的延迟： 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线程池 。