putIfAbsentで「get/check/put」の使用を置き換えます。

Question

私はCassandraと協力してDatastax Javaドライバを使用しています。準備した文をキャッシュして再利用しようとしています。

private static final Map<String, PreparedStatement> holder = new ConcurrentHashMap<>(); 

    public BoundStatement getStatement(String cql) { 
    Session session = TestUtils.getInstance().getSession(); 
    PreparedStatement ps = holder.get(cql); 
    // no statement is cached, create one and cache it now. 
    if (ps == null) { 
     synchronized (this) { 
     ps = holder.get(cql); 
     if (ps == null) { 
      ps = session.prepare(cql); 
      holder.put(cql, ps); 
     } 
     } 
    } 
    return ps.bind(); 
    } 

私はそれは、スレッドセーフであることを確認する必要がありますのでマイ上記getStatement方法は、複数のスレッドによって呼び出されます。私はJava 7を使用しているので、残念ながらcomputeIfAbsentを使用することはできません。

静的アナライズツールに対してコードを実行したとき、Java 7のコードを書く上でより良い方法があると思うようになりました。

Might be better to replace use of get/check/put with putIfAbsent 

更新：

public BoundStatement getStatement(String cql) { 
    Session session = TestUtils.getInstance().getSession(); 
    PreparedStatement ps = holder.get(cql); 
    // no statement is cached, create one and cache it now. 
    if (ps == null) { 
     ps = session.prepare(cql); 
     PreparedStatement old = holder.putIfAbsent(cql, ps); 
     if (old!=null) 
     ps=old; 
    } 
    return ps.bind(); 
    } 

Answer 1

これは、1つのスレッドが、彼らは同じプリペアドステートメントをしようとしていない場合でも、別のをブロックすることができることを除いて、あなたはそれを持ってあまりにも悪くはありません。

Java 8でcomputeIfAbsentを使用すると、実際にはるかに優れています。 Javaの7では、あなたはこれを行うことができます。

ps = holder.get(cql); 
if (ps == null) { 
    ps = session.prepare(cql); 
    PreparedStatement old = holder.putIfAbsent(cql, ps); 
    if (old!=null) 
    ps=old; 
} 

2つのスレッドが同時に同じものを作ってみる場合は、時折が、キャッシュを使用しないようにだけで同等だ最悪の場合には、不必要なPreparedStatementを行います。

あなたはグアバのライブラリを使用することができればまた、その後、グアバLoadingCacheが正確に何をしたいん：https://google.github.io/guava/releases/16.0/api/docs/com/google/common/cache/CacheBuilder.html