zoukankan      html  css  js  c++  java
  • Avoid using implementation types like 'HashMap'; use the interface instead

    5

    I am getting this warning on Sonar:

    Avoid using implementation types like 'HashMap'; use the interface instead

    What does it mean?

    The class in which i get this warning is as:

    class A {
       private HashMap<String, String> map=new HashMap<String, String>();
    
       //getters and setters
    
    }
    

    Please, I want proper solution to avoid warning on Sonar.

    share  improve this question   
    • 1
      Why are you casting the map to a set? – kennytm Jan 18 '13 at 10:07

    3 Answers

    17
     

    You should always code to an interface. ie. in this case you should declare your field like this:

    private Map<String, String> map= new HashMap<String, String>();
    

    This way anything using the map variable will treat it as type Map rather than HashMap.

    This allows you to swap out the underlying implementation of your map at a later date without having to change any code. You are no longer tied to HashMap

    Read through this question: What does it mean to "program to an interface"?

    Also I am not sure what you were doing casting to a Set there?

    share  improve this answer   
    •  
      I'd agree it's a generally good practice, but I take issue with the "always" in your first sentence. It's perfectly likely that you might end up needing specialized methods of a LinkedList or LinkedHashSet or LinkedHashMap, for instance, in which case you won't be able to resort to the interface. At least not in Java where there's no compostive datatype declaration syntax and you aren't able to say something like "I want something that's the intersection of the Map and List interfaces". If you end up passing around the interface, then you lose context. – haylem Jul 8 '14 at 7:41
    •  
      Beware also that always using the base interface might hide behavioral implementation details. For instance, if you use an immutable List, maybe resorting to the List interface instead of directly an ImmutableList (or another type) is not the best idea, as you convey the wrong impression that your object satisfies the List interface, which it doesn't. So, there definitely are exceptions to this rule. – haylem Jul 8 '14 at 7:42
    •  
      Thats true regarding the use of specialized methods, though these cases are very rare so I wouldnt worry too much abouyt the wording :). I think with regard to your second point I would still use the interface as it leaves you with the option to reuse code if you decide to use other list implementations later. You just need to use correct exception handling. – cowls Jul 8 '14 at 7:50 
  • 相关阅读:
    elasticsearch7.6.2实战(1)-单机快速部署
    greenplum数据库常用操作
    kafka集群搭建及结合springboot使用
    msyql主从库配置
    文本相似性算法实现(二)-分组及分句热度统计
    一点区块链资料
    使用.Net Core实现的一个图形验证码
    python中的collections.namedtuple
    Linux中的定时任务at、crontab
    es3中使用es6/7的字符串扩展
  • 原文地址:https://www.cnblogs.com/deepalley/p/13140865.html
Copyright © 2011-2022 走看看