Ist der folgende Inkrementierungscode-Thread in Java sicher?

Question

Java-Code:

public class IncreaseTest { 
    public static int value = 0; 

    public synchronized int increment() { 
     return value++; 
    } 
} 

Ist Methode increment() Thread-sicher? Muss ich den Modifikator Stichwort volatile hinzufügen wie folgt:

public static volatile int value = 0; 

Answer 1

Dieser Code ist nicht threadsicher. Die Instanzmethode wird auf einer Instanz synchronisiert, wenn Sie mehrere Instanzen haben, werden sie nicht den gleichen Monitor verwenden, und daher können die Aktualisierungen verschachtelt werden.

Sie müssen entweder die statische aus dem value Feld entfernen oder statisch zur increment() Methode hinzufügen.

Auch, wie Sie value public gemacht haben, gibt es das zusätzliche Problem, dass Wert außerhalb dieser Methode geändert oder gelesen werden kann, ohne Synchronisierung zu verwenden, die zum Lesen alter Werte führen könnte.

Code So ändert den folgenden wird es machen Thread-sicher-:

public class IncreaseTest { 
    private int value = 0; 

    public synchronized int increment() { 
     return value++; 
    } 
} 

Answer 2

Wenn Sie diese Methode in zwei Threads verwenden, dann müssen Sie das Schlüsselwort volatile. Ohne es wird ein anderer Thread möglicherweise nicht den aktuellsten Wert erhalten. (C#)

Answer 3

Ich glaube nicht, dieser Thread sicher ist, da die statische Variable ist öffentlich und kann von anderen Threads in einem nicht zugegriffen werden fadensichere Art und Weise. Um sicher zu fädeln müssen Sie die Variable wie folgt deklarieren:

public static volatile int value; 

Jetzt value flüchtig ist, wird in einem synchronisierten Block zugegriffen werden.

Answer 4

Sie sollten wahrscheinlich verwenden atomicvars