Javaでこの単純なロジックを実行する代替/より良い方法はありますか？

Question

メソッドを実行するのに時間がかかるmethod1（）などのメソッドがあります。実行中に、method1（）への別の呼び出しがある場合、それは無視する必要があります。私は、大体、この

boolean mFlag = false; 

    void method1() 
    { 
     if(!mFlag) 
     { 
      mFlag=true; 

      // do Stuff 

      mFlag=false; 
     } 
    } 

この作品のようなものを持っています。しかし、これを行うためのよりよい方法があるかどうかは、好ましくは、フラグを含まないかどうか疑問に思っていました。

Answer 1

はい、あなたが本当にあなたの例では、ブールニーズが揮発することを、かなり厳密には正しくないjava.util.concurrent.locksから何かを使用する必要がありますを使用する必要があります。

ReentrantLock lock = new ReentrantLock(); 

    void method1() 
    { 
     if(lock.tryLock()) 
     { 
      try { 
      if (!(lock.getHoldCount() > 1)) { 
       //do Some Stuff 
      } 
      } finally { 
      lock.unlock(); 
      } 
     } 
    } 

あなたのコメントでは、再入力の実行をスキップするように編集されています。残念なことに、組み込みのライブラリではそれを行うには本当に素晴らしい方法はありません。ちょっと変わった用途ですが、組み込みのライブラリを使用する方が良い方法だと思います。

Answer 2

たぶん、あなたはsynchronizedメソッド

http://download.oracle.com/javase/tutorial/essential/concurrency/syncmeth.html

Answer 3

同じスレッドまたは同時にアクセスしている複数のスレッドからの再入力を防止しようとしていますか？

マルチスレッドアクセスを仮定すると、軽いアプローチはjava.util.concurrent.atomicを使用することです。ロックとして「重い」ものは必要ありません（これ以上の要件はありません）。あなたは同じスレッド内で再入国を許可したい場合は、それはロックを使用するのに十分な乱雑取得

private final AtomicBoolean inMethod = new AtomicBoolean(); 

void method1() { 
    if (inMethod.compareAndSet(true, false)) { // Alternatively getAndSet 
     try { 
      // do Stuff 
     } finally { 
      inMethod.set(false); // Need to cover exception case! 
     } 
    } 
} 

：

private final AtomicReference<Thread> inMethod = new AtomicReference<Thread>(); 

void method1() { 
    final Thread current = Thread.currentThread(); 
    final Thread old = inMethod.get(); 
    if (
     old == current || // We already have it. 
     inMethod.compareAndSet(null, current) // Acquired it. 
    ) { 
     try { 
      // do Stuff 
     } finally { 
      inMethod.set(old); // Could optimise for no change. 
     } 
    } 
} 

を使用することができ、同様の方法から無再突入を想定していない

このためにイディオムを実行します。