今天廷鹏师弟来的java建议

如下一段获取数据代码的问题：

public Serializable getById(Serializable id) throws BaseBusinessException {
  if (id == null)
   throw new BaseBusinessException("根据给出的Id获得对象时,ID不得为空！");
  try {
   return basicDAO.findById(id);

} catch (BaseDBException e) {
   log.error("根据给出的Id获得对象：" + getObjectName() + "失败");
   e.printStackTrace();
   throw new BaseBusinessException(e.getMessage());
   // }
  }
}

问题：

问题1： e.printStackTrace(); 会造成有一部分日志不在控制范围内，

应该这样写

log.error("根据给出的Id获得对象：" + getObjectName() + "失败",e);

问题2： throw new BaseBusinessException(e.getMessage());

这种方法直接抛出异常到界面，会非常不友好

对于一种get数据的方法，应该将这些异常捕获,用log.erro打印到后台，前端返回null即可

改良1 ：可采用标注型事务

原来我们都是采用前缀控制的声明式事务，这种方法对方法名有所限制，而且不太灵活

可以采用标注型控制，在方法上加上 @Transactional 即可

@Transactional
public Serializable getById(Serializable id) throws BaseBusinessException

改善后的代码

@Transactional
public Serializable getById(Serializable id) throws BaseBusinessException {
  if (id == null)
   throw new BaseBusinessException("根据给出的Id获得对象时,ID不得为空！");
  try {
   return basicDAO.findById(id);

} catch (BaseDBException e) {
   log.error("根据给出的Id获得对象：" + getObjectName() + "失败", e);
   //e.printStackTrace();
   //throw new BaseBusinessException(e.getMessage());
   // }
  }
  return null;
}

巴特西

今天廷鹏师弟来的java建议

最新文章

热门文章