为什么线程仍然不一致？

Question

我有一些模拟银行转账的java代码。账户类只有一个余额字段和一个向余额字段添加一些余额的转账方法。

TransferManager 定义了一个 Transfer 类，该类采用两个 Account 对象将给定金额从一个帐户转移到另一个帐户（作为参数传递）。

Manager 本身有两个重要的方法需要同步，因为它们都在同一资源上操作，并且将以线程方式调用：

public synchronized void issueTransfer(Account from, Account to, int amount) {
    openTransfers.add(new Transfer(from, to, amount));
    issuedTransfers++;
}

public synchronized void performTransfers() {
    for(Transfer transaction : openTransfers) {
        transaction.performTransfer();
            performedTransfers++;
    }       
    openTransfers.clear();
}

如果没有这里的同步语句，我会在存储和读取传输的数组列表上得到 NullPointerExceptions。

BankTest 产生 10 个线程，每个线程发出 10 笔转账。看看 BankTest.java 就可以了。问题是并不总是发出 10*10 的转账。有时有 98 或 99：

我是否必须向 BankTest.java 添加同步？我应该怎么办？还有其他想法或建议吗？

TransferManager.java：http://pastebin.com/Je4ExhUz

BankTest .java：http://pastebin.com/cdpWhHPb

Exersice3.java: http ://pastebin.com/v7pwJ5T1

Account.java：http://pastebin.com/QYEeWy5Z

Answer 1

try {
    Thread.sleep(60);
} catch (InterruptedException e) {
    e.printStackTrace();
}

这并不能保证所有线程都完成，

而是将所有线程保留在列表中并对所有线程调用 join

try {
    for(Thread thr:threads)thr.join();
} catch (InterruptedException e) {
    e.printStackTrace();
}

Answer 2

竞争条件？

在不同步 openTransfers.add 语句的情况下，两个线程可以同时向 openTransfers 添加一个对象列表。假设列表为空，两个线程都可以将元素添加到第一个位置（第二个线程覆盖第一个位置），然后都增加大小。这将为您提供一个大小为 2、包含 1 个对象和 1 个空指针的列表。

这只是可能发生的许多不正确的事情之一。如果 10 个线程都执行 size = size + 1，则完成后 size 可以是 1 到 10 之间的任何值。这可以解释为什么有时会出现 99 次传输。 size 是 99，现在并不意味着有 99 个项目，列表中可能有 100 个元素，或者 98，或者只是 1。

您确实应该尝试同步多个线程可以写入的所有内容相同的数据。但要注意不要陷入僵局。