断言单例对象不为空

Question

是否有必要对单例对象进行 Assert.notNull ？

我有一个类：

public class ComponentFactory {
    private static LibraryFrame libraryFrame;

    public static synchronized LibraryFrame getLibraryFrame() {
        if (libraryFrame == null) {
           libraryFrame = new LibraryFrame();
        }
        return libraryFrame;
    }
}

现在需要使用为：

LibraryFrame libraryFrame = ComponentFactory.getLibraryFrame();
Assert.notNull(libraryFrame);
// other part

这里断言类是org.springframework.util.Assert。

如果断言失败，是否可以在失败发生后调用 System.exit(0) ？

Answer 1

Assert 不是必需的，因为 LibraryFrame 实例将始终在此时初始化。

Answer 2

根据该类的文档，它的真正目的是由方法使用来验证其参数。 （也就是说，它的目的是执行先决条件。）您应该对 ComponentFactory 进行单元测试，并且它们应该断言其静态 getLibraryFrame()< /code> 方法不会返回 null，但不值得在运行时断言这一点。

Answer 3

当您使用 JUnit 中的 Assert 类时，其中的任何方法都是 void，但它使用 JUnit 框架来告诉您测试是否通过或失败。

如果这是单元测试的方法，那么您已经完成了测试，但如果这是运行时的方法，则使用 if 条件。

Answer 4

我建议您的单例使用按需初始化模式（即延迟加载单例）。在该模式中，如果实例的构造失败，您将得到一个异常，否则您将获得该对象。

这从两个方面改进了您的解决方案：它没有同步开销，也没有 Assert 开销。如果未创建单例对象，这两种方法都会引发异常 - 在您的代码中，您将从断言中获得 IllegalArgumentException。

使用按需初始化，您的代码将是：

public class ComponentFactory 
{
  private ComponentFactory() 
  {
  }

  private static class LazyHolder 
  {
    public static final LibraryFrame INSTANCE = new LibraryFrame();
  }

  public static LibraryFrame getLibraryFrame() 
  {
    return LazyHolder.INSTANCE;
  }
}

要使用它仍然是：

LibraryFrame libraryFrame = ComponentFactory.getLibraryFrame();

...除非您不再需要 Assert 也不需要 Synchronized。