java中map的遍历

今天写完代码做find bugs时在map的遍历这方面出现了一下的一个提示：
”inefficient use of keySet iterator instead of entrySet iterator“
大概意思就是效率不高。
经过研究比较，发现以下两种方式遍历map都可以，只是效率不同而已
Map<String, Integer> catalogIds = new HashMap<String, Integer>();
方式一、
           Set<Map.Entry<String, Integer>> set = catalogIds.entrySet();
                 Iterator<Entry<String, Integer>> it = set.iterator();
                    while (it.hasNext())
                    {
                        String catalogId = it.next().getKey();
                        if (catalogIds.get(catalogId) < 2)
                        {
                            it.remove();
                            catalogIds.remove(catalogId);
                        }
                    }

方式二、
         Iterator<String> it = catalogIds.keySet().iterator();
                    while (it.hasNext())
                    {
                        String catalogId = it.next();
                        if (catalogIds.get(catalogId) < 2)
                        {
                            it.remove();
                            catalogIds.remove(catalogId);
                        }
                    }

根据find bugs提示来看，方式一比方式二的效率更高...至于为什么，作为java小菜鸟的我还在研究中...

评论

2 楼 vanceinfo_xuefei 2012-02-28  

按照find bugs来看...应该是第一种效率更高

1 楼 ljc1551008936 2012-02-28  

飞飞，好东西；只是不知道那个效率会更高！？