Problem mit synchronisierten Methode in Java

Question

Ich habe folgende Implementierung in Java, wo ich eine synchronisierte Methode zu verwenden, versuchen:

class dbAccess{ 
    public synchronized void getGUID(){ 
      counter=/*Access last count from txn_counter table */ 
      /*Insert a unique value to txn_counter table based on the acquired value of counter */ 
      /*Insert new counter value to GUID_log table */ 
    } 
} 

Der Abschnitt zwischen/* */einigen SQL-Abfragen darstellen. Die Implementierung hat 10 Threads. Ich hatte gehofft, dass der Rückgabewert jedes Mal einmalig sein würde. Es kommt jedoch vor, dass mehrere Läufe den gleichen Wert des Zählers zurückgeben.

Können Sie bitte darauf hinweisen, wenn ich etwas falsch mache. Und ist es der richtige Weg, dies zu tun?

Answer 1

Vielleicht haben Sie mehrere Instanzen von dbAccess? (Das Schlüsselwort synchronized funktioniert auf Objektebene nicht auf Klassenebene.) In diesem Fall müssen Sie die Methode statisch machen (in Ihrer Situation möglicherweise nicht möglich) oder versuchen, eine statische Sperre zu erstellen, die den Methodenrumpf schützt:

class dbAccess{ 
    private final static Object o = new Object(); 

    public void getGUID(){ 
     synchronized (o) { 
      counter=/*Access last count from txn_counter table */ 
      // Insert a unique value to txn_counter table based on 
      // the acquired value of counter 
      // Insert new counter value to GUID_log table 
     } 
    } 
} 

class dbAccess{ 
    private final static Object o = new Object(); 

    public void getGUID(){ 
     synchronized (o) { 
      counter=/*Access last count from txn_counter table */ 
      // Insert a unique value to txn_counter table based on 
      // the acquired value of counter 
      // Insert new counter value to GUID_log table 
     } 
    } 
} 

Answer 2

Nur weil es in Java synchronisiert wird, heißt das nicht, dass es in der Datenbank synchronisiert ist. Diese Methode muss in einer Datenbanktransaktion mit aktivierter Lesesperre ausgeführt werden.