try-catch之后就万事大吉不会抛出异常给调用者了吗

Posted 2022-03-10 石头StoneWang

背景

作为Java程序员，spring一定很熟悉了，对于@Transactional 那肯定是再熟悉不过了。下面会展示一个大坑，可能90%的人都不知道这里可能会出问题，见伪代码

@Configuration
@EnableScheduling
public class A 
	@Autowired
	private B b;
	
	// 每天夜里2点
	@Scheduled(cron = "0 0 2 * * ?")
	public void task() 
		b.doJob();
	



@Service
public class B 
	@Transational
	public void doJob() 
		try 
			// 其他代码，有可能抛出异常
		 catch(Throwable e) 
			// 不抛出
		
	

你是不是觉得 B#doJob() 肯定将异常抛给调用者 A#task() 。我以前也是这么认为的，觉得只要自己try-catch住了肯定不可能让调用者出现异常，但是我发现是错的

为什么

为什么try-catch了还可能往外抛出异常？ 原因是task()并不是直接调用doJob()，因为有 @Transactional 注解，被代理了，在这个代理这里可能出现异常。

举个例子，肯定遇到这事情吧？数据库连接池对象长时间没被操作被数据库收回但是Java的连接池以为这个连接对象还是可用的，结果一用就抛出异常。这就是我们为什么有个 testWhenIdle 的数据库参数，就是时不时就激活一下免得连接对象被收回。

要怎么避免这种事情？

建议可以在 A类 的task() 方法里进行try-catch，这样肯定不可能出现抛出异常没有记录到，在task() 里可以在catch 里头进行重试，或者记录日志。