sonar相信大家都不陌生了，很多公司都把sonar当做代码质量检测的工具，通过sonar来考察软件的质量，以及程序员的水平。我们在修改代码异味时，也逐渐提高了自己的编码水平。
下面我们来看看在Sonar中常见的一些代码问题吧。
包装对象的比较
案例代码
代码语言：javascript代码运行次数：0运行复制
//两边都为Integer类型if(xxx.getLevel() == xxx.getCurrentLevel()) { ...}sonar检测问题
对于Integer var=?在-128至127之间的赋值，Integer对象是在IntegerCache.cache产生，会复用已有对象，这个区间内的Integer值可以直接使用==进行判断，但是这个区间之外的所有数据，都会在堆上产生，并不会复用已有对象，这是一个大坑，推荐使用equals方法进行判断。
也就是说如果你使用的是包装类型的Integer，虽然在-128到127之间可以用==判断，但是超过这个值，就是两个对象之间的比较，用等号是不成立的。而我们是比较的内容，所以要用equals。
代码语言：javascript代码运行次数：0运行复制
@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)));}运行结果：
代码语言：javascript代码运行次数：0运行复制
a1 == a2: truea3 == a4: falsea3.equals(a4): true手动创建线程池，效果会更好
案例代码
代码语言：javascript代码运行次数：0运行复制
private static ExecutorService executor = Executors.newSingleThreadExecutor();Executors各个方法的弊端：
newFixedThreadPool和newSingleThreadExecutor：主要问题是堆积的请求处理队列可能会耗费非常大的内存，甚至OOM。newCachedThreadPool和newScheduledThreadPool：主要问题是线程数最大数是Integer.MAX_VALUE，可能会创建数量非常多的线程，甚至OOM。
这里建议使用ThreadPoolExecutor的方式创建。
使用示例：
代码小浣熊
代码小浣熊是基于商汤大语言模型的软件智能研发助手，覆盖软件需求分析、架构设计、代码编写、软件测试等环节
51 查看详情 
代码语言：javascript代码运行次数：0运行复制
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();在使用正则表达式时，利用好其预编译功能，可以有效加快正则匹配速度
案列代码
代码语言：javascript代码运行次数：0运行复制
Pattern pattern = Pattern.compile("^[1]d{10}$");不要在方法体内定义：Pattern pattern = Pattern.compile(规则);
正确使用方式：
代码语言：javascript代码运行次数：0运行复制
public class XxxClass { private static Pattern NUMBER_PATTERN = Pattern.compile("^[1]d{10}$"); public static Pattern getNumberPattern() { return NUMBER_PATTERN; } }集合初始化时，指定集合初始值大小
案例代码
代码语言：javascript代码运行次数：0运行复制
Map cache = new HashMap();HashMap使用如下构造方法进行初始化，如果暂时无法确定集合大小，那么指定默认值（16）即可。
代码语言：javascript代码运行次数：0运行复制
Map map = new HashMap(16);Arrays.asList()与Collections.singletonList()
Arrays.asList(strArray)返回值是仍然是一个可变的集合，但是返回值是其内部类，不具有add方法，可以通过set方法进行增加值，默认长度是10Collections.singletonList()返回的同样是不可变的集合，但是这个长度的集合只有1，可以减少内存空间。但是返回的值依然是Collections的内部实现类，同样没有add的方法，调用add，set方法会报错。
卫语句减少if-else嵌套
超过 3 层的 if-else 的逻辑判断代码可以使用卫语句、策略模式、状态模式等来实现。
示例：
代码语言：javascript代码运行次数：0运行复制
public void today() { if (isBusy()) { System.out.println(“change time.”); return; } if (isFree()) { System.out.println(“go to travel.”); return; } return; }SimpleDateFormat线程不安全不要定义为static变量
我们可以使用本地线程处理：
代码语言：javascript代码运行次数：0运行复制
private static final ThreadLocal df = new ThreadLocal() { @ Override protected DateFormat initialValue() { return new SimpleDateFormat("yyyy-MM-dd"); }};以上就是从Sonar上学习代码优化-代码异味案例的详细内容，更多请关注创想鸟其它相关文章！
版权声明：本文内容由互联网用户自发贡献，该文观点仅代表作者本人。本站仅提供信息存储空间服务，不拥有所有权，不承担相关法律责任。
如发现本站有涉嫌抄袭侵权/违法违规的内容， 请发送邮件至 chuangxiangniao@163.com 举报，一经查实，本站将立刻删除。
发布者：程序猿，转转请注明出处：https://www.chuangxiangniao.com/p/299294.html
微信扫一扫 
支付宝扫一扫 
