Notwendigkeit der Synchronisation auf lokalem Variable

Question

In JSON-Java-Bibliothek (org.json.JSONArray) Ich habe diesen Code-Schnipsel mit einem synchronized Block um ein Verfahren lokalen Variable gefunden

public String toString(int indentFactor) throws JSONException { 
    StringWriter sw = new StringWriter(); 
    synchronized (sw.getBuffer()) { 
     return this.write(sw, indentFactor, 0).toString(); 
    } 
} 

ich die Notwendigkeit nicht für die Synchronisation verstehen hier, wie die StringWriter ist nur lokal für die angegebene Methode (und warum die Synchronisation auf dem Puffer ist). Ist die Synchronisation hier wirklich notwendig und wenn, warum?

Answer 1

Dies kann eine Performance-Optimierung sein. Im Orakel jvm ist das Wiedererlangen eines bereits gehaltenen Schlosses sehr schnell. Vermutlich führt der Anruf write zahlreiche Aufrufe in den StringBuffer. Durch das Sperren vor dem Aufruf von write wird die Sperre durch alle diese Anrufe gehalten, anstatt für jeden Anruf freigegeben und erneut abgerufen zu werden.

Answer 2

Der leere Konstruktor für StringWriter ist

/** 
* Create a new string writer using the default initial string-buffer 
* size. 
*/ 
public StringWriter() { 
    buf = new StringBuffer(); 
    lock = buf; 
} 

Nichts geteilt, deshalb ist die synchronized Block nicht erforderlich ist.

Es sei denn ... write Delegierten zu einem anderen Thread, aber ich bezweifle ernsthaft, dass.

Answer 3

getBuffer() liefert einen StringBuffer und entsprechend der Dokumentation a StringBuffer bereits synchronisiert:

String-Puffer für die Verwendung durch mehrere Threads sicher sind. Die Methoden werden bei Bedarf synchronisiert, sodass sich alle Vorgänge in einer bestimmten Instanz so verhalten, als ob sie in einer Reihenfolge auftreten, die der Reihenfolge der Methodenaufrufe der einzelnen beteiligten Threads entspricht.

Dies bedeutet, dass die erneute Synchronisierung auf dem StringBuffer völlig übertrieben ist. Änderungen an der StringWriter werden automatisch synchronisiert, da intern die synchronisierte StringBuffer verwendet wird.

Da die StringWriter Instanz lokal für den Methodenaufruf ist, ist es unmöglich, mehrere Threads gleichzeitig auf dieselbe Instanz zugreifen zu lassen, was ebenfalls die Synchronisation unnötig macht.

Answer 4

Es ist ein Fehler. Jeder Thread erstellt seine eigene lokale Variable in der Methode und synchronisiert sich darauf. Jedes Mal, wenn die Methoden-Threads eingegeben werden, erstellen sie ihren eigenen Objekt-Monitor, der nicht von einem anderen Thread gehalten werden kann, weil er lokal ist und nur auf dem Thread-Stack lebt.