if分支中不能出现两个分支一样的情况。
把重复的删除掉就可以了。
提示资源没有关闭，需要在finally中进行资源关闭，但是把资源关闭放到finally中由提示这样写不规范有异味。所以它推荐的写法是将创建资源流的代码放在try()中，这样系统会自动的关闭资源，不需要我们写.close()方法，如图
这种提示是指可能存在空指针异常，需要增加空值检测。
这里提示的是在进行操作运算之前应该先进行类型转换。下面是他提供的一些规范和不规范的实例。
可以看出来，基本上是给提示的参数进行强类型转换，怎么知道应该转换什么类型呢，可以点击对应的方法查看这个参数是什么类型的，就将这个参数强转成这个类型就好了，比如上面的doubleTostring()方法的第二个参数是double类型的，所以强转一下就好了，如下：
提示是删除或纠正这种无用的自我分配。一看上面的代码，因该是哪位大佬写错了吧，所以改成temp2[i]=temp1[i];也算消除了异味
7.Saveandre-usethis“Random”.
这种提示是随机数应该需要重用，然后他给出的参考是这样的
在类中定义一个Random，然后在方法中可以重复使用，但是我按照这个方法试了一下没发现并不可行，导入下面的两个包都会报错，应该是需要什么额外的jar包才行感觉。
所以做了一个消除bug的方法。如下图，其实感觉变复杂了哈哈，但是消除了bug
上面代码都是报这个bug,提示不应该按照名称来比较类。不要求类名是唯一的，只要它们在包中是唯一的。因此，尝试根据类名确定对象的类型是一种充满危险的练习。其中一个危险是恶意用户将发送与受信任类同名的对象，从而获得可信访问。相反，应该使用instanceof运算符或Class.isAssignableFrom（）方法来检查对象的基础类型。下面我是通过isAssignableFrom（）来消除bug的。如下图：
提示原始字节值不应与逐位运算结合使用，读取字节以构建其他原始值（如整数或长整数）时，将自动提升字节值，但该提升可能会产生意外结果。要防止此类意外值转换，请使用按位和（＆）将字节值与0xff（255）组合，然后关闭所有较高位。所以在后面就加上“&0xff”如图：
提示非线程安全字段不应该是静态的，所以把前面的static修饰符去掉就可以。
这种interrupt异常，需要在catch中加上Thread.currentThread().interrupt();
提示是这个Integer封装是多余的，所以直接用强类型转换就可以吧。
这个提示用其他的方式初始化，不要在定义的时候就初始化，修改如下：
遇到的基本上就是这些bug了，还有css中的一些bug,按照提示修改的
将这两个body的样式合二为一，可以消除bug和漏洞
Display重复了，删除掉一个。
空样式，直接删除掉。
(更改此条件，以便它不总是评估为“false")这是很多webservice文件中出现的一个bug。如果后期会对webservice文件过滤，这个就可以不管了。不过也可以把这行删除掉，obj不可能为nullif(obj==null)returnfalse;这样这个bug也没有了。
这种提示就是异常应该用日志打印出来。
提示密码不能直接这样传递，不安全。但是也没有提供参考的案例。所以我是这样的改的，也能消除漏洞。如下图：
类变量字段不应具有公共可访问性。所以把public访问修饰符，改成其他的修饰符，最好是private.
提示应该保护XML变换器。创建javax.xml.transform.Transformer但未启用“安全处理”或创建一个而不禁用外部DTD时，可能会发生XML外部实体或XSLT外部实体（XXE）漏洞。如果该外部实体被攻击者劫持，则可能导致机密数据泄露，拒绝服务，服务器端请求伪造，从解析器所在机器的角度进行端口扫描，以及其他系统影响。进行修改如下可以消除漏洞：
提示当包含操作状态代码时，不应忽略返回值。也就是说不应该忽略文件删除操作的结果。所以进行如下修改，但是如下修改虽然修复了漏洞，但是新增了异味。
异味提示"java.nio.Files#delete"shouldbepreferred(squid:S4042)。应该使用Files.delete()方法，而不能之间文件delete.所以最后修改成：
这种“publicstatic”字段应该成员变量应该是不变的，所以需要加上final修饰，如下：
还有几种漏洞不好修复，暂时没有思路
这种提示，TrustManagers不应盲目接受任何证书。通常会创建X509TrustManager接口的空实现，以允许连接到未由根证书颁发机构签名的主机。这样的实现将接受任何证书，这使得应用程序容易受到中间人攻击。正确的解决方案是提供适当的信任存储。
这种原来用DES加密的提示不应使用DES（数据加密标准）和DESede（3DES）。它推荐的使用AES.但是将DES加密改成AES加密虽然程序异味消除了，但是程序肯定不对吧，加密方式换了肯定会出问题的吧。
异味太多了，我也没有消除太多了异味，一个项目异味一般都是好几k,所以消除起来一两个异味对基数没有什么影响，并且异味太多，消除一部分后，并没有感受到那种异味数量巨减的感觉，导致消除异味的积极性不是很高。并且有的异味是真的不好消除，有时候打开一个文件，几千行代码一片都是标异味。直接就放弃了。
swatch中没有default，也没有break；虽然上面的代码不要break也不会有什么问题。但是万一哪天变了，不是return就很容易出错了吧。
两个分号，代码中有很多地方有这种情况感觉，删掉多余的。
上图，定义的变量但是没有使用，就会抛出这种异味，解决这种异味，是需要看看这个变量有什么作用，没有作用的可以删除掉，如果不改随意改动，可以在他们下面增加一条日志打印他们，这样也能消除这个异味。
String.valueOf（）不应附加到String。这里我的理解是，result是string类型，arerType是int类型，在拼接的时候会自动的将int类型转换成string，不用多此一举。
类似这种，当一个不变的字符串在一个文件中多次出现，就应该给这些字符串提取成常量，这样方便以后修改和维护。但是说实话提取常量这个异味真的很枯燥，并且代码中有大量的这种情况，感觉每个项目或者每个模块都应该提取一个常量类，这样这个模块用到这些不变字符串，就直接从这个类中获取，但是这个工作量有点大哈哈，我就简单的尝试了一下，把自定义报表那部分提取常量，但是进行了一部分就没有进行下去了，枯燥且工作量大
但是我觉得这个工作还是做比较好，这样便于以后的维护，常量值改变只用改一个地方就好了，不用所有地方都修改。
当字符串需要频繁变化时，用stringBuilder代替string.这里还有一种异味是代码中有的地方用的是StringBuffer,也需要转成StringBuilder.因为Stringbuffer是确保线程安全的，stringBuilder不用，所以stringBuilder效率更高。那会不会引发线程安全问题呢，不会，因为这个是在方法内部定义的变量，所以对这个方法而言是线程封闭的，不会引发线程安全问题。
提示修饰符的顺序应该符合java语言规范，它给出的参考如下：
所以把static修饰符放到final就好了。
定义变量的时候，一个变量一行，便于查看
最好不要直接返回null,应返回空数组和集合.如下：
判断集合时候为空是，不要使用size()，建议使用isEmpty()方法。如下:
这种可以直接删除掉，或者不想删除的可以用/***/注释，对于单行的可以把后面的分号去掉就不会报错异味了。
布尔文字不应该是多余的。用true或者false在if中判断是不好的写好，直接可以通过本身进行判断，如下：
如果一个类的里面的方法都是static修饰的静态方法，那么需要给这个类定义一个非公共构造函数（添加私有构造函数以隐藏隐式公共构造函数）如下：
关于代码圈复杂度大于15的异味，以及代码过长的异味，说实话也没有什么好的方法，只能读代码，然后抽离函数出来，当然抽离函数不可能一次就能做到代码简洁之道要求的那样，一个函数只做一件事，单一层次原则。但是我们也不能因为做不到这样就就什么都不做，还是要迈出这一步，先抽离函数，虽然没有做到单一层次原则，但是消除了异味。下面代码是对上面的进行简单的函数抽离，消除异味
这样代码上看去显得非常的臃肿且感觉特别low,但是这却是我们最喜欢写的包括我自己，因为简单进本不用思考。并且如下图，18个case都没有报异味，所以SonarQube上也没有检查出来，所以大家都将就先改其他，后来被数据端同事看到了下面这个代码，说代码简洁之道不是说不能用这么多的swatch么？我竟然一时不知道怎么回答，只能说修改成本太大，不好改。但是事后自己仔细想想自己接受这段代码看的也感觉特别low,那自己为什么不能对自己的要求高一点呢，所以痛定思痛打算改这段代码。
我修改这部分代码采用的是枚举类型，先创建一个枚举，并将所有的case换成对应的枚举值，然后创建两个成员变量和一个带两个参数的枚举的构造方法。然后实现这两个成员变量的get方法，使得其他类可以访问。如下图：
然后在原来的swatch的代码中，删除这些分支，创建这个枚举，并根据分支创建对应的枚举值，如下：
这样就完成啦，看起来总算比那么多的swatch-case舒服很多吧，并且也没有那么low了。
两个分支一模一样的，需要删掉其中一个。然后像这种有很多if-elseif的getsql()方法的圈复杂度肯定是超过了，这里比较好的方法我也不知道怎么做，但是我是将整个分成多个一样的if-elseif的方法。但是这样只能消除异味，代码的可读性变差。