Mappage synchronisé dans le service onStartCommand

Question

J'essaie d'éviter d'exécuter des tâches en double sur un Service en utilisant une Map synchronisée dans onStartMethod, puis en vérifiant qu'une clé n'est pas déjà stockée. Cependant, jusqu'ici ne fonctionne pas, il exécute la même chose deux fois si j'appelle démarrer le service deux fois assez tôt.

public void onCreate() { 
      SYNCED_TABLES = Collections.synchronizedMap(new Hashtable<>()); 
} 
public int onStartCommand(Intent intent, int flags, int startId) { 
      synchronized (SYNCED_TABLES){ 
       if(!SYNCED_TABLES.containsKey(intent.getStringExtra(KEY))){ 
        SYNCED_TABLES.put(intent.getStringExtra(KEY), true); 
        /* Do stuff on a Handler thread */ 
       } 
       else{ 
        Log.d(TAG, "Tried to execute the same task twice " + intent.getStringExtra(KEY)); 
       } 
      } 

} 

Answer 1

L'initialisation de SYNCED_TABLES n'est pas adaptée aux threads. En conséquence, vous risquez d'avoir plusieurs objets assignés à ce champ sur lequel vous verrouillez, c'est-à-dire que vous avez plusieurs verrous.

Declare SYNCED_TABLES comme un champ final et l'initialiser là:

public class Foo { 
    final SYNCED_TABLES = Collections.synchronizedMap(new Hashtable<>()); 
} 

Cela garantit que vous avez un seul verrou sur la durée de vie de votre objet.