Java 异常安全可以变得不那么难看吗？

Question

给定以下代码：我们需要锁定模型，然后启动一个事务（这可能会引发异常，因此我们必须确保释放锁），然后执行类似获取数据库连接的操作（这可能抛出异常），然后做一些可能抛出需要恢复事务的异常的事情。这是 Java 6，所以我们没有可用的 Java 7 好东西。

SomeClass someMethod() 
throws SomeException {
  acquireWriteLock();
  try {
    startTransaction();
    try {
      DBConnection d = openDBConnection();
      try {
        doStuff(d);
        commitTransaction();
      } finally {
        d.close();
      }
    } catch (SomeException e) {
      handleSomeException(e);
      revertTransaction();
      throw e;
    } catch (Throwable t) {
      revertTransaction();  // Error: method must return a value of SomeClass
    }
  } finally {
    releaseWriteLock();
  }
}

是否可以将其重写为更具可读性和更少冗长？

只是为了好玩：当您看到以下内容时，您会怎么做？

DBConnection d = null;
try {
  acquireWriteLock();
  startTransaction();
  d = openDBConnection();
  try {
    doStuff(d);
    commitTransaction();
  } catch (SomeException e) {
    handleSomeException(e);
    revertTransaction();
  }
} finally {
  d.close();
  releaseWriteLock();
}

Answer 1

如果您切换到 Java 7，则可以大大简化它。否则，您可能应该对此做些什么。

您可以尝试以下想法：

• 将DB连接+事务管理+锁管理重构为通用基类中的通用方法，每个事务类型都有一个子类

• 将DB连接+事务管理+锁管理重构为final类中的通用方法，事务类型有一个接口，每个都有一个实现类。

但是，除非这种特定模式重复很多很多次，否则这种重构可能是个坏主意。 （您将创建一个本地成语，并且读者必须在含义变得清晰之前学习该成语。）

Answer 2

在不升级到 Java 7 的情况下，您可以组合其中的一些，因此您只需要一个 try-catch。

SomeClass someMethod() throws SomeException {
   boolean committed = false;
   DBConnection d = null;
   acquireWriteLock();
   try {
      startTransaction();
      d = openDBConnection();
      doStuff(d);
      commitTransaction();
      committed = true;
   } catch (SomeException e) {
      handleSomeException(e);
      throw e;
   } finally {
     if( d != null ) d.close();
     if( !committed ) revertTransaction();
     releaseWriteLock();
   }
}

诀窍是确保 revertTransaction() 和 DBConnection.close() 不抛出任何东西，但如果你不能这样做，那么你可以将 releaseWriteLock() 放在另一个 finally 子句中。如果你真的想进一步改进，你可以做一些 Stephen C 提到的重构。将获取/释放writeLock、启动/回滚事务、打开/关闭DBConnection等操作放在基类或一些抽象操作类中。