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

一时失言乱红尘 2022-11-14 15:53 285阅读 0赞

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

![图片][91fa1625ac2f805caea8d256fd624f00.png]

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

## 包装对象的比较 ##

案例代码

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

sonar检测问题

> “
> 
> 所有的包装类对象之间值的比较，全部使用equals方法比较
> 
> ”

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

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

@Test
    public 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: true
    a3 == a4: false
    a3.equals(a4): true

## 手动创建线程池，效果会更好 ##

案例代码

private static ExecutorService executor = Executors.newSingleThreadExecutor();

Executors各个方法的弊端：

*  `newFixedThreadPool`和`newSingleThreadExecutor`：主要问题是堆积的请求处理队列可能会耗费非常大的内存，甚至OOM。
 *  `newCachedThreadPool`和`newScheduledThreadPool`：主要问题是线程数最大数是`Integer.MAX_VALUE`，可能会创建数量非常多的线程，甚至OOM。

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

使用示例：

ExecutorService pool = new ThreadPoolExecutor(5, 200,
                0L, TimeUnit.MILLISECONDS,
                new LinkedBlockingQueue <Runnable>(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<String, Object> cache = new HashMap<>();

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

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

## Arrays.asList()与Collections.singletonList() ##

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

## 卫语句减少if-else嵌套 ##

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

示例：

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

## SimpleDateFormat线程不安全不要定义为static变量 ##

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

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

# **往期推荐** #

*  [我写出这样干净的代码，老板直夸我][Link 1]
 *  [云南丽江旅游攻略][Link 2]
 *  [使用ThreadLocal怕内存泄漏？][ThreadLocal]
 *  [Java进阶之路思维导图][Java]
 *  [程序员必看书籍推荐][Link 3]
 *  [3万字的Java后端面试总结（附PDF）][3_Java_PDF]

扫码二维码，获取更多精彩。或微信搜**Lvshen\_9**，可后台回复获取资料

回复"java" 获取java电子书；
    
    回复"python"获取python电子书；
    
    回复"算法"获取算法电子书；
    
    回复"大数据"获取大数据电子书；
    
    回复"spring"获取SpringBoot的学习视频。
    
    回复"面试"获取一线大厂面试资料
    
    回复"进阶之路"获取Java进阶之路的思维导图
    
    回复"手册"获取阿里巴巴Java开发手册(嵩山终极版)
    
    回复"总结"获取Java后端面试经验总结PDF版
    
    回复"Redis"获取Redis命令手册，和Redis专项面试习题（PDF）
    
    回复"并发导图"获取Java并发编程思维导图（xmind终极版）

另：点击【**我的福利**】有更多惊喜哦。

![format_png][]

[91fa1625ac2f805caea8d256fd624f00.png]: /images/20221022/f9f65dcf8ed44f0596625736f90db665.png
[Link 1]: http://mp.weixin.qq.com/s?__biz=Mzg4NDMyNTIwMw==&mid=2247487966&idx=1&sn=7810bd4f62f2b92bec831db2e1f35b97&chksm=cfb8bfe2f8cf36f4295a3560fc8fde094bf73e8e187ff589c6559ae7ed38584d8c67c1c20330&scene=21#wechat_redirect
[Link 2]: http://mp.weixin.qq.com/s?__biz=Mzg4NDMyNTIwMw==&mid=2247487391&idx=1&sn=5847806bce7a105d6503eab52ea725be&chksm=cfb8a1a3f8cf28b517b36ded4dc1ec97334b0a685e5deb6f75d9a668c96d3fbc0c058e5432d8&scene=21#wechat_redirect
[ThreadLocal]: http://mp.weixin.qq.com/s?__biz=Mzg4NDMyNTIwMw==&mid=2247486830&idx=1&sn=3aa8095371b2e04dc0c954443a890ef4&chksm=cfb8a352f8cf2a444a4c54d19a5e74b03879c03e79209a1f7688fd9edd81edb35b43fea9578d&scene=21#wechat_redirect
[Java]: http://mp.weixin.qq.com/s?__biz=Mzg4NDMyNTIwMw==&mid=2247486701&idx=2&sn=8b6c8da8802d4fceb4ad06c229af5391&chksm=cfb8a2d1f8cf2bc71415098ecc807dfcb6bcfdde399ec92f79ead84f0f4352a5fdb7cf766593&scene=21#wechat_redirect
[Link 3]: http://mp.weixin.qq.com/s?__biz=Mzg4NDMyNTIwMw==&mid=2247486232&idx=1&sn=ac1ba6a9f543f7115098f820a7b41165&chksm=cfb8a524f8cf2c326861c7a0783a6ada2ba7a3e76c633cb9632af99f32de64a984fef5ba8a75&scene=21#wechat_redirect
[3_Java_PDF]: http://mp.weixin.qq.com/s?__biz=Mzg4NDMyNTIwMw==&mid=2247488123&idx=1&sn=9567232df10afbfede9688e4762c988c&chksm=cfb8bc47f8cf35512bced50381acb94713b51bb8691e895ba880a2b7863d7c882f4b54aec190&scene=21#wechat_redirect
[format_png]: /images/20221022/784135991f8545b3b6a1a54851d89f76.png