Моя синхронизация не работает - PullRequest
0 голосов
/ 06 февраля 2012

Я пытался использовать synchronized и ReentrantLock в Bank.transfer, но я получаю вывод, подобный этому:

"От 7 до 3 передано 82,0. Всего 918", "От 0 до 4 переведено 27,0. Всего 973"

Хотя Total должен быть равен 1000. Скажите, что я не так?

public class expr {
    public static Bank b = new Bank();
    public static void main(String[] args) throws IOException, InterruptedException {
        for (int i = 0; i < 4; i++) {
            new BankTransfer();
        }
    }
}

public class BankTransfer implements Runnable{

    public BankTransfer() {
        Thread t = new Thread(this);
        t.start();
    }

    @Override
    public void run() {

        while (true){
            int from = (int) (expr.b.size * Math.random());
            int to = (int) (expr.b.size * Math.random());
            int amount = (int) (100 * Math.random());
            expr.b.transfer(from, to, amount);

            try {
                Thread.sleep((long) (2000 * Math.random()));
            } catch (InterruptedException e) {
                System.out.println("Thread was interrupted!");
                return;
            }

        }
    }


}

public class Bank {
    private int[] accounts;
    public int size = 10;
    private Lock block = new ReentrantLock();
    public boolean transfer(int from, int to, double amount){
        block.lock();
        try{
            if(accounts[from] >= amount && from != to){
                accounts[from] -= amount;
                System.out.println("From " + from + " to " + to + " transfered " + amount + ". Total " + getTotal());
                accounts[to] += amount;
                return true;
            }
        }finally {
            block.unlock();
        }
        return false;
    }
    public Bank(){
        accounts = new int[size];
        for (int i = 0; i < size; ++i) {
            accounts[i] = 100;
        }
    }
    private int getTotal(){
        int sum = 0;
        for (int i = 0; i < size; ++i) sum += accounts[i];
        return sum;
    }
}

Ответы [ 3 ]

3 голосов
/ 06 февраля 2012

Подсчитайте сумму ПОСЛЕ того, как вы завершили оба конца трансфера ... т.е. переместите System.println после счета [в] + = сумма.

1 голос
/ 06 февраля 2012

Вы звоните getTotal() после вычета денег с одного счета, но до добавления их на другой счет. В зависимости от суммы перевода всегда будет отображаться меньше 100.

1 голос
/ 06 февраля 2012

Эта часть выглядит странно:

accounts[from] -= amount;
System.out.println("From " + from + " to " + to + " transfered " + amount + ". Total " + getTotal());
accounts[to] += amount;

Вы печатаете сумму до передачи завершена.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...