应用多线程遇到的坑

场景描述：获取产品列表，数据维护方在外部系统，通过外部接口获取到全量数据，自己在对数据进行处理；在访问接口获取数据的时候采用多线程，在子线程中通过JSONObject传递接口参数

public class TestThreadTest implements Runnable {

    JSONObject json;

    @Override
    public void run() {
        System.out.println(Thread.currentThread().getName() + ": " + json.getString("count"));
    }

    public TestThreadTest(JSONObject json) {
        this.json = json;
    }
}


class Test {
    public static void main(String[] args) {
        //前端传过来的参数
        JSONObject params = new JSONObject();
        params.put("param1","param1");
        params.put("param2","param2");
        params.put("param3","param3");
        params.put("count",1);

        for (int i = 0; i < 4; i++) {
            //创建新的json，传到子线程中
            JSONObject jsonObject = new JSONObject(params);
            TestThreadTest testSyn = new TestThreadTest(jsonObject);
            jsonObject.put("count",i);
            Thread thread1 = new Thread(testSyn);
            thread1.start();
        }
    }
}

//结果：
Thread-0: 2
Thread-3: 3
Thread-2: 3
Thread-1: 2

问题出现：参数中有一个递增的变量，但是在子线程中没有变化

在run方法中打印json对象hashcode，发现是一个对象，点开源码看了一下，发现：

//JSONObject对象的构造方法
public JSONObject(Map<String, Object> map) 
{
    this.map = map;
}

new JSONObject()的时候直接将新对象指向了老的对象， 所以一直都是一个对象。传入方式改为：

//创建新的json，传到子线程中
JSONObject jsonObject = new JSONObject();
jsonObject.putAll(params);

正解！

原因分析：

 

public void putAll(Map<? extends String, ? extends Object> m) {
    this.map.putAll(m);
}

putAll是像流一样新增可新增key-value,并返回这个JSONObject，可以继续调用JSONObject其他方法，是新的对象！

欢迎大家批评指正！