Öffentlicher finaler Mutex für Java-Thread-Sicherheit

Question

In meinem Projekt muss ich eine Klasse X verwenden, die viele Methoden bietet, aber das Dokument erwähnt nicht, ob diese Methoden Thread-sicher sind, und ich habe nicht den Quellcode entweder.

So habe ich X in einer anderen Klasse mit einem Mutex eingekapselt:

public class MyX { 
    private X instance; 
    public final Object mutex = new Object(); 
    public MyX() { 
     this.instance = new X(); 
    } 
    public X getMethods() { 
     return this.instance; 
    } 
} 

Wenn ich Methoden der X anrufen müssen, verwende ich einen synchronized Block:

MyX myX = new MyX(); 
synchronized (myX.mutex) { 
    X methods = myX.getMethods(); 
    methods.method1(); 
    methods.method2(); 
    ... ... ... 
} 

Oder ich vielleicht kann direkt auf eine Instanz der Klasse X:

X instance = new X(); 
synchronized(instance) { 
    instance.method1(); 
    instance.method2(); 
    ... ... ... 
} 

synchronisieren

Ich würde gerne wissen, welcher Weg besser ist, und gibt es ein besseres Design für dieses Problem.

Danke.

Answer 1

Wenn Sie zwischen zwei Methoden derselben Klasse synchronisieren möchten, sollten Sie sich für mutex entscheiden.

Freilegen von Mutex als öffentliche Variable ist gegen objektorientierte Prinzipien und daher nicht zu empfehlen.

Wenn Sie außerhalb der Klasse arbeiten, ist die Erfassung des Objekts, an dem Sie gerade arbeiten, die beste Option, würde ich sagen.

X instance = new X(); 
synchronized(instance) { 
instance.method1(); 
instance.method2(); 
... ... ... 
}