5

同期の詳細を抽象化しながら、メソッドを同時に実行するためのインターフェイスを作成しています（必要に応じて分散実装と交換するため）。異なる参照の文字列が渡された場合でも、文字列をマップに格納して1つの参照が使用されるようにすることで、文字列をミューテックスとして使用できる単一のjvm実装を作成しました。並行性は正常に機能しているようですが、テストが参照カウントが決して減少していないことを示しているのを見て驚いた。WeakValues（）を使用すればメモリリークを防ぐのに十分だと思いましたが、そうではないようです。このリークの原因を誰かが指摘できますか？

public class SynchronousMethodExecutorSynchronizedImpl implements ISynchronousMethodExecutor {

// mutex map to provide string references
final Map<String, String> mutexMap = new MapMaker()
    .weakValues()
    .makeComputingMap(
        new Function<String, String>() {
        @Override
        public String apply(String id) {
            return id;
        }
    });

@Override
public Object doSynchronousMethod(String domain, String id, ISynchronousMethod synchronousMethod) {
    synchronized(mutexMap.get(domain + "." + id))
    {
        return synchronousMethod.execute();
    } 
}

}

最後のアサーションで失敗したテストは次のとおりです。

public class SynchronousMethodExecutorSynchronizedImplTest extends TestCase {
int counter;
SynchronousMethodExecutorSynchronizedImpl methodExecutor;

@Override
public void before() throws Exception {
    super.before();

    methodExecutor = new SynchronousMethodExecutorSynchronizedImpl();
}

@Test
public void concurrentExecute() throws InterruptedException {
    assertEquals(0, counter);

    for(int i=0; i<1000; i++)
        getConcurrentExecutorThread().start();

    // wait for threads to complete
    Thread.sleep(1000);

    assertEquals(1, methodExecutor.mutexMap.size());

    try
    { 
        final List<long[]> infiniteList = new LinkedList<long[]>(); 

       for(long i = Long.MIN_VALUE; i < Long.MAX_VALUE; i++)
            infiniteList.add(new long[102400]); 

        fail("An OutOfMemoryError should be thrown");
    } 
    catch(OutOfMemoryError e)
    { 

    }

    assertEquals(2000, counter);
    assertEquals(0, methodExecutor.mutexMap.size());
}

// synchronous method
private ISynchronousMethod method = new ISynchronousMethod() {
    @Override
    public Object execute() {
        counter++;  
        return null;
    }
};

/**
 * Executes a line of code.
 * 
 * @return Thread
 */
private Thread getConcurrentExecutorThread() {
    return new Thread() {
        @Override
        public void run() {
            methodExecutor.doSynchronousMethod("TEST", "1", method);
            try
            {
                Thread.sleep(500);
            }
            catch (InterruptedException e)
            {

            }
            methodExecutor.doSynchronousMethod("TEST", new String("1"), method);        
        }

    };
}

}

この最後のアサーションは、テストを破るものです。assertEquals（0、methodExecutor.mutexMap.size（））;

4

2 に答える 2

5

Stringキーと値の両方とまったく同じオブジェクトを保存しています。キーはオブジェクトへの強参照であり、オブジェクトへの強参照が存在する限り、オブジェクトへの弱参照は無意味です。到達可能性が低い（ここで）の定義は、次のように述べています。

オブジェクトが強くもソフトにも到達できない場合、オブジェクトは弱く到達可能ですが、弱い参照をトラバースすることで到達できます。

ちなみに、これを修正しても、最後はマップが常に空になるとは思えません。おそらく空に近いでしょうが、それについて言えることはそれだけだと思います。

于 2011-06-21T01:22:26.597 に答える
1

弱参照は、JVMが絶対により多くのメモリを必要とする場合にのみ収集されます。

于 2011-06-21T01:08:58.280 に答える