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("查询异常");
         }
     }

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

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