ThreadLoacl的反思

在我的随笔 spring mvc：注解@ModelAttribue妙用  中使用ThreadLocal来简化spring mvc控制层controller中的ModelMap，Response、Json数据封装结构的声明。

这种方式很明显地可以减少方法参数以及变量数量，于是乎，我在每个新系统上都将上述一套搬了过去！

经过一段时间，（经人指点）发现会有以下问题：

1. 因为没有在每个线程结束后清除变量，在线程池（比如：tomcat线程池）的作用下就会出现脏数据；
2. Json数据封装结构其实每次只在返回客户端的时候使用，没必要使用ThreadLocal，在Json数据封装结构中定义一个静态方法即可

定义如下（类名称：JsonData）：

    /**
     * 静态工厂方法
     * 
     * @return
     */
    public static JsonData newInstance() {
        return new JsonData();
    }

/**
     * 设置成功json
     * 
     * @param message
     * @return
     */
    public static JsonData successJson(String message) {
        return successJson(null, message);
    }

    /**
     * 设置成功的json
     * 
     * @param data
     * @return
     */
    public static JsonData successJson(Object data) {
        JsonData jsonData = newInstance();
        jsonData.setSuccessJson(data);
        return jsonData;
    }

    /**
     * 设置成功的json
     *
     * @param data
     */
    public void setSuccessJson(Object data) {
        setSuccessJson(data, null);
    }

使用如下：

 @RequestMapping("/xx/list")
    @ResponseBody
    public JsonData list(@ModelAttribute final DataQuery query) {
        try {
            //逻辑
            return JsonData.successJson(queryResult);
        } catch (Exception e) {
            LOGGER.error("查询异常", e);
            return JsonData.exceptionJson("查询异常");
        }
    }

这样不仅可以达到减少代码的目的，另外也去除了线程脏数据问题，也更加简洁明了！

事实上，这是一个很小的问题，却暴露了一个问题：一旦掌握一个新的技能，在未详细论证的情况下就开始使用，是极其危险的！