Синхронизированная карта в службе onStartCommand

Question

Я пытаюсь избежать выполнения повторяющихся задач в службе с помощью синхронизированной карты в onStartMethod, а затем проверять, что ключ еще не сохранен. Однако до сих пор он не работает, он выполняет одно и то же дважды, если я позвоню в начале службы дважды.

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

Инициализация SYNCED_TABLES не является потокобезопасной. В результате вы рискуете иметь несколько объектов, назначенных этому полю, на котором вы блокируете, т. Е. У вас есть несколько блокировок.

Declare SYNCED_TABLES как final поле и инициализировать его прямо там:

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

Это гарантирует, что у вас есть один замок в течение срока службы объекта.