将“get/check/put”的使用替换为 putIfAbsent

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

你拥有它的方式并不算太糟糕，除了一个线程可以阻塞另一个线程，即使他们没有尝试做出相同的准备好的语句。

在 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;
}

如果两个线程同时尝试创建同一个线程，您偶尔会创建一个不必要的 PreparedStatement，但在最坏的情况下，这相当于不使用缓存。

或者，如果您可以使用 guava 库，那么 guava LoadingCache 完全可以满足您的需求：https://google.github.io/guava/releases/16.0/api/docs/com/google/common/cache/CacheBuilder.html