原文路径：https://blog.csdn.net/luckystar689/article/details/53968102

----------------------------------------------------------------------------------------

之前，在看《笑傲江湖》的时候，郭德纲在给一对夫妻的表演做评审的时候，说了一句话让我印象特别深刻，原话 我记不清了，大概的意思是：你们的表演很好，但是废话很多，我讲相声的时候，如果去掉这句话，不会影响这个节目 的效果，OK，那就去掉这句话，一句废话也不能多。

 

   这让我想到了，我们最近在做的工作：修改由sonar测试出来的不规范代码。也是同样的道理，声明一个永远都不会用的变量，定义一个不会有其他方法调用的方法，无用的import等等~  去掉它，不会影响你的系统功能实现，加上它会占用空间，同时还会影响代码阅读的效果。

 

   我们有ITOO的代码规范手册，例如如何命名，注释应该怎么写之类的等的规范，但这只是很小的，很基础的一部分。但是像sonar这样检测出来的规范，例如“.equals()”方法，要测试的value值要放在"."的左侧；判断list是否为空，用isEmpty，而不是直接用".size()"方法。我们之前写的这种方法都没有错，只不过对比规范的代码来说，我们写的代码安全性低，有潜在的安全隐患问题，效率低。

 

   sonar，虽然只是一个代码检测工具，但是它教会了我们如何写高效优雅的代码~