List集合嵌套Map集合遇到的bug

记最近做项目上遇到的一个Map集合BUG

最近在做传智健康项目，遇到了一个有趣的bug。找bug的过程异常简单，bug解决起来也很简单。既然bug这么简单，为啥我还要特地写一篇文档来记录呢？主要还是想练习一下markdown语法，顺便记录一下学习。

话不多说，进入正题。

场景是这样，后端需要向前端返回一个类型为 List<Map<String,Integer>> 的数据，看起来挺复杂的吧，其实还有更多层嵌套的呢。。。

我不把项目里面的代码搬过来，这样重点就不在bug本身了。所以这里我模拟了原来的场景。

首先我们先创建一个User类，如下所示。

class User {
    private Integer age;
    private Integer money;

    public Integer getAge() {
        return age;
    }

    public Integer getMoney() {
        return money;
    }

    public User(Integer age, Integer money) {
        this.age = age;
        this.money = money;
    }
}

然后，在一个测试类的main方法中写下如下代码

   public static void main(String[] args) {
        List<User> users = Arrays.asList(
                new User(10, 15), new User(15, 20),
                new User(10, 15), new User(20, 25),
                new User(25, 30));
        List<Map<String, Integer>> list = new ArrayList<>();
        Map<String, Integer> map = new HashMap<>();
        for (User user : users) {
            map.put("age", user.getAge());
            map.put("money", user.getMoney());
            list.add(map);
        }
        list.forEach(System.out::println);
    }

我们点击运行。运行结果如下。

{money=30, age=25}
{money=30, age=25}
{money=30, age=25}
{money=30, age=25}
{money=30, age=25}

当时得到这个结果我都懵了，但是也很快反应过来了问题的所在。

问题所在

我们 List 集合里面存储的 map 集合，实际上存储的map对象的地址，当我们一次次的循环时，map对象的地址一直不变，只不过map集合里面存储的数据发生了变化。假设创建map对象的时候他分配的地址是0x11，那么循环结束之后list集合里面存储的就是 {0x11,0x11,0x11,0x11,0x11},而这些地址指向的map集合里面储存的值就是最后一次循环时put进去的值。

现在我们终于搞清了问题所在，那么我们接下来就要去解决它。解决办法很简单，我们只需要将创建map对象的代码移入循环内，这样我们list集合里面存储的map对象地址就不会是同一个了。

        for (User user : users) {
	        Map<String, Integer> map = new HashMap<>();
            map.put("age", user.getAge());
            map.put("money", user.getMoney());
            list.add(map);
        }

我们再次点击运行。

{money=15, age=10}
{money=20, age=15}
{money=15, age=10}
{money=25, age=20}
{money=30, age=25}

现在就得到了我想要的结果了。