java - 单例模式中如何处理异常？

Question

我试图了解如何在这里处理异常处理。我有下面的 Singleton 类，它连接到 cassandra，在第一次调用时初始化所有元数据，然后启动一个定期后台线程，每 15 分钟更新一次所有元数据。

public class CassUtils {
  private final ScheduledExecutorService scheduler = Executors.newSingleThreadScheduledExecutor();
  private final CassSession cassSession;

  private static class Holder {
    private static final CassUtils INSTANCE = new CassUtils();
  }

  public static CassUtils getInstance() {
    return Holder.INSTANCE;
  }

  private CassUtils() {
    CassConfig config = Utils.getConfig();
    try {
      this.cassSession = new CassSession(config);
      initializeMetadata(); // initializes metadata on the very first call
    } catch (Exception ex) {
      // log error
      throw new IllegalStateException("cannot initialize metadata: " + ex.getMessage());
    }
  }

  private void startScheduleTask() {
    scheduler.scheduleAtFixedRate(new Runnable() {
      public void run() {
        try {
          List<ProcessMetadata> processMetadata = getProcessMeta();
          List<ClientMetadata> clientMetadata = getClientMeta();
          List<ProcMetadata> procMetadata = getProcMeta();
          if (!processMetadata.isEmpty() && !clientMetadata.isEmpty())
            MetadataManager.setMetadata(processMetadata, clientMetadata, procMetadata);
        } catch (Exception ex) {
            // log error
        }
      }
    }, 30, 15, TimeUnit.MINUTES);
  }

  private void initializeMetadata() {
    List<ProcessMetadata> processMetadata = getProcessMeta(true);
    List<ClientMetadata> clientMetadata = getClientMeta();
    List<ProcMetadata> procMetadata = getProcMeta();
    if (processMetadata.isEmpty() || clientMetadata.isEmpty() || procMetadata.isEmpty()) {
      throw new IllegalStateException(); // is it ok to throw exception without any message here?
    }
    MetadataManager.setMetadata(processMetadata, clientMetadata, procMetadata);
    startScheduleTask();
  }

如果有任何问题，我需要通知调用此单例的调用者。

• 现在，无论出于何种原因，如果 CassSession 抛出异常，因为该类连接到 cassandra，并且由于某种原因它无法连接到 cassandra(大多数情况下它永远不会发生)，那么我会捕获CassUtils 类中的 catch block 中出现异常，并向用户抛出 IllegalStateException 并包含一条消息。
• 另外，如果我们能够通过 CassSession 连接到 cassandra(我们将始终这样做)，但在 initializeMetadata 方法中，我们的所有元数据都是空的，那么我不想继续，所以我抛出 IllegalStateException ，其中没有任何消息。这可以吗？因为最终它会被 CassUtils 类的 catch block 捕获，并向调用用户抛出异常。

如何处理单例情况下的异常处理，以便我们可以通知调用此单例的调用者。 IllegalStateException 是在这里抛出的正确异常吗？上述情况将在初始化时的第一次调用期间发生。有没有更好的办法？

Answer 1

您正在使用静态/类初始化来创建ClassUtils实例。作为一般规则:

• 必须捕获类初始化中抛出的所有已检查异常。必须。编译器坚持认为。
• 任何从类静态初始化中传播出来的未经检查的异常最终都会导致抛出错误。即使在您能够捕获此错误的上下文中，您也会遇到初始化失败的类。 JVM 永远不会重试初始化，并且该类...以及依赖于它的任何其他类将无法使用。

因此，如果您要处理此问题，则需要在静态初始化期间捕获并(适本地)处理异常。

下一个问题是，如果您想向 CassUtils 单例的用户报告失败，则 getInstance() 必须返回 CassUtils< 的实例 (或者可能是一个子类)能够:

1. 记住异常或报告失败的任何内容，并且
2. 将其重新抛出给 CassUtils API 上实例方法的调用者。

这是可能的，但很复杂。

在我看来，这是不可取的。我认为你最好不要尝试恢复。将未能初始化 Casandra session 视为 fatal error 。如果您做不到这一点，那么 getInstance() 方法应该是抛出/重新抛出“我们坏了”的异常的方法。至少，这避免了任何 CassUtils 方法的潜在调用者必须处理“我们坏了”异常。

<小时/>

有人向你提过单例是个坏主意吗？考虑改用依赖注入(inject)。