Threads Methode eines Threads synchornisieren

[Bali]

Hallo erstmal,

als absoluter Java-Anfänger habe ich mich mit dem Thema Threadsicherheit beschäftigt. Dabei habe ich eine Methode als synchronized deklariert in der Hoffnung, dass erst ein Thread diese Methode vollständig bearbeitet bevor es der zweite macht. Diesen Gefallen wollte mir das Programm nicht tun. Was mach ich nur falsch? ???:L

public class ThreadTest extends Thread 
{
	int zaehler;	
		
	public static void main(String[] args) 
	{
		Thread thread1 = new ThreadTest(1);
		Thread thread2 = new ThreadTest(2);			
		thread1.start();
		thread2.start();
	}
		
	public ThreadTest(int zaehler)
	{
		this.zaehler = zaehler;
	}
		
	public void run()
	{
		machWas();
	}
		
	public synchronized void machWas()
	{
		for(int i = 0; i < 1000; i++)
		{	
			System.out.print(zaehler);
		}
	}
}

 

[SlaterB]

hast du dir zu synchronized etwas vorher durchlesen oder probierst du herum?

du synchronisierst auf eine nicht statische ganz normale Methode der ThreadTest-Klasse,
das interessiert nur das jeweilige Objekt, es gibt keine globale Verknüpfung,

du könntest hier die Methode statisch machen, was allgemein eher nicht ratsam ist,
auf ein globales Objekt wie ThreadTest.class synchronisieren
oder irgend ein anderes gemeinsames Objekt, einen Monitor, finden und an beide Threads übergeben, etwa Parameter im Konstruktor

 

[fastjack]

In diesem Fall brauchst Du sowieso keine Synchronisierung, da die Threads zwei unterschiedliche Objekte sind und auch die Zähler verkörpern. Synchronisierung bräuchtest Du, wenn sich beide Threads ein "Zählerobjekt" teilen.

 

[Bali]

Ich habe schon vorher gelesen und gedacht, es verstanden zu haben bis zu dem ThreadTest. Ich finde das Thema ziemlich kompliziert.

Also wenn ich meinem Konstruktor ein gemeinsames Objekt übergebe und diesen dann aufrufe mit ThreadTest.class ...

public class ThreadTest extends Thread 
{
	int zaehler;	
	Object o;
	
	public static void main(String[] args) 
	{
		Thread thread1 = new ThreadTest(1, ThreadTest.class);
		Thread thread2 = new ThreadTest(2, ThreadTest.class);			
		thread1.start();
		thread2.start();		
	}
		
	public ThreadTest(int i, Object obj)
	{
		this.zaehler = i;
		this.o = obj;
	}
...

... wie erkläre ich der Methode machWas(), dass sie jetzt auf das Objekt der Klasse synchronizieren soll ohne eine synchornizierte Blockstruktur zu erstellen.

 

[Michael...]

... wie erkläre ich der Methode machWas(), dass sie jetzt auf das Objekt der Klasse synchronizieren soll ohne eine synchornizierte Blockstruktur zu erstellen.

ohne synchronisierte Blockstruktur geht nicht. Interessant wäre zu wissen warum du zwei Thread Methoden, die scheinbar völlig unabhängig arbeiten könnten synchronisieren willst bzw. warum du dann überhaupt Threads verwendest.

 

[SlaterB]

> bzw. warum du dann überhaupt Threads verwendest.
na allein schon um das ganze zu testen, ohne Threads kann man auch synchronized vergessen

 

[fastjack]

* Mach ein Objekt Zaehler, das hat eine Methode um den Zähler um eins zu erhöhen.
* Dieses Objekt instanzierst Du jetzt EINMAL.
* Dann machst Du zwei Threads, die beide dieses Zaehlerobjekt benutzen und die Zaehlmethode aufrufen

Damit hast Du die Basis, die Du Dir jetzt gerade vorstellst und kannst mit synchronize experimentieren.

 

[Michael...]

> na allein schon um das ganze zu testen, ohne Threads kann man auch synchronized vergessen
aber sucht man sich dafür nicht irgendwelche sinnvolle oder realitätsnahe Beispiele
Woher soll man denn wissen, wenn zwei Methoden nacheinander ausgeführt werden, ob das threading überhaupt funktioniert hat ;-)

 

[Bali]

Ich habe nicht versucht ein Beispiel zu finden in dem es Sinn macht, Threads zu synchonisieren. Natürlich können in meinem Beispiel die Threads auch arbeiten ohne sich zu stören, wenn sie nicht synchronisiert sind. Es geht halt nur darum eine Methode zu synchonisieren und die Ausgabe soll mir zeigen, ob es denn funktioniert hat.

Also noch mal nur um Missverständnisse auszuschließen: Wenn ich eine Methode Thread-sicher machen will, dann geht das nur über eine synchronisierte Blockstruktur?

Aber für was braucht man dann die synchronisierte Methode?

 

[hdi]

Der Begriff synchronisierte Methode ist irreführend. Synchronisiert wird immer auf ein Objekt, nie auf eine Methode. Man kann das zwar durchaus so ausdrücken, ist aber nicht wirklich korrekt. Denn:

synchronized void method(){
   doStuff();
}

ist nur eine Kurzschreibweise für:

void method(){
   synchronized(this){
      doStuff();
   }
}

Nicht mehr, nicht weniger. Es ist einfach das selbe.

Edit: Das mal ganz unabhängig davon, dass das so nicht funktioniert in deinem Beispiel. Es gilt: Alle Threads müssen auf das selbe Lock synchronisieren. Und in der ausgeschriebenen Version erkennst du: synchronisieren tut ein Thread in deiner Methode auf "this". Und zwei verschiedene Threads = zwei verschiedene this, also nicht das selbe Lock. Demnach auch keine Synchronisierung. Das meinte Michael mit:

ohne synchronisierte Blockstruktur geht nicht.

Da du nicht die Kursschreibweise verwenden kannst, die implizit auf this synchronisiert. Alle Threads müssen sich ein Lock teilen, und das geht halt dann nicht mehr mit der Kurzschreibweise. Zumindest nicht, wenn die Methode nicht-statisch sein soll.

 

[Bali]

Jetzt hab ich's!

Danke an alle!