在用Coverity(代码检视工具)进行代码检视时,凡是对于HashMap中使用keyset方法的地方全部报warning,说使用了低效的方法,然后建议使用entrySet替代,当时不太清楚entrySet的用法,也没管神马warning,结果发现它在每个这样使用的地方都报了warning,不得不重视了,然后就对这二者进行了一些学习,现将学习心得贴出来。。。
被警告的代码如下:
翻阅JDK,看entrySet()的用法,发现返回的Set对象中的泛型类型为Map.Entry<K,V>,而keySet()返回的Set对象中的泛型类型为K, 这个估计看着区别会比较晕,还是直接看代码爽快。修改后的代码:
使用后发现使用entrySet其实也挺方便的,而且按照Coverity的说法,也更加高效了,至于为何更加高效,还没有花费时间去琢磨,正如我之前的文章中 Integer.valueOf() 和new Integer()二者效率的比较 http://feikiss.iteye.com/admin/blogs/1285283 一样,只要愿意花时间去琢磨,肯定能有新的收获的。不过暂时木有这么多精力,还是先用再探究吧~~
被警告的代码如下:
- public static void setValues2SessionMap(ControllerData cd,Map<String,Object> valueMap)
- {
- Map<String,Object> sessionMap = DSViewHelper.getInstance().getSessionMap();
- for(String valName :valueMap.keySet())
- {
- String key = cd.getDeckId()+"."+valName;
- Object value = valueMap.get(valName);
- //other code here...
- }
- }
翻阅JDK,看entrySet()的用法,发现返回的Set对象中的泛型类型为Map.Entry<K,V>,而keySet()返回的Set对象中的泛型类型为K, 这个估计看着区别会比较晕,还是直接看代码爽快。修改后的代码:
- public static void setValues2SessionMap(ControllerData cd,Map<String,Object> valueMap)
- {
- Map<String,Object> sessionMap = DSViewHelper.getInstance().getSessionMap();
- Set<Map.Entry<String, Object>> keyEntrySet = valueMap.entrySet();
- for(Map.Entry<String, Object> entry:keyEntrySet)
- {
- String key = cd.getDeckId()+"."+entry.getKey();
- Object value = entry.getValue();
- //other code here...
- }
- }
使用后发现使用entrySet其实也挺方便的,而且按照Coverity的说法,也更加高效了,至于为何更加高效,还没有花费时间去琢磨,正如我之前的文章中 Integer.valueOf() 和new Integer()二者效率的比较 http://feikiss.iteye.com/admin/blogs/1285283 一样,只要愿意花时间去琢磨,肯定能有新的收获的。不过暂时木有这么多精力,还是先用再探究吧~~