catch (Throwable ex)的问题

[abc45628]

在看关于hibernate的书的时候有上面这段代码，然后用sonarqube分析过后对于静态代码块部分（大约15-20行的位置）报了 Catch Exception instead of Throwable.
上网查了一下ExceptionInInitializerError是一个error，按照道理出现这个问题jvm应该就是死了，那么还有必要catch（Throwable）以及throw new ExceptionInInitializerError(ex);吗？

package util;

import org.hibernate.HibernateException;
import org.hibernate.Session;
import org.hibernate.SessionFactory;
import org.hibernate.cfg.Configuration;

public class HibernateUtil {

	private static SessionFactory sessionFactory;
	private static Configuration configuration = new Configuration();
	// 创建线程局部变量threadLocal，用来保存Hibernate的Session
	private static final ThreadLocal<Session> threadLocal = new ThreadLocal<Session>();
	// 使用静态代码块初始化Hibernate
	static {
		try {
			Configuration cfg = new Configuration().configure(); // 读取配置文件hibernate.cfg.xml
			sessionFactory = cfg.buildSessionFactory(); // 创建SessionFactory
		} catch (Throwable ex) {
			throw new ExceptionInInitializerError(ex);
		}
	}

	// 获得SessionFactory实例
	public static SessionFactory getSessionFactory() {
		return sessionFactory;
	}

	// 获得ThreadLocal 对象管理的Session实例.
	public static Session getSession() throws HibernateException {
		Session session = (Session) threadLocal.get();
		if (session == null || !session.isOpen()) {
			if (sessionFactory == null) {

				rebuildSessionFactory();
			}
			// 通过SessionFactory对象创建Session对象
			session = (sessionFactory != null) ? sessionFactory.openSession() : null;
			// 将新打开的Session实例保存到线程局部变量threadLocal中
			threadLocal.set(session);
		}
		return session;
	}

	// 关闭Session实例
	public static void closeSession() throws HibernateException {
		// 从线程局部变量threadLocal中获取之前存入的Session实例
		Session session = (Session) threadLocal.get();
		threadLocal.set(null);
		if (session != null) {
			session.close();
		}
	}

	// 重建SessionFactory
	public static void rebuildSessionFactory() {
		try {
			configuration.configure("/hibernate.cfg.xml"); // 读取配置文件hibernate.cfg.xml
			sessionFactory = configuration.buildSessionFactory(); // 创建SessionFactory
		} catch (Exception e) {
			System.err.println("Error Creating SessionFactory ");
			e.printStackTrace();
		}
	}

	// 关闭缓存和连接池
	public static void shutdown() {
		getSessionFactory().close();
	}
}

[nickycheng]

我是这样理解的： sonar的这个检查是基于java 错误捕获的最佳实践，对于有些jvm层面的错误，比如内存溢出，outofmemory的错误，你catch实际上没有意义，已经不是业务层面的错误了，如果碰到这样的错误，你没法处理，也不应该处理。 Java Lanuage Spec 7 中也提到：Error继承自Throwable而不是继承自Exception，是为了方便程序可以使用 “catch (Exception)“来捕捉异常而不会把Error也捕捉在内，因为Exception发生后可以进行一些恢复工作的，但是Error发生后一般是不可恢复的。 所以一般都是建议最多catch到Exception，而不是catch Throwable。 关于你提的那个代码上的 catch (Throwable ex) { throw new ExceptionInInitializerError(ex); } 我认为没有太大意义。和不catch的效果是一样的，无非就是打印堆栈的时候会有个ExceptionInInitializerError的堆栈信息，方便你定位问题出在哪里。 实际的代码加不加出现异常的情况都会导致这个类的初始化失败，再次调用里面的方法会出现ClassNotFound Error。

• 打赏
• 举报

回复 赞