添加列表中包含的元素并将它们映射到字符串

Question

另一个名字另一个好

SomeName OtherFine

我想创建一个列表 > ，以便创建一个名称列表和对他们的罚款总额

public List<Map<String, Integer>> calculateTotalFine(){

        List<Map<String, Integer>> myMapList = new ArrayList<Map<String, Integer>>();

        ListIterator<CrimeInfo> crimeIterator = list.listIterator();
        while(crimeIterator.hasNext()){
            String key = crimeIterator.next().getName();
            Integer value = crimeIterator.next().getFine();

            if(myMapList.isEmpty()){
                Map<String, Integer> aMap = new HashMap<String, Integer>();
                aMap.put(key, value);
                myMapList.add(aMap);
            }

            else{
                Iterator<Map<String, Integer>> mapIterator = myMapList.iterator();
                while(mapIterator.hasNext()){
                    if(mapIterator.next().containsKey(key)){  //<-- Line no. 29
                        Integer newFine = mapIterator.next().get(key) + value;
                        mapIterator.remove();

                        Map<String, Integer> nMap = new HashMap<String, Integer>();
                        nMap.put(key, newFine);
                        myMapList.add(nMap);
                    }
                    else{
                        Map<String, Integer> newMap = new HashMap<String, Integer>();
                        newMap.put(key, value);
                        myMapList.add(newMap);
                    }
                }
            }
        }
        return myMapList;
    }

有人能告诉我哪里出了问题吗？

Answer 1

问题是，您正在迭代mymaplist，但在迭代时却修改了它：

myMapList.add(newMap);

我仍然没有完全理解您正在尝试做什么，但从根本上说，在迭代集合时，您不应该添加到集合中。一种常见的方法是创建一个在迭代过程中修改的新集合，然后（如果需要）在此之后对原始集合执行批量修改。

（正如Titus所说，您还在循环中调用next()两次...您需要更加注意如何使用迭代器，并在可能的情况下使用增强型for循环。）