Блокировка всех экземпляров класса в Java

В этом случае вы можете использовать ReadWriteLock. Метод transferMoney будет использовать блокировку чтения, поэтому его можно выполнять одновременно. Метод sumAccounts будет использовать блокировку записи, поэтому, когда он выполняется, ни один transferMoney (или sumAccounts) не может выполняться из других потоков.

Использование ReentrantLock и синхронизация обоих методов на уровне класса будет вести себя так же, как Вы заявили, потому что они не позволят одновременное выполнение метода transferMoney.

образец кода:

final ReadWriteLock rwl = new ReentrantReadWriteLock();

public boolean transferMoney(Account from, Account to, int amount) {
  rwl.readLock().lock();
  try{
    .... Your current code here
  }
  finally {
       rwl.readLock().unlock();
  }
}

public int sumAccounts(List<Account> accounts) {
  rwl.writeLock().lock();
  try{
    // You dont need atomic integer here, because this can be executed by one thread at a time
    int sum = 0;
    for (Account a : accounts) {
        sum += a.getBalance();
    }
    return sum;
  }
  finally {
       rwl.writeLock().unlock();
  }
}

Также справедливый режим реентерабельных блокировок будет работать медленнее, чем несправедливые режимы. Подробности смотрите в документации.

https://docs.oracle.com/javase/7/docs/api/java/util/concurrent/locks/ReentrantReadWriteLock.html.

Как указано в комментарии, блокировка объекта класса не приведет к блокировке всех экземпляров этого класса, а просто заблокирует объект Class, представляющий ваш класс Account. Эта блокировка не несовместима с блокировками на объектах Account, поэтому у вас вообще не происходит синхронизация.

Взятие блокировок на отдельные объекты Account можно выполнить внутри цикла for (в sumAccounts), но это не предотвратит выполнение таких расписаний:

- sumAccounts locks 'first' Account and reads balance (and releases lock again at end of the synchronized block taking the lock)
- system schedules a moneyTransfer() from 'first' to 'last'
- sumAccounts locks 'last' Account and reads balance, which includes the amount that was just transferred from 'first' and was already included in the sum

Поэтому, если вы хотите предотвратить это, вам также необходимо синхронизировать обработку moneyTransfer () в Account.class (что затем устраняет необходимость блокировки отдельных объектов).

Очень сложно просмотреть ваш код, потому что у нас нет возможности узнать, являются ли учетные записи объектов, с которыми вы синхронизируете, одними и теми же экземплярами во всех функциях.
Прежде всего, мы должны согласовать, будет ли сумма балансов и перевод сумм - это две операции, которые должны выполняться одновременно.
Я ожидаю, что сумма остатков будет одинаковой до и после перевода сумм.
Кроме того, вы используете synchronized(Account.class) в сумме балансов, что неверно. Вы должны синхронизировать объекты, которые вы перебираете в цикле.
Теперь, даже если вы действительно координируете одни и те же экземпляры, вы все равно можете иметь следующее расписание:

Thread-1 (transfer)  
  locks from  
Thread-2 (sum balance)  
  locks first object in the list and adds the balance to the running sum and moves to next object
Thread-1  
   locks to (which is the object Thread-2) processed
   moves money from => to  

Вы уже суммировали to с суммой до увеличения, и вы можете добавить from с суммой после вычета в зависимости от графика.

Проблема в том, что вы обновляете 2 объекта в передаче, но блокируете только 1 в сумме.
Я бы посоветовал либо:

1. либо синхронизировать оба метода на одной и той же блокировке и заставить их запускаться последовательно
2. установить какой-то грязный флаг, когда объекты попадают в метод transfer, и если он установлен, пропустить их в сумме баланса и закончить сумму, когда все обновления будут выполнены
3. Почему вы вообще это делаете на Java? Это должно происходить в базе данных с использованием транзакций со свойствами ACID.