从Sonar上学习代码优化-代码异味案例

sonar相信大家都不陌生了，很多公司都把sonar当做代码质量检测的工具，通过sonar来考察软件的质量，以及程序员的水平。我们在修改代码异味时，也逐渐提高了自己的编码水平。

下面我们来看看在Sonar中常见的一些代码问题吧。

案例代码

//两边都为Integer类型if(xxx.getLevel() == xxx.getCurrentLevel()) {    ...}

sonar检测问题

对于Integer var=?在-128至127之间的赋值，Integer对象是在IntegerCache.cache产生，会复用已有对象，这个区间内的Integer值可以直接使用==进行判断，但是这个区间之外的所有数据，都会在堆上产生，并不会复用已有对象，这是一个大坑，推荐使用equals方法进行判断。

也就是说如果你使用的是包装类型的Integer，虽然在-128到127之间可以用==判断，但是超过这个值，就是两个对象之间的比较，用等号是不成立的。而我们是比较的内容，所以要用equals。

@Testpublic void testInteger() {    Integer a1 = 120;    Integer a2 = 120;    System.out.println("a1 == a2: " + (a1 == a2));    Integer a3 = 200;    Integer a4 = 200;    System.out.println("a3 == a4: " + (a3 == a4));    System.out.println("a3.equals(a4): " + (a3.equals(a4)));}

运行结果：

a1 == a2: truea3 == a4: falsea3.equals(a4): true

案例代码

private static ExecutorService executor = Executors.newSingleThreadExecutor();

Executors各个方法的弊端：

这里建议使用ThreadPoolExecutor的方式创建。

使用示例：

晓象AI资讯阅读神器

晓象-AI时代的资讯阅读神器

下载

ExecutorService pool = new ThreadPoolExecutor(5, 200,            0L, TimeUnit.MILLISECONDS,            new LinkedBlockingQueue (1024), namedThreadFactory, new ThreadPoolExecutor.AbortPolicy());        pool.execute(()-> System.out.println(Thread.currentThread().getName()));        pool.shutdown();

案列代码

 Pattern pattern = Pattern.compile("^[1]\\d{10}$");

不要在方法体内定义：Pattern pattern = Pattern.compile(规则);

正确使用方式：

public class XxxClass {          private static Pattern NUMBER_PATTERN = Pattern.compile("^[1]\\d{10}$");        public static Pattern getNumberPattern() {             return NUMBER_PATTERN;        }    }

案例代码

Map cache = new HashMap<>();

HashMap使用如下构造方法进行初始化，如果暂时无法确定集合大小，那么指定默认值（16）即可。

Map map = new HashMap(16);

Arrays.asList(strArray)返回值是仍然是一个可变的集合，但是返回值是其内部类，不具有add方法，可以通过set方法进行增加值，默认长度是10Collections.singletonList()返回的同样是不可变的集合，但是这个长度的集合只有1，可以减少内存空间。但是返回的值依然是Collections的内部实现类，同样没有add的方法，调用add，set方法会报错。

超过 3 层的 if-else 的逻辑判断代码可以使用卫语句、策略模式、状态模式等来实现。

示例：

    public void today() {      if (isBusy()) {       System.out.println(“change time.”);       return;       }      if (isFree()) {       System.out.println(“go to travel.”);       return;      }      return;    }

我们可以使用本地线程处理：

private static final ThreadLocal df = new ThreadLocal() {     @ Override     protected DateFormat initialValue() {      return new SimpleDateFormat("yyyy-MM-dd");     }};