getClass()-Methode

Okay, wenn ich das mit dem Klassenobjekt und den Kopien davon (den Instanzen) richtig verstanden habe, dann sollte mein Beispiel

public void insertThread( ThreadMetaInfo newThread ) {
        synchronized(getClass()) {...

aber genauso funktionieren wie

public synchronized void insertThread( ThreadMetaInfo newThread ) {...

Wobei letzteres natürlich schöner ist.

tfa

tomatesalat schrieb:

Okay, wenn ich das mit dem Klassenobjekt und den Kopien davon (den Instanzen) richtig verstanden habe, dann sollte mein Beispiel
public void insertThread( ThreadMetaInfo newThread ) {
        synchronized(getClass()) {...
aber genauso funktionieren wie
public synchronized void insertThread( ThreadMetaInfo newThread ) {...
Wobei letzteres natürlich schöner ist.

Das sind zwei verschiedene Sachen! Die synchronisierte (nicht-statische) Methode entspricht einem synchronized(this)-Block. Auf getClass() zu synchronisieren hat eine völlig andere Bedeutung. Hier wird auf das Class-Objekt synchronisiert und davon gibt es nur eins für deine Klasse. Objekte kann es aber viele geben.

Wenn du in Java Nebenläufigkeit unterstützen willst, solltest du dir auf jeden Fall das Concurrency-Framework ansehen (ab Java 1.5 in der Standard-API). Damit wird vieles einfacher und sicherer als mit dem lowlevel synchronized.

tfa schrieb:

Das sind zwei verschiedene Sachen! Die synchronisierte (nicht-statische) Methode entspricht einem synchronized(this)-Block. Auf getClass() zu synchronisieren hat eine völlig andere Bedeutung. Hier wird auf das Class-Objekt synchronisiert und davon gibt es nur eins für deine Klasse. Objekte kann es aber viele geben.

Ja und? Dann ist halt das Klassenobjekt das Synchronisationsobjekt. Wo ist denn jetzt der Unterschied zu synchronized(this), außer dass nun auch unterschiedliche Instanzen ausgeschlossen werden? Die Auswirkungen sollten die gleichen sein, wenn es nur eine Instanz meines Objektes gibt.

tfa

Wenn es nur ein Exemplar gibt, kommt es auf das selbe raus (aber du schriebst was von "den Instanzen"). Trotzdem sollte man gegen "this" synchronisieren, man weiß nie, was in der Zukunft noch dazu kommen kann.

Super, vielen Dank für Eure Antworten.

cof

Die Synchronisation ist nicht gewährleistet, wenn du auf this lockst, falls du mehrer Objekte hast und die Threads auf verschiedene Objekte zugreifen.
In dem Fall auf das Klassenobjekt oder ein dezidiertes Lockobjekt locken.

tfa

cof schrieb:

Die Synchronisation ist nicht gewährleistet, wenn du auf this lockst, falls du mehrer Objekte hast und die Threads auf verschiedene Objekte zugreifen.

Jedes Objekt hat seinen eigenen Zustand. Selbstverständlich können verschiedene Threads gleichzeitig auf verschiedene Objekte zugreifen, ohne dass es Synchronisazionsprobleme mit synchronized(this) gibt.

cof schrieb:

In dem Fall auf das Klassenobjekt oder ein dezidiertes Lockobjekt locken.

Klar geht das, aber es hat eine andere Bedeutung.

cof

ich schriebe nicht, dass es probleme gibt. nur, dass du aufpassen musst auf welches Objekt du synchronisierst. wenn du auf this syncs und du hast mehr als ein Objekt, dann nützt dir nein Lock genau null, weil es eben nicht das selbe Objekt ist. in dem fall sicher stellen, dass es das selbe objekt ist, also entweder class objekt oder ein lock-objekt.

tfa

cof schrieb:

ich schriebe nicht, dass es probleme gibt. nur, dass du aufpassen musst auf welches Objekt du synchronisierst. wenn du auf this syncs und du hast mehr als ein Objekt, dann nützt dir nein Lock genau null, weil es eben nicht das selbe Objekt ist. in dem fall sicher stellen, dass es das selbe objekt ist, also entweder class objekt oder ein lock-objekt.

Nein, nochmal: verschiedene Objekte, verschiedene Zustände. Threads werden pro Objekt synchronisiert. Siehe z.B. Vector oder Hashtable. Die sind auch nur gegen "this" synchroniert, obwohl es viele Objekte davon geben kann.

Alles andere (globales Monitor-Objekt oder Klassenobjekt) ist ein völlig anderer Anwendungsfall und eher selten.

cof

MeinObjekt ob1 = new MeinObjekt();

wenn irgendwo im MeinObjekt Code

synchonized(this) {..}

steht und 2 verscheidene Threads eine Instanz von MeinObjekt haben, dann ist der Block NICHT synchronisiert, weil 2 verschiedene Instanzen.
sollte allerdings

synchronized(MeinObjekt.class) {..}

stehen, dann ist der entsprechende Codeblock synchronisiert, da MeinObjekt.class für beide Threads das gleiche Objekt ist.

Shade Of Mine

cof schrieb:

MeinObjekt ob1 = new MeinObjekt();
wenn irgendwo im MeinObjekt Code
synchonized(this) {..}
steht und 2 verscheidene Threads eine Instanz von MeinObjekt haben, dann ist der Block NICHT synchronisiert, weil 2 verschiedene Instanzen.
sollte allerdings
synchronized(MeinObjekt.class) {..}
stehen, dann ist der entsprechende Codeblock synchronisiert, da MeinObjekt.class für beide Threads das gleiche Objekt ist.

Da die beiden Objekte aber keinen shared state haben, ist eine synchroniesierung zwischen den beiden objekten _nicht_ notwendig.