比如我这一段service中的代码,到底是否应该在整个service方法里面用一个trycatch将所有代码全包裹起来,还是在调用的地方捕捉一下?
public ResultBean<VenueCardResponseEntity> getVenueCardList(VenueCardRequestEntity venueCard) {
ResultBean<VenueCardResponseEntity> bean = new ResultBean<VenueCardResponseEntity>();
String queryVenueCardListResultStr = super.getRPCVenueServiceInstance().queryVenueCardList(JSON.toJSONString(venueCard));
if(StringUtils.isBlank(queryVenueCardListResultStr)){
bean.setCode(ErrorEnum.ERR_500.getIndex());
bean.setMsg("查询失败,服务层未返回结果!");
return bean;
}
PageDto<String> queryVenueCardPageDto;
List<VenueCardResponseEntity> venueCardList;
try {
ResultDto<String> queryVenueCardListResult = JSON.parseObject(queryVenueCardListResultStr,ResultDto.class);
if(!ErrorEnum.SUCCESS_200.getLongIndex().equals(queryVenueCardListResult.getCode())){
bean.setCode(queryVenueCardListResult.getCode());
bean.setMsg(queryVenueCardListResult.getErrMsg());
return bean;
}
queryVenueCardPageDto = JSON.parseObject(queryVenueCardListResult.getObject(),PageDto.class);
if(!(queryVenueCardPageDto.getTotalItems() > Long.parseLong("0"))){
bean.setCode(ErrorEnum.SUCCESS_200.getIndex());
bean.setMsg("未查询到符合条件的数据!");
return bean;
}
venueCardList = JSON.parseArray(queryVenueCardPageDto.getResult2(), VenueCardResponseEntity.class);
} catch (JSONException e) {
bean.setCode(ErrorEnum.ERR_500.getIndex());
bean.setMsg("查询失败,解析服务层返回值异常!");
return bean;
}
bean.setCode(ErrorEnum.SUCCESS_200.getIndex());
bean.setMsg("查询成功");
bean.setSuccess(true);
bean.setRows(venueCardList);
bean.setTotal(queryVenueCardPageDto.getTotalItems());
return bean;
}
我在解析JSON的时候,try catch一下,只捕捉JSONException,是否还应该再包裹一层捕捉exception的trycatch呢?
还是应该在action调用这个service时捕捉一下??
service建议只处理业务异常,捕获持久层或其往下一层的异常,并将其转换为service层的异常,视情况抛出。action调用时捕获所有异常做相应处理。