通过切面捕捉方法异常，并添加错误码返回前端的一点思考

这几天看到某个源码，返回的是Object类型的对象，再通过切面来对Object返回类型添加正确与否的包装类型返回json，这种类型的返回值有个巨大的问题是，无法返回自定义的错误码，虽然节省了工作，导致错误码的添加不能在代码里进行处理，只能通过抛出异常来解决，切面捕捉到异常以后，返回

    @Around("within(com.*.controller..*) && @annotation(rh))")
    public Object handleReq(ProceedingJoinPoint jp, ReqHandle rh) throws Throwable {
        HttpServletRequest request = ((ServletRequestAttributes) RequestContextHolder.getRequestAttributes())
                .getRequest();
        if (this.isAjax(request)) {
            return ajaxHandle(jp);
        } else {
            return viewHandle(jp);
        }
    }

上面是一个切面类，对拥有自定义注解@ReqHandle 的控制器类的方法进行拦截，如果是ajax请求，就进入ajaxHandle方法处理，代码如下：

    private Object ajaxHandle(ProceedingJoinPoint jp) throws Throwable {
        // 记录请求参数，实际请求为post
        HttpServletRequest request = ((ServletRequestAttributes) RequestContextHolder.getRequestAttributes())
                .getRequest();
        String path = request.getRequestURI();
        String arg = QueryUtil.getParams(request);
        logger.debug("AJAX_START path:" + path + "\nquery:" + arg);
        // 全局异常处理
        Object data = null;
        try {
            data = jp.proceed();
        } catch (Exception e) {
            logger.error("AJAX_EXCEPTION path:" + path + "\nquery:" + arg, e);
            return getExcJson(e);
        }
        // 返回结果处理
        logger.debug("AJAX_END path:" + path + "\nquery:" + arg);
        System.out.println(getSuccessJson(data));
        return getSuccessJson(data);
    }

被切方法因为是返回的Object类型。直接去掉@ReqHandle 注解后可以直接调用fastjson的FastJsonHttpMessageConverter类的writeInternal方法。但是此前如果发生了异常，并不能返回可理解的错误码与错误原因。

添加注解后，被切的方法进入异常捕捉方法。异常捕捉方法代码如下：

    /**
     * ajax接口异常处理
     */
    private Object getExcJson(Exception e) {
        Map<String, Object> result_map = new HashMap<String, Object>();
        result_map.put("success", false);
        if (e instanceof BusiException) {
            result_map.put("code", 15);
        } else if (e instanceof ParamException) {
            result_map.put("code", 10);
        } else if (e instanceof LoginException) {
            result_map.put("code", -1);
            result_map.put("msg", ((LoginException) e).getMsg());
        } else if (e instanceof SendMessageException) {
            if (((SendMessageException) e).getErrorCode() != -1)
                result_map.put("code", ((SendMessageException) e).getErrorCode());
            else
                result_map.put("code", 3);
            result_map.put("msg", ((SendMessageException) e).getMsg());
        } else {
            result_map.put("code", 3);
        }
        if (!result_map.containsKey("msg"))
            result_map.put("msg", "发现未知错误，请联系管理员！");
        return JSON.toJSONString(result_map, SerializerFeature.WriteMapNullValue);
    }

此时在我们的代码里面只要抛出SendMessageException异常即可把异常信息捕获到，返回给前端的也是正常的json格式，并且有简单的错误码以及错误原因。

如果是正确的返回到切面，即被@ReqHandle注解修饰的类，此时返回正确的json。

    private Object getSuccessJson(Object data) {
        Map<String, Object> result_map = new HashMap<String, Object>();
        result_map.put("success", true);
        result_map.put("data", data);
        System.out.println(result_map);
        return JSON.toJSONString(result_map, SerializerFeature.WriteMapNullValue);
    }

但是此时发现需要手动调用JSON.toJSONString方法，没有自动调用writeInternal方法。这是一个比较奇怪的地方，至于为什么还有待继续深究。但是如果return JSON.toJSONString(result_map, SerializerFeature.WriteMapNullValue); 更改为这return result_map那么writeInternal方法依旧会调用，私自猜测可能是因为fastjson在转换对象前就判断了是不是json字符串，是的话直接返回了。

 这种切面形式与异常抛出的结合来实现错误码的返回有一定的可取之处。我个人偏向于在Controller的方法里面指定具体的错误原因和错误码，下次分析此源码