今天廷鹏师弟来的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;
 }