这两天遇到代码中出现的两次多线程问题

这两天在做两个非常小的项目，就是对现有的代码改改异常处理方式或者显示内容神马的。在修改的过程中，发现了两个线程安全的问题。把这个记下来，算作是对大家工作的提示吧。 

1. 使用static HashMap的问题

代码非常简单，类似是这样子的： 

@Controller
public class MyController {
   private static HashMap<String,String> cachedData = new HashMap<String,String>();
  
   public void processRequest(){
       cachedData.clear();
       cachedData.put("string", "string");
       ........
       mav.addModel("cachedData", cacheData);
   }
}

 这个错误非常明显，因为定义的是Spring的Controller，spring容器的bean默认情况是单例的，这样这个类中的cachedData就会被多个线程同时读和写，cachedData使用的标准的HashMap，不是线程安全的，所以多线程下必然会出现意料不到的结果。 

有人会问，那直接把new HashMap()改成new CocurrentHashMap(),是不是就没有问题了？ 依然有问题，尽管不会出现多线程下的CocurrentModificationException, 但可能会出现数据不一致的情况，举个例子来说，线程A写入string=1, 线程B写入string=2，这时Map中string=2，恰好线程A吧map放到model让spring输入，这是必然会变成string=2. 

修改方法：很简单不要使用static的HashMap，也不要把他作为成员变量，把map作为局部变量使用即可。

 

2. 小心使用线程不安全的类

class A {
    private String a; 
    private String b;
    public void setA(String a ){
       this.a = a;
   }

    public void SetB(String b){
       this.b = b;
    }
    
    public void proces(String c, String d){
       if(b==null){
          b=c;
       }
       ......
    }
}

 

这个非常简单的类，在我们写代码的时候，用这样的模式去写，但是在多个线程下使用同一个类的实例时，就会有问题。 当第一个线程调用process时，b=null， 这时候b被设置为c。 但是当第二个线程调用proces时，b已经变成了c，这时候process就会使用第一个线程设置的变量了。这样第二个线程就无法正常的使用，从而得到错误的process方式了。

 

解决方法： 将b设置成一个final的，意味着只有在对象初始化的时候才能给这个变量赋值，以后其他方法都不能给她赋值 。

 

这两个问题虽然很简单，但是一旦出现问题，带来的后果不堪设想。所以大家在写代码的时候，务必注意，你写的类是否会在多线程下使用，如果多线程下使用会不会出现上述问题。