时间戳

首页 > 解决方案 > 如何去除代码异味：getSessionFactory，仅在必要时创建 SessionFactory

java - 如何去除代码异味：getSessionFactory，仅在必要时创建 SessionFactory

问题描述

我使用休眠来处理与数据库的连接。我有几个连接到不同模式的会话工厂。

在启动时构建所有 SessionFactory 至少需要 60 秒。所以我只在必要时建造它们：

public class HibernateUtil {
    private static SessionFactory factory_db1;
    private static SessionFactory factory_db2;
    //...

    public enum DbSchema {
      db1, db2 //...
    }        

    private createSessionFactory(Configuration conf){
    //...
    }

    public static SessionFactory getFactory(DbSchema dbSchema) {

      try {
        switch (dbSchema) {
            case db1:
                if (factory_db1== null){
                    Configuration conf = new Configuration().configure(HIBERNATE_CFG_DB1);
                    factory_db1= createSessionFactory(conf);
                }
                return factory_db1;
            case db2:
                if (factory_db2 == null){
                    Configuration conf = new Configuration().configure(HIBERNATE_CFGXML_DB2);
                    factory_ooarchive = createSessionFactory(conf);
                }
                return factory_ooarchive;
            //... more factories created
            default:
                assert false : "Switch default should not be reachable.";
                return null;
        }
      } catch (Throwable ex) {
        log.error("Failed to initialize SessionFactory." + ex);
        throw new ExceptionInInitializerError(ex);
    }
}

现在当我访问这个工厂时：

Session session = HibernateUtil.getFactory(db1).openSession();
// **Compiler warning: method invocation may produce java.lang.nullpointerexception**

只能通过 getFactory() 方法获取工厂，因此我认为 NPE 永远不可能。我知道问题是工厂实例变量的静态关键字，并且构造函数中没有初始化。我不想要这种“永远在线”的初始化！它应该只在需要时至少初始化一次。

我阅读了一些设计模式和代码质量书籍，但我很难实现我学到的东西。我想我创造了一种代码味道。我该如何解决这个设计？请解释我的错误以及为什么我的选择有问题。

标签： javahibernateoopdesign-patternsfactory

解决方案

我不确定编译器警告（可能由 IDE 而不是 IDE 发出javac）与 . 的static修饰符有关getFactory()。

实际上getFactory()实现声明了一个switch语句，它有一个default返回的案例null：

default:
    assert false : "Switch default should not be reachable.";
    return null;

因此，如果传递的参数不允许在前面的一种情况下输入， 那么getFactory()确实可能会返回。null

但我认为主要问题getFactory()是它缺乏线程安全性。实际上，如果多个线程同时访问它，您可以创建多个会话并可能生成不一致的状态。
作为替代方案，要按需创建会话，您可以使用特定风格的单例模式：按需初始化持有者习惯用法：

在软件工程中，按需初始化持有者（设计模式）习惯用法是延迟加载的单例。在所有 Java 版本中，该习惯用法都可以实现安全、高度并发的延迟初始化，并具有良好的性能。

推荐阅读