catch (Throwable ex)的问题

abc45628 2015-07-27 10:53:23
在看关于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();
}
}
...全文
244 点赞 收藏 1
写回复
1 条回复
切换为时间正序
当前发帖距今超过3年,不再开放新的回复
发表回复
nickycheng 2015-07-27
我是这样理解的: 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。
回复
相关推荐
发帖
Web 开发
创建于2007-09-28

8.0w+

社区成员

Java Web 开发
申请成为版主
帖子事件
创建了帖子
2015-07-27 10:53
社区公告
暂无公告