Konieczność zsynchronizowane w zmiennej lokalnej

Question

W bibliotece JSON Java (org.json.JSONArray) Znalazłem ten fragment kodu z synchronized bloku wokół zmiennej metody lokalnego

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

Nie rozumiem konieczność synchronizacji tutaj, ponieważ StringWriter jest tylko lokalny dla podanej metody (i dlaczego synchronizacja dotyczy bufora). Czy synchronizacja jest naprawdę potrzebna tutaj i jeśli, dlaczego?

Answer 1

Może to być optymalizacja wydajności. W wyroczni jvm, ponowne przejęcie już posiadanego zamka jest bardzo szybkie. Przypuszczalnie wywołanie write wywołuje wiele wywołań w StringBuffer. Blokując przed wywołaniem write, blokada będzie odbywać się przez wszystkie te połączenia zamiast być zwolnione i ponownie nabyte dla każdego połączenia.

Answer 2

Pusty konstruktor StringWriter jest

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

Nic nie jest udostępniony, więc blok synchronized jest niepotrzebna.

O ile ... write nie deleguje do innego wątku, ale poważnie w to wątpię.

Answer 3

getBuffer() zwraca StringBuffer i zgodnie z dokumentacją StringBuffer jest już zsynchronizowane:

bufory strunowe są bezpieczne do użytku przez wiele wątków. Metody są zsynchronizowane w razie potrzeby, tak aby wszystkie operacje na konkretnej instancji zachowywały się tak, jakby występowały w jakiejś kolejności szeregowej zgodnej z kolejnością wywołań metod tworzonych przez każdy z poszczególnych wątków.

Oznacza to, że synchronizacja ponownie na StringBuffer jest całkowicie przesadzona. Zmiany w StringWriter zostaną automatycznie zsynchronizowane, ponieważ wewnętrznie używają zsynchronizowanego StringBuffer.

Ponieważ instancja StringWriter jest lokalna dla wywołania metody, niemożliwe jest posiadanie wielu wątków uzyskujących dostęp do tej samej instancji w tym samym czasie, co również powoduje, że synchronizacja jest niepotrzebna.

Answer 4

To jest błąd. Każdy wątek tworzy w metodzie swoją własną zmienną lokalną i synchronizuje ją. Każdy czas wprowadzania wątków metod tworzy własny monitor obiektu, który nie może być trzymany przez inny wątek, ponieważ jest lokalny i działa tylko na stosie wątku!