Czy poniższy kod wątku inkrementującego jest bezpieczny w java?

Question

kod Java:

public class IncreaseTest { 
    public static int value = 0; 

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

Czy metoda increment() wątku bezpieczny? Czy muszę dodawać słowa kluczowego modyfikujący volatile następująco:

public static volatile int value = 0; 

Answer 1

Ten kod nie jest bezpieczny wątku. Metoda instancji zsynchronizuje się na instancji, jeśli masz wiele instancji, nie będą używać tego samego monitora, a zatem aktualizacje mogą się przeplatać.

Musisz usunąć statyczne z pola value lub dodać statyczne do metody increment().

Ponadto, ponieważ publicznie wprowadzono value, istnieje dodatkowy problem, że wartość tę można zmienić lub przeczytać poza tą metodą, bez korzystania z synchronizacji, która mogłaby spowodować odczytanie starych wartości.

Więc zmienia swój kod poniżej uczyni to bezpieczny wątku:

public class IncreaseTest { 
    private int value = 0; 

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

Answer 2

Jeśli używasz tej metody w dwóch wątków następnie trzeba zrobić lotną słowa kluczowego. Bez tego inny wątek może nie uzyskać najbardziej aktualnej wartości. (C#)

Answer 3

nie sądzę, to jest wątek bezpieczne, ponieważ zmienna statyczna jest jawne i mogą być dostępne przez innych wątków w sposób nie- bezpieczny sposób wątku. Aby być bezpieczeństwo wątków należy zadeklarować zmienną w następujący sposób:

public static volatile int value; 

Teraz value są lotne, są dostępne w zsynchronizowany bloku będzie.

Answer 4

Powinieneś raczej użyć atomicvars