Thread - Banktransfer

[bvdcomp]

Hallo zusammen

Ich habe eine Aufgabe zu lösen, und zwar soll ich mittels Thread eine Simulation bauen die folgendes macht:

Eine Person zahlt auf ein Konto 1Mio Mal einen Betrag ein, und eine andere Person hebt gleichzeitig einen anderen Betrag ebenfalls 1Mio Mal ab.

Bis jetzut habe Ich folgendes realisiert:

public class Bank{
	
	public static void main (String[] args) {
		Account k = new Account();
		Person p1 = new Person(k,458);
		Person p2 = new Person(k,-99);
		
		p1.start();
		p2.start();
		
		try{
			p1.join();
		}
		catch(InterruptedException e)
		{
			System.out.println("Thread Person 1 was interrupted");
		}
		try{
			p2.join();
		}
		catch(InterruptedException e)
		{
			System.out.println("Thread Person 2 was interrupted");
		}
		
		System.out.println("Schlussaldo: "+ k.getSaldo());
	}
}

public class Person extends Thread {
	Account konto;
	private int betrag;
	
	public Person(Account k, int zahl) {
		this.konto = k;
		this.betrag = zahl;
	}

	public synchronized void run()
	{
		for(int i = 0; i <=1000000; i++)
		{
			konto.transfer(konto,betrag);
				try {
					Thread.sleep((int)Math.random()*100);
				}
				catch(InterruptedException e) {
			    }
		}
	}
}

public class Account{
	Account konto;
	private int saldo;
	
	public synchronized void transfer(Account konto, int n)
	{
		this.konto = konto;
		if (n > 0){
			konto.saldo += n;
		}
		else{
			konto.saldo -=n;
		}
	}
	
	public synchronized int getSaldo()
	{
		return saldo;
	}
}

Ich finde hier keinen Fehler, es ist aber ganz sicher einer da, denn die Ausgabe stimmt nicht.

Wer kann weiterhelfen?

Danke

 

[Marcinek]

Was stimmt an der Ausgabe nicht?

 

[bvdcomp]

Im Debug-Mode erhalte ich folgendes:
<terminated, exit value: 0>C:\Program Files (x86)\Java\jre1.6.0_20\bin\javaw.exe (15.09.2011 00:46:42)

Die Augabe stimmt auch nicht:
Schlussaldo: 557000557

Ich starte ja mit:
Person p1 = new Person(k,458);
Person p2 = new Person(k,-99);

Person 1 zahlt 1 Mio Mal 458 ein. -> 458000000.--
Person 2 hebt -99 ab. --> -99000000.--

Müsste doch 359000000.-- geben?

 

[norm]

der fehler ist in deiner transfer methode glaube ich:

    public synchronized void transfer(Account konto, int n)
    {
        this.konto = konto;
        if (n > 0){
            konto.saldo += n;
        }
        else{
            konto.saldo -=n;
        }
    }

ist n<0 darfst du nicht -= machen! dadurch addierst du ja wieder und das erklärt auch (fast) dein ergebnis!

 

[bvdcomp]

Wenn ich

if (n > 0){
			konto.saldo += n;
		}
		else{
			konto.saldo +=n;
		}

mach erhalte ich dann:
Schlussaldo: 359000359, das Ergebnis stimmt aber nur teilweise.

mit anderen Beträgen ist das Ergebnis wie folgt: die letzen Drei Zahlen sind gleich wie die ersten Drei!
Wie das sein kann, verstehe ich nicht.

 

[norm]

du machst 1000001 durchläufe
< statt <=

hier:

public synchronized void run()
    {
        for(int i = 0; i <=1000000; i++)
        {
            konto.transfer(konto,betrag);
                try {
                    Thread.sleep((int)Math.random()*100);
                }
                catch(InterruptedException e) {
                }
        }
    }

 

[bvdcomp]

Ohhhhhhhhhhhhh

Schlau ist, wer lesen kann.

Danke vielmals.
Aufgabe verstanden und gelöst.:toll: