onStartCommandサービス内の同期マップ

Question

onStartMethodで同期マップを使用し、キーがまだ格納されていないことを確認することで、サービス上で重複したタスクを実行しないようにしようとしています。しかし、これまでのところ動作していません。すぐにサービスを2回起動すると同じことが2回実行されます。

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の初期化はスレッドセーフではありません。結果として、このフィールドに複数のオブジェクトが割り当てられている可能性があります。つまり、複数のオブジェクトがロックされている可能性があります。

がfinalフィールドとしてSYNCED_TABLESを宣言し、右がそれを初期化します。

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

これは、あなたのオブジェクトの寿命にわたって単一のロックを持っていることを保証します。