最近接近年底了,公司代码质量考核,进行了一波sonar扫描,哎,头疼,下面说一下常见的sonar能扫到的问题,时刻告诉自己,要写个优质的代码。-------顺便提一下,不要看同事的代码,否则苦不堪言。。。。。。。
bug类型:
1、".equals()" should not be used to test the values of "Atomic" classes.
bug 主要
不要使用equals方法对AtomicXXX进行是否相等的判断
Atomic变量永远只会和自身相等,Atomic变量没有覆写equals()方法.
2、"=+" should not be used instead of "+="
bug 主要
"=+" 与 "=+" 意义不同
a =+ b;虽然正确但写法不合规,应写成 a = +b;
3、"@NonNull" values should not be set to null
bug 次要
标注非空假定非空且在使用之前不进行非空检查,设置为空会导致空指针异常
4、"BigDecimal(double)" should not be used
bug 主要
因为浮点的不精确,可能使用BigDecimal(double)得不到期望的值
5、"compareTo" results should not be checked for specific values
bug 次要
compareTo可能返回不是具体的值(除0外),建议用 >0、<0、=0
6、"compareTo" should not return "Integer.MIN_VALUE"
bug 次要
compareTo只代表一个不等标识,不代表不等的程度,应返回-1,0,1标识即可
7、 "Double.longBitsToDouble" should not be used for "int"
bug 主要
Double.longBitsToDouble返回给定的位所代表的double值,需要一个64位的long类型参数.
8、 "equals" method overrides should accept "Object" parameters
bug 主要
equals作为方法名应该仅用于重写Object.equals(Object)来避免混乱.
9、 "equals(Object obj)" should test argument type
bug 次要
要比较obj的class type是否一样
10、"equals" methods should be symmetric and work for subclasses
bug 次要
equals应是对等并且在有子类参与时能正常工作
11、"equals(Object obj)" and "hashCode()" should be overridden in pairs
bug 次要
成对重写
12、"Externalizable" classes should have no-arguments constructors
bug 主要
Externalizable(可序列化与返序列化)类应该有无参构造器
13、"getClass" should not be used for synchronization
bug 主要
{synchronized (this.getClass())} 错误 子类继承此方法时不能做到同步
{synchronized (MyClass.class)} 正确
14、"hashCode" and "toString" should not be called on array instances
bug 主要
使用Arrays.toString(args)和Arrays.hashCode(args)代替.
15、"instanceof" operators that always return "true" or "false" should be removed
bug 主要
16、"InterruptedException" should not be ignored
bug 主要
try {
while (true) {
// do stuff
}
}catch (InterruptedException e) {
LOGGER.log(Level.WARN, "Interrupted!", e);
// Restore interrupted state...
Thread.currentThread().interrupt();
}
17、"Iterator.hasNext()" should not call "Iterator.next()"
bug 主要
18、"Iterator.next()" methods should throw "NoSuchElementException"
bug 次要
public String next(){
if(!hasNext()){
throw new NoSuchElementException();
}
...
}
19、"notifyAll" should be used
bug 主要
notify可能不能唤醒正确的线程,notifyAll代之。
20、"null" should not be used with "Optional"
bug 主要
把判空包装起来使用而不直接使用!=null
21、"PreparedStatement" and "ResultSet" methods should be called with valid indices
bug 阻断
PreparedStatement与ResultSet参数设置与获取数据由序号1开始而非0
22、"read" and "readLine" return values should be used
bug 主要
BufferedReader.readLine(), Reader.read()及子类中的相关方法都应该先存储再比较
buffReader = new BufferedReader(new FileReader(fileName));
String line = null;
while ((line = buffReader.readLine()) != null) {
// ...
}
23、"runFinalizersOnExit" should not be called
bug 严重
JVM退出时不可能运行finalizers,System.runFinalizersOnExit 和 Runtime.runFinalizersOnExit可以在jvm退出时运行但是因为他们不安全而弃用.
正确用法:
Runtime.addShutdownHook(new Runnable() {
public void run(){
doSomething();
}
});
24、"ScheduledThreadPoolExecutor" should not have 0 core threads
bug 严重
java.util.concurrent.ScheduledThreadPoolExecutor由属性corePoolSize指定线程池大小,如果设置为0表示线程执行器无线程可用且不做任何事.
25、"Serializable" inner classes of non-serializable classes should be "static"
bug 次要
序列化非静态内部类将导致尝试序列化外部类,如果外部类不是序列化类,会产生运行时异常,内部类静态化会避免这种情况
26、"SingleConnectionFactory" instances should be set to "reconnectOnException"
bug 主要
使用Spring SingleConnectionFactory而不启用reconnectOnException设置当连接恶化将阻止自动连接恢复。
27、"StringBuilder" and "StringBuffer" should not be instantiated with a character
bug 主要
StringBuffer foo = new StringBuffer('x'); 错 equivalent to StringBuffer foo = new StringBuffer(120);
StringBuffer foo = new StringBuffer("x"); 对
28、 "super.finalize()" should be called at the end of "Object.finalize()" implementations
bug 严重
protected void finalize() {
releaseSomeResources();
super.finalize(); //调用,最后调用
}
29、"toArray" should be passed an array of the proper type
bug 次要
toArray()无参且强制类型转换会产生运行时异常,应传入一个合适的类弄作参数
public String [] getStringArray(List
return strings.toArray(new String[0]);
}
30、"toString()" and "clone()" methods should not return null
bug 主要
可返回""
31、 "wait" should not be called when multiple locks are held
bug 阻断
32、 "wait", "notify" and "notifyAll" should only be called when a lock is obviously held on an object
bug 主要
先要获得对象锁才能进行上述操作
private void removeElement() {
synchronized(obj) {
while (!suitableCondition()){
obj.wait();
}
... // Perform removal
}
}
or
private synchronized void removeElement() {
while (!suitableCondition()){
wait();
}
... // Perform removal
}
33、"wait(...)" should be used instead of "Thread.sleep(...)" when a lock is held
bug 阻断
当持有锁的当前线程调用Thread.sleep(...)可能导致性能和扩展性问题,甚至死锁因为持有锁的当前线程已冻结.合适的做法是锁对象wait()释放锁让其它线程进来运行.
34、A "for" loop update clause should move the counter in the right direction
bug 主要
检查for循环下标递增或递减正确
35、All branches in a conditional structure should not have exactly the same implementation
bug 主要
分支中不应该有相同的实现
36、Blocks should be synchronized on "private final" fields or parameters
bug 主要
synchronized同步块应该锁在private final fields或parameters对象上,因为同步块内非final锁对象可能改变导致其它线程进来运行.
37、Boxing and unboxing should not be immediately reversed
bug 次要
自动拆箱和装箱不需手动转换
38、Child class methods named for parent class methods should be overrides
bug 主要
以下情况不是重写:
a、父类方法是static的而子类方法不是static的
b、子类方法的参数或返回值与父类方法不是同一个包
c、父类方法是private
为了不产生混乱,不要与父类方法同名
39、Classes extending java.lang.Thread should override the "run" method
bug 主要
线程类应该重写run方法
40、Classes should not be compared by name
bug 主要
不要用类名称比较类是否相同,而用instanceof或者Class.isAssignableFrom()进行底动类型比较
41、Classes that don't define "hashCode()" should not be used in hashes
bug 主要
没有定义hashCode()方法的类不能作为hash集合中的键值,因为equal相同的实例对像可能返回不同的hash值.
42、Collections should not be passed as arguments to their own methods
bug 主要
集合实例不应该作为参数被传给集合实你还自已的方法中
43、Conditionally executed blocks should be reachable
bug 主要
条件执行块应该可达
44、Constructor injection should be used instead of field injection
bug 主要
构造器注入应该替代属性注入(非Spring framework)
因为任何非Spring framework实例化而是通过构造器实例化的实例不能注入属性,这样公有的构造器实化化后可能产生NullPointerException,除非所有的构造器都是私有的
45、Consumed Stream pipelines should not be reused
bug 主要
流不应该重用
46、Custom resources should be closed
bug 阻断
资源应该关闭
47、Custom serialization method signatures should meet requirements
bug 主要
自定义类序列化方法签名应该合法
49、Dependencies should not have "system" scope
bug 严重
maven依赖不要在system scope
50、Dissimilar primitive wrappers should not be used with the ternary operator without explicit casting
bug 主要
不同的原始包装类如果没有明确的类转换不能用于三元操作中
51、Double Brace Initialization should not be used
bug 次要
双构造初始不要用
Map source = new HashMap(){{ // Noncompliant
put("firstName", "John");
put("lastName", "Smith");
}};
此操作如一个anonymous inner class,如果anonymous inner class返回且被其它对象引用,可能产生memory leaks,既使不产生memory leaks也会让大多维护者感到迷惑
52、Double-checked locking should not be used
bug 阻断
重复检查的锁块不要使用
public static Resource getInstance() {
if (resource == null) {
synchronized (DoubleCheckedLocking.class) {
if (resource == null)
resource = new Resource();
}
}
return resource;
}
应
public synchronized static Resource getInstance() {
if (resource == null)
resource = new Resource();
return resource;
}
53、Equals Hash Code
bug 严重
成对重写equals()与hashCode()
54、Exception should not be created without being thrown
bug 主要
不被抛出的异常不要创建
55、Expressions used in "assert" should not produce side effects
bug 主要
assert表达式不要产生负影响,不要改变数据状态
56、Failed unit tests should be fixed
bug 主要
失败的单元测试应该尽快解决掉
57、Floating point numbers should not be tested for equality
bug 主要
浮点数不要进行比较
58、Getters and setters should be synchronized in pairs
bug 主要
get与set应该成对进行同步操作
59、Identical expressions should not be used on both sides of a binary operator
bug 主要
相同的表达式不要作为二进制操作的操作数使用,应该简化
60、Inappropriate "Collection" calls should not be made
bug 主要
正确使用集合元素类型
61、Inappropriate regular expressions should not be used
bug 主要
正确使用正则表达式
62、Intermediate Stream methods should not be left unused
bug 主要
中间流应该被使用
63、Ints and longs should not be shifted by zero or more than their number of bits-1
bug 次要
整型与长整型位移操作数应该价于1与类型占位数-1
64、Invalid "Date" values should not be used
bug 主要
正确使用日期
65、Jump statements should not occur in "finally" blocks
bug 主要
finally块中使用return, break, throw等Jump statements,会阻止在try catch中抛出的未处理异常的传播
66、Locks should be released
bug 严重
保证锁的能够释放
67、Loop conditions should be true at least once
bug 主要
循环应该至少走一次
68、Loops should not be infinit
bug 阻断
循环不应该死循环
69、Math operands should be cast before assignment
bug 次要
数字操作在操作或赋值前要转化
70、Math should not be performed on floats
bug 次要
BigDecimal代替floats进行大数精确运算
71、Methods "wait(...)", "notify()" and "notifyAll()" should not be called on Thread instances
bug 阻断
不要在线程中使用"wait(...)", "notify()" and "notifyAll()"
72、Methods should not be named "hashcode" or "equal"
bug 主要
除非Override重写这些方法
73、Multiline blocks should be enclosed in curly braces
bug 主要
多列块应用大括号括起来
74、Neither "Math.abs" nor negation should be used on numbers that could be "MIN_VALUE"
bug 次要
不要对数值类型的MIN_VALUE值或返回值为此值进行Math.abs与取反操作,因为不会起作用。
75、Non-public methods should not be "@Transactional"
bug 主要
非public方法不要注解Transactional,调用时spring 会抛出异常
76、Non-serializable classes should not be written
bug 主要
执行写操作的类要序列化,否则会抛出异常
77、Non-serializable objects should not be stored in "HttpSession" objects
bug 主要
HttpSession要保存序列化的对象
78、Non-thread-safe fields should not be static
bug 主要
非线程安全的域不应该静态化
79、Null pointers should not be dereferenced
bug 主要
空指针引用不应被访问
80、Optional value should only be accessed after calling isPresent()
bug 主要
Optional实例值的获取要isPresent()之后再做操作
90、Printf-style format strings should not lead to unexpected behavior at runtime
bug 阻断
因为Printf风格格式化是在运行期解读,而不是在编译期检验,会存在风险
91、Raw byte values should not be used in bitwise operations in combination with shifts
bug 主要
原始字节值不应参与位运算
result = (result << 8) | readByte(); // Noncompliant
正:
result = (result << 8) | (readByte() & 0xff);
92、Reflection should not be used to check non-runtime annotations
bug 主要
反射操作不应该运于检查非运行时注解
93、Related "if/else if" statements should not have the same condition
bug 主要
if/else if中不应该有相同的条件
94、Resources should be closed
bug 阻断
打开的资源应该关闭并且放到finally块中进行关闭
95、Return values from functions without side effects should not be ignored
bug 主要
操作对函数返回值没有影响的应该忽略
public void handle(String command){
command.toLowerCase(); // Noncompliant; result of method thrown away
...
}
96、Servlets should not have mutable instance fields
bug 主要
servlet容器对每一个servlet创建一个实例导致实例变量共享产生问题
struts1.x 也是单例
97、Short-circuit logic should be used to prevent null pointer dereferences in conditionals
bug 主要
应正确使用短路逻辑来防止条件中的空指针引用访问
98、Silly equality checks should not be made
bug 主要
愚蠢的相等检查不应该做
非同类型的对象equal
99、Spring "@Controller" classes should not use "@Scope"
bug 主要
保持spring controller的单例
100、Synchronization should not be based on Strings or boxed primitives
bug 主要
字符串和封箱类不应该被用作锁定对象,因为它们被合并和重用。
101、The non-serializable super class of a "Serializable" class should have a no-argument constructor
bug 次要
序列化的类的非序列化父类应有一个无参构造器
102、The Object.finalize() method should not be called
bug 主要
Object.finalize()不要人为去调用
103、The Object.finalize() method should not be overriden
bug 主要
Object.finalize()不要重写
104、The signature of "finalize()" should match that of "Object.finalize()"
bug 主要
Object.finalize()不要重写
105、The value returned from a stream read should be checked
bug 次要
从流中读取的值应先检查再操作
106、Thread.run() should not be called directly
bug 主要
调用start()
107、Useless "if(true) {...}" and "if(false){...}" blocks should be removed
bug 主要
无用的if(true)和if(false)块应移除
108、Value-based classes should not be used for locking
bug 主要
基于值的类不要用于锁对象
109、Value-based objects should not be serialized
bug 次要
基于值的对象不应被用于序列化
110、Values should not be uselessly incremented
bug 主要
值增减后不存储是代码浪费甚至是bug
111、Variables should not be self-assigned
bug 主要
变量不应该自分配如下:
public void setName(String name) {
name = name;
}
112、Week Year ("YYYY") should not be used for date formatting
bug 主要
日期格式化错误
113、Zero should not be a possible denominator
bug 严重
零不应该是一个可能的分母
114、Loops should not be infinite
Bug 阻断
循环不应该是无限的
漏洞类型:
1、"@RequestMapping" methods should be "public"
漏洞 阻断
标注了RequestMapping是controller是处理web请求。既使方法修饰为private,同样也能被外部调用,因为spring通过反射调用方法,没有检查方法可视度,
2、"enum" fields should not be publicly mutable
漏洞 次要
枚举类域不应该是public,也不应该进行set
3、"File.createTempFile" should not be used to create a directory
漏洞 严重
File.createTempFile不应该被用来创建目录
4、"HttpServletRequest.getRequestedSessionId()" should not be used
漏洞 严重
HttpServletRequest.getRequestedSessionId()返回客户端浏览器会话id不要用,用HttpServletRequest.getSession().getId()
5、"javax.crypto.NullCipher" should not be used for anything other than testing
漏洞 阻断
NullCipher类提供了一种“身份密码”,不会以任何方式转换或加密明文。 因此,密文与明文相同。 所以这个类应该用于测试,从不在生产代码中。
6、"public static" fields should be constant
漏洞 次要
public static 域应该 final
7、Class variable fields should not have public accessibility
漏洞 次要
类变量域应该是private,通过set,get进行操作
8、Classes should not be loaded dynamically
漏洞 严重
不应该动态加载类,动态加载的类可能包含由静态类初始化程序执行的恶意代码.
Class clazz = Class.forName(className); // Noncompliant
9、Cookies should be "secure"
漏洞 次要
Cookie c = new Cookie(SECRET, secret); // Noncompliant; cookie is not secure
response.addCookie(c);
正:
Cookie c = new Cookie(SECRET, secret);
c.setSecure(true);
response.addCookie(c);
10、Credentials should not be hard-coded
漏洞 阻断
凭证不应该硬编码
11、Cryptographic RSA algorithms should always incorporate OAEP (Optimal Asymmetric Encryption Padding)
漏洞 严重
加密RSA算法应始终包含OAEP(最优非对称加密填充)
12、Default EJB interceptors should be declared in "ejb-jar.xml"
漏洞 阻断
默认EJB拦截器应在“ejb-jar.xml”中声明
13、Defined filters should be used
漏洞 严重
web.xml文件中定义的每个过滤器都应该在
14、Exceptions should not be thrown from servlet methods
漏洞 次要
不应该从servlet方法抛出异常
15、HTTP referers should not be relied on
漏洞 严重
不应依赖于http,将这些参数值中止后可能是安全的,但绝不应根据其内容作出决定。
如:
String referer = request.getHeader("referer"); // Noncompliant
if(isTrustedReferer(referer)){
//..
}
16、IP addresses should not be hardcoded
漏洞 次要
ip 地址不应该硬编码
17、Member variable visibility should be specified
漏洞 次要
应指定成员变量的可见性
18、Members of Spring components should be injected
漏洞 严重
spring组件的成员应注入,单例注入非静态成员共享会产生风险
19、Mutable fields should not be "public static"
漏洞 次要
多变在域不应为 public static
20、Neither DES (Data Encryption Standard) nor DESede (3DES) should be used
漏洞 阻断
不应使用DES(数据加密标准)和DESEDE(3DES)
21、Only standard cryptographic algorithms should be used
漏洞 严重
标准的加密算法如 SHA-256, SHA-384, SHA-512等,非标准算法是危险的,可能被功能者攻破算法
22、Pseudorandom number generators (PRNGs) should not be used in secure contexts
漏洞 严重
伪随机数生成器(PRNG)不应在安全上下文中使用
23、Return values should not be ignored when they contain the operation status code
漏洞 次要
当函数调用的返回值包含操作状态代码时,应该测试此值以确保操作成功完成。
24、Security constraints should be definedin
漏洞 阻断
应定义安全约束,当web.xml文件没有
25、SHA-1 and Message-Digest hash algorithms should not be used
漏洞 严重
不应该使用SHA-1和消息摘要散列算法,已证实不再安全
26、SQL binding mechanisms should be used
漏洞 阻断
应该使用SQL绑定机制
27、Struts validation forms should have unique names
漏洞 阻断
struts验证表单应有唯一名称
28、Throwable.printStackTrace(...) should not be called
漏洞 次要
Throwable.printStackTrace(...)会打印异常信息,但会暴露敏感信息
29、Untrusted data should not be stored in sessions
漏洞 主要
不受信任的数据不应存储在会话中。
Web会话中的数据被认为在“信任边界”内。 也就是说,它被认为是值得信赖的。 但存储未经身份验证的用户未经验证的数据违反信任边界,并可能导致该数据被不当使用。
30、Values passed to LDAP queries should be sanitized
漏洞 严重
传递到LDAP查询的值应该被清理
31、Values passed to OS commands should be sanitized
漏洞 严重
传递给OS命令的值应该被清理
32、Web applications should not have a "main" method
漏洞 严重
web 应用中不应有一个main方法
坏味道:
1、"==" and "!=" should not be used when "equals" is overridden
坏味道 次要
当类重写equals方法后,不应该再用"=="与"!="进行对象比较
2、"@Deprecated" code should not be used
坏味道 次要
弃用方代码不应再用,弃用代码意味首将会移除,应该使用替代代码
3、"@Override" should be used on overriding and implementing methods
坏味道 主要
重写的和实现在方法要加Override标注
4、"action" mappings should not have too many "forward" entries
坏味道 次要
默认 4
action不要有太多的forward
5、"Arrays.stream" should be used for primitive arrays
坏味道 主要
Arrays.stream用于原始流类型(IntStream, LongStream, DoubleStream)会有更好的性能
6、"catch" clauses should do more than rethrow
坏味道 次要
只是重新抛出捕获的异常和完全放弃异常捕获效果一样,但会给维护者带来疑惑
7、"clone" should not be overridden
坏味道 阻断
不应重写clone方法
8、"Cloneables" should implement "clone"
坏味道 严重
Cloneables类应该实现clone方法
9、"collect" should be used with "Streams" instead of "list::add"
坏味道 次要
虽然您可以使用forEach(list :: add)或使用Stream收集,但是收集是更好的选择,因为它自动线程安全并且可并行
10、"Collections.EMPTY_LIST", "EMPTY_MAP", and "EMPTY_SET" should not be used
坏味道 次要
11、"DateUtils.truncate" from Apache Commons Lang library should not be used
坏味道 主要
使用Java 8中引入的Instant类来截断日期可能会比Commons Lang的DateUtils类快得多。
12、"deleteOnExit" should not be used
坏味道 主要
不推荐使用File.deleteOnExit()
13、"entrySet()" should be iterated when both the key and value are needed
坏味道 主要
当循环中只需要一个map的键时,迭代keySet就是有意义的。 但是,当需要键和值两者时,迭代entrySet更有效,这将允许访问键和值
14、"equals(Object obj)" should be overridden along with the "compareTo(T obj)" method
坏味道 次要
“equals(Object obj)”应该与“compareTo(T obj)”方法一起被重写
15、"Exception" should not be caught when not required by called methods
坏味道 次要
如果被调方法没有抛出“Exception”时不要捕获"Exception",应捕后被调方法抛出的异常
16、"final" classes should not have "protected" members
坏味道 次要
最终类意味首不可继承所以不要有受保存成员,这样没有意义
17、"finalize" should not set fields to "null"
坏味道 次要
finalize不应设置域空,对垃圾收集是没必要的,还可能为垃圾收集带来额处开消
18、"for" loop increment clauses should modify the loops' counters
坏味道 严重
loop循环应该增加递增序号变量
19、"for" loop stop conditions should be invariant
坏味道 主要
循环停止条件应为不变
20、"indexOf" checks should not be for positive numbers
坏味道 严重
检查indexOf返回不应使用正数
21、"indexOf" checks should use a start position
坏味道 次要
如果您需要查看一个子字符串是否位于字符串中某个特定点之外,则可以测试该子字符串与该目标点的indexOf,也可以使用该起始点参数的indexOf版本。 后者可以更清楚,因为结果是针对-1测试的,这是一个容易识别的“未找到”指标
22、"java.lang.Error" should not be extended
坏味道 主要
java.lang.Error及其子类表示异常情况,例如OutOfMemoryError,它只能由Java虚拟机进行处理。
23、"java.nio.Files#delete" should be preferred
坏味道 主要
Files.delete(path)道选
24、"java.time" classes should be used for dates and times
坏味道 主要
Date 和 Calendar类非线程同步,推荐使用LocalDate
25、"Lock" objects should not be "synchronized"
坏味道 主要
“锁定”对象不应“同步”
java.util.concurrent.locks.Lock提供比同步块更强大和灵活的锁定操作,应该使用tryLock()和unlock()锁定和解锁这些对象
26、"main" should not "throw" anything
坏味道 阻断
main方法不应该抛出异常
27、"NullPointerException" should not be caught
坏味道 主要
空指针不应捕获处理,应该避免NullPointerException,而不是被捕获
28、"NullPointerException" should not be explicitly thrown
坏味道 主要
“NullPointerException”不应该被显式抛出
29、"Object.finalize()" should remain protected (versus public) when overriding
坏味道 严重
重写finalizey方法应为protected
30、"Object.wait(...)" and "Condition.await(...)" should be called inside a "while" loop
坏味道 严重
“Object.wait(...)”和“Condition.await(...)”应该在“while”循环内调用
31、"Object.wait(...)" should never be called on objects that implement "java.util.concurrent.locks.Condition"
坏味道 主要
不应该在实现“java.util.concurrent.locks.Condition”的对象上调用“Object.wait(...)”
32、"Optional" should not be used for parameters
坏味道 主要
Optional不要被用作参数
33、"Preconditions" and logging arguments should not require evaluation
坏味道 主要
将连接的字符串传递到日志记录方法也可能导致不必要的性能消耗,因为每次调用该方法时将执行级联,无论日志级别是否足够低以显示消息
34、"private" methods called only by inner classes should be moved to those classes
坏味道 次要
只被内部类调用的方法,应该在内部类内部
35、"private" methods that don't access instance data should be "static"
坏味道 次要
不访问实例数据的“私有”方法应该是“静态”
36、"readObject" should not be "synchronized"
坏味道 主要
“readObject”不应该被“同步”
37、"readResolve" methods should be inheritable
坏味道 严重
“readResolve”方法应该是可继承的
38、"ResultSet.isLast()" should not be used
坏味道 主要
ResultSet.isLast()不应用
39、"Serializable" classes should have a version id
坏味道 严重
“Serializable”类应该有一个版本号
40、"Serializable" inner classes of "Serializable" classes should be static
坏味道 次要
实现序列化的内部类应该是静态的
41、"static" members should be accessed statically
坏味道 主要
“静态”成员应类访问
42、"Stream.anyMatch()" should be preferred
坏味道 次要
“Stream.anyMatch()”应该是首选的
43、"switch case" clauses should not have too many lines of code
坏味道 主要
默认 5
“switch case”子句不应该有太多的代码行
44、"switch" statements should end with "default" clauses
坏味道 严重
“switch”语句应以“default”子句结尾
45、"switch" statements should have at least 3 "case" clauses
坏味道 次要
“switch”语句应具有至少3个“case”子句,可以用if替代
46、"switch" statements should not contain non-case labels
坏味道 阻断
switch语句不用包含非case标签
47、"switch" statements should not have too many "case" clauses
坏味道 主要
默认 30
switch语句不应包含太多case语句
48、"Thread.sleep" should not be used in tests
坏味道 主要
“Thread.sleep”不应该在测试中使用
49、"ThreadLocal.withInitial" should be preferred
坏味道 次要
“ThreadLocal.withInitial”应该是首选
ThreadLocal> myThreadLocal = ThreadLocal.withInitial(ArrayList::new);
50、"Threads" should not be used where "Runnables" are expected
坏味道 主要
Noncompliant Code Example
public static void main(String[] args) {
Thread r =new Thread() {
int p;
@Override
public void run() {
while(true)
System.out.println("a");
}
};
new Thread(r).start(); // Noncompliant
Compliant Solution
public static void main(String[] args) {
Runnable r =new Runnable() {
int p;
@Override
public void run() {
while(true)
System.out.println("a");
}
};
new Thread(r).start();
51、"throws" declarations should not be superfluous
坏味道 次要
“抛出”声明不应该是多余的
52、"toString()" should never be called on a String object
坏味道 次要
不应该在String对象上调用“toString()”
53、"URL.hashCode" and "URL.equals" should be avoided
坏味道 主要
应避免使用“URL.hashCode”和“URL.equals”
54、"writeObject" should not be the only "synchronized" code in a class
坏味道 主要
“writeObject”不应该是类中唯一的“同步”代码
55、 @FunctionalInterface annotation should be used to flag Single Abstract Method interfaces
坏味道 严重
一个只有一个抽象方法的接口应加FunctionalInterface注释
56、A "while" loop should be used instead of a "for" loop
坏味道 次要
当在for循环中仅定义条件表达式,并且缺少初始化和增量表达式时,应使用while循环来增加可读性
57、A close curly brace should be located at the beginning of a line
坏味道 次要
共享编码约定使得团队有可能有效地进行协作。 这个规则使得强制要在行的开头放置一个大括号。
58、A field should not duplicate the name of its containing class
坏味道 主要
字段不应该重复其包含的类的名称
59、Abbreviation As Word In Name
坏味道 主要
检查验证标识符名称中的缩写(连续大写字母)长度,还允许执行骆驼案例命名
60、Abstract Class Name
坏味道 主要
检查抽象类名是否符合指定的格式
ignoreName
Controls whether to ignore checking the name. Realistically only useful if using the check to identify that match name and do not have the abstract modifier name. Default is false.
默认值
false
ignoreModifier
Controls whether to ignore checking for the abstract modifier on classes that match the name. Default is false.
默认值
false
format
Regular expression
默认值
61、Abstract class names should comply with a naming convention
坏味道 次要
抽象类名称应符合命名约定
Regular expression used to check the abstract class names against.
默认值
^Abstract[A-Z][a-zA-Z0-9]*$
62、Abstract classes without fields should be converted to interfaces
坏味道 次要
没有字段的抽象类应该转换为接口
63、Abstract methods should not be redundant
坏味道 次要
抽象方法不应该是多余的
64、An abstract class should have both abstract and concrete methods
坏味道 次要
抽象类应该有抽象和具体的方法
65、An open curly brace should be located at the beginning of a line
坏味道 次要
开放的大括号应位于一行的开头
66、An open curly brace should be located at the end of a line
坏味道 次要
开放的大括号应位于一行的末尾
67、Annotation arguments should appear in the order in which they were declared
坏味道 次要
注释参数应按其声明顺序显示
68、Annotation Location
坏味道 主要
注释位置
allowSamelineSingleParameterlessAnnotation
To allow single parameterless annotation to be located on the same line as target element.
默认值
true
allowSamelineParameterizedAnnotation
To allow parameterized annotation to be located on the same line as target element.
默认值
false
allowSamelineMultipleAnnotations
To allow annotation to be located on the same line as target element.
默认值
false
tokens
tokens to check
默认值
CLASS_DEF,INTERFACE_DEF,ENUM_DEF,METHOD_DEF,CTOR_DEF,VARIABLE_DEF
69、Annotation repetitions should not be wrapped
坏味道 次要
注释重复不应包装
70、Annotation Use Style
坏味道 主要
trailingArrayComma
Defines the policy for trailing comma in arrays. Default is never.
closingParens
Defines the policy for ending parenthesis. Default is never.
elementStyle
Defines the annotation element styles. Default value is compact_no_array.
71、Anon Inner Length
坏味道 主要
检查长匿名内部类。
max
maximum allowable number of lines. Default is 20.
72、Anonymous inner classes containing only one method should become lambdas
坏味道 主要
只有一个方法的匿名内部类应该变成lambdas
jdk8以下自动禁用
73、Array designators "[]" should be located after the type in method signatures
坏味道 次要
数组代号“[]”应位于方法签名类型之后
74、Array designators "[]" should be on the type, not the variable
坏味道 次要
数组代号“[]”应位于类型之后而不是变量之后
75、Array Trailing Comma
坏味道 主要
检查数组初始化是否包含逗号
76、Array Type Style
坏味道 次要
数组类型样式
javaStyle
Controls whether to enforce Java style (true) or C style (false). Default is true.
77、Arrays should not be created for varargs parameters
坏味道 次要
不应为varargs参数创建数组
78、Artifact ids should follow a naming convention
坏味道 次要
共享命名约定允许团队有效协作。 当pom的artifactId与提供的正则表达式不匹配时,此规则引发了一个问题
regex
The regular expression the "artifactId" should match
默认值
[a-z][a-z-0-9]+
79、Assertions should be complete
坏味道 阻断
断言应该是完整的
80、Assignments should not be made from within sub-expressions
坏味道 主要
不应在子表达式中作出赋值操作当赋值变量没有用到
81、At-clause Order
坏味道 主要
检查从句顺序
tagOrder
allows to specify the order by tags.
默认值
@author,@version,@param,@return,@throws,@exception,@see,@since,@serial,@serialField,@serialData,@deprecated
target
allows to specify targets to check at-clauses.
82、Avoid Escaped Unicode Characters
坏味道 主要
避免转义的Unicode字符
allowIfAllCharactersEscaped
Allow if all characters in literal are escaped.
默认值
false
allowNonPrintableEscapes
Allow non-printable escapes.
默认值
false
allowByTailComment
Allow use escapes if trail comment is present.
默认值
false
allowEscapesForControlCharacters
Allow use escapes for non-printable(control) characters.
默认值
false
83、Avoid Inline Conditionals
坏味道 次要
避免内联条件
84、Avoid Nested Blocks
坏味道 主要
避免嵌套块
allowInSwitchCase
Allow nested blocks in case statements. Default is false.
85、Avoid Star Import
坏味道 次要
检查发现使用*符号的导入语句
excludes
packages where star imports are allowed. Note that this property is not recursive, subpackages of excluded packages are not automatically excluded.
allowStaticMemberImports
whether to allow starred static member imports like import static org.junit.Assert.*;
. Default is false.
默认值
false
allowClassImports
whether to allow starred class imports like import java.util.*;
. Default is false.
默认值
false
86、Avoid Static Import
坏味道 次要
避免静态导入
87、Boolean checks should not be inverted
坏味道 次要
布尔检查不应该被反转
Noncompliant Code Example
if ( !(a == 2)) { ...} // Noncompliant
boolean b = !(i < 10); // Noncompliant
Compliant Solution
if (a != 2) { ...}
boolean b = (i >= 10);
88、Boolean Expression Complexity
坏味道 主要
将嵌套布尔运算符(&&,||和^)限制为指定的深度(默认= 3)。
max
the maximum allowed number of boolean operations in one expression. Default is 3.
默认值
3
tokens
tokens to check. Default is LAND,BAND,LOR,BOR,BXOR.
默认值
LAND,BAND,LOR,BOR,BXOR
89、Boolean expressions should not be gratuitous
坏味道 主要
如果boolean表达式的值是已定的,那么boolean表达式是没有必要的可以移除
90、Boolean literals should not be redundant
坏味道 次要
boolean不需再与true,false比较作为boolean表达式
91、Branches should have sufficient coverage by tests
坏味道 主要
分支应有足够的测试覆盖
minimumBranchCoverageRatio
默认值
65
92、Case insensitive string comparisons should be made without intermediate upper or lower casing
坏味道 次要
使用toLowerCase()或toUpperCase()来使不区分大小写的比较无效,因为它需要创建临时的中间String对象。
93、Catch Parameter Name
坏味道 主要
检查catch参数名是否符合format属性指定的格式
format
Specifies valid identifiers. Default is ^(e|t|ex|[a-z][a-z][a-zA-Z]+)$
默认值
^(e|t|ex|[a-z][a-z][a-zA-Z]+)$
94、Catches should be combined
坏味道 次要
由于Java 7可以一次捕获多个异常。 因此,当多个catch块具有相同的代码时,它们应该被组合以便更好的可读性,sonar.java.source低于7时,此规则将自动禁用
95、Checked exceptions should not be thrown
坏味道 主要
检查的异常不应该被抛出,要处理
96、Child class fields should not shadow parent class fields
坏味道 阻断
子类字段不应该private父类的非private字段
97、Class Data Abstraction Coupling
坏味道 主要
度量衡量给定类中其他类的实例化数。
max
the maximum threshold allowed. Default is 7.
excludedClasses
User-configured class names to ignore.
excludeClassesRegexps
User-configured regular expressions to ignore classes
excludedPackages
User-configured packages to ignore
98、Class Fan Out Complexity
坏味道 主要
类的依赖类数量
max
the maximum threshold allowed. Default is 20.
excludedClasses
User-configured class names to ignore
excludeClassesRegexps
User-configured regular expressions to ignore classes
excludedPackages
User-configured packages to ignore
99、Class names should comply with a naming convention
坏味道 次要
类名应符合命名约定
format
Regular expression used to check the class names against.
默认值
^[A-Z][a-zA-Z0-9]*$
100、Class names should not shadow interfaces or superclasses
坏味道 严重
类名称不应该影响接口或超类(相同)
101、Class Type(Generic) Parameter Name
坏味道 主要
泛型参数名称符合指定的格式
format
Regular expression
默认值
^[A-Z]$
102、Classes and enums with private members should have a constructor
坏味道 主要
有私有成员的类和枚举应该有一个构造函数
103、Classes and methods that rely on the default system encoding should not be used
坏味道 次要
不应使用依赖于默认系统编码的类和方法
104、Classes from "sun.*" packages should not be used
坏味道 主要
不得使用“sun.*”软件包的类,sun类*或com.sun *包被视为实现细节,不属于Java API
Exclude
Comma separated list of Sun packages to be ignored by this rule. Example: com.sun.jna,sun.misc
105、Classes named like "Exception" should extend "Exception" or a subclass
坏味道 主要
名为“异常”的类应该扩展“异常”或者一个子类
106、Classes should not access their own subclasses during initialization
坏味道 严重
类在初始化期间不应访问自己的子类
107、Classes should not be coupled to too many other classes (Single Responsibility Principle)
坏味道 主要
类不应与太多其他类(单一责任原则)相耦合(依赖)
max
Maximum number of classes a single class is allowed to depend upon
默认值
20
108、Classes should not be empty
坏味道 次要
空类没意义,作为公共扩展点可以作为接口
109、Classes should not be too complex
坏味道 严重 废弃
类不应太复杂
max
Maximum complexity allowed.
默认值
200
110、Classes should not have too many "static" imports
坏味道 主要
静态导入类允许您使用其公共静态成员,而不必使用类名。 这可以很方便,但如果静态导入太多的类,你的代码可能会变得混乱,很难维护
threshold
The maximum number of static imports allowed
默认值
4
111、Classes should not have too many fields
坏味道 主要
类不应有太多字段
countNonpublicFields
Whether or not to include non-public fields in the count
默认值
true
maximumFieldThreshold
The maximum number of fields
默认值
20
112、Classes should not have too many methods
坏味道 主要
类不应该有太多方法
countNonpublicMethods
Whether or not to include non-public methods in the count.
默认值
true
maximumMethodThreshold
The maximum number of methods authorized in a class.
默认值
35
113、Classes that override "clone" should be "Cloneable" and call "super.clone()"
坏味道 次要
覆盖“克隆”的类应该是“可克隆”,并调用“super.clone()”
114、Classes with only "static" methods should not be instantiated
坏味道 主要
只有“静态”方法的类不应该被实例化
115、Classes without "public" constructors should be "final"
坏味道 次要
只有私有构造函数的类应该被标记为final,以防止任何错误的扩展尝试。
116、Close curly brace and the next "else", "catch" and "finally" keywords should be located on the same line
坏味道 次要
关闭大括号,下一个“else”,“catch”和“finally”关键字应位于同一行
117、Close curly brace and the next "else", "catch" and "finally" keywords should be on two different lines
坏味道 次要
关闭大括号和下一个“else”,“catch”和“finally”关键字应该在两个不同的行
118、Cognitive Complexity of methods should not be too high
坏味道 严重
认知复杂度是衡量一种方法的控制流程难以理解的度量。 认知复杂性较高的方法难以维持。
Threshold
The maximum authorized complexity.
默认值
15
119、Collapsible "if" statements should be merged
坏味道 主要
可合并的“if”语句应该合并
120、Collection methods with O(n) performance should be used carefully
坏味道 次要
应仔细使用具有O(n)性能的集合方法
121、Collection.isEmpty() should be used to test for emptiness
坏味道 次要
应该使用Collection.isEmpty()来测试空集合
122、Comment pattern matcher
坏味道 次要
该规则允许在TODO,NOPMD,...之外的任何类型的内容中找到任何类型的模式,NOSONAR除外
123、Comments Indentation
坏味道 次要
注释缩进
tokens
tokens to check
默认值
SINGLE_LINE_COMMENT,BLOCK_COMMENT_BEGIN
124、Comments should not be located at the end of lines of code
坏味道 次要
注释不应位于代码行的末尾
legalTrailingCommentPattern
Description Pattern for text of trailing comments that are allowed. By default, comments containing only one word.
默认值
^\s*+[^\s]++$
125、Comparators should be "Serializable"
坏味道 严重
Comparators should be "Serializable"
126、Conditionals should start on new lines
坏味道 严重
条件表达式应该起始新行
127、Constant Name
坏味道 次要
检查常数名称是否符合指定的格式
applyToPackage
Controls whether to apply the check to package-private member
默认值
true
format
Regular expression
默认值
^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$
applyToPublic
Controls whether to apply the check to public member
默认值
true
applyToProtected
Controls whether to apply the check to protected member
默认值
true
applyToPrivate
Controls whether to apply the check to private member
默认值
true
128、Constant names should comply with a naming convention
坏味道 严重
常数名称应符合命名约定
format
Regular expression used to check the constant names against.
默认值
^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$
129、Constants should not be defined in interfaces
坏味道 严重
常量不应在接口中定义
130、Constructors should not be used to instantiate "String" and primitive-wrapper classes
坏味道 主要
构造函数不应用于实例化“String”和原始包装类
131、Constructors should only call non-overridable methods
坏味道 严重
构造函数只应该调用不可覆盖的方法
132、Control flow statements "if", "for", "while", "switch" and "try" should not be nested too deeply
坏味道 严重
控制流程语句“if”,“for”,“while”,“switch”和“try”不能嵌套太深
max
Maximum allowed control flow statement nesting depth.
默认值
3
133、Control structures should use curly braces
坏味道 严重
控制结构应使用花括号
134、Covariant Equals
坏味道 严重
检查一个类是否定义了一个协变方法equals,那么它定义了方法equals(java.lang.Object)
135、Custom Import Order
坏味道 主要
检查导入声明组按照用户指定的顺序显示。 如果有导入,但是在组态中未指定其组,则导入应放在导入列表的末尾。
thirdPartyPackageRegExp
RegExp for THIRDPARTY_PACKAGE group imports.
默认值
^$
separateLineBetweenGroups
Force empty line separator between import groups.
默认值
true
sortImportsInGroupAlphabetically
Force grouping alphabetically.
默认值
false
specialImportsRegExp
RegExp for SPECIAL_IMPORTS group imports.
默认值
^$
customImportOrderRules
List of order declaration customizing by user.
standardPackageRegExp
RegExp for STANDARD_JAVA_PACKAGE group imports.
默认值
java|javax
136、Cyclomatic Complexity
坏味道 主要
检查针对特定限制的方法的循环复杂性
switchBlockAsSingleDecisionPoint
whether to treat the whole switch block as a single decision point
默认值
false
max
the maximum threshold allowed.
默认值
10
tokens
tokens to check
默认值
LITERAL_WHILE,LITERAL_DO,LITERAL_FOR,LITERAL_IF,LITERAL_SWITCH,LITERAL_CASE,LITERAL_CATCH,QUESTION,LAND,LOR
137、Dead stores should be removed
坏味道 主要
没用的存储应该除移
138、Declaration Order
坏味道 提示
声名的顺序
ignoreModifiers
Whether to ignore modifiers
默认值
false
ignoreConstructors
Whether to ignore constructors
默认值
false
139、Declarations should use Java collection interfaces such as "List" rather than specific implementation classes such as "LinkedList"
坏味道 次要
声明应该使用Java集合接口,例如“List”,而不是特定的实现类,如“LinkedList”
140、Default annotation parameter values should not be passed as arguments
坏味道 次要
默认注解参数值不应作为参数传递
141、Default Comes Last
坏味道 主要
检查在switch语句中的所有情况之后的默认值。
skipIfLastAndSharedWithCase
whether to allow default along with case if they are not last
默认值
false
142、Deprecated "${pom}" properties should not be used
坏味道 次要
不应使用不推荐使用的“$ {pom}”属性
143、Deprecated code should be removed
坏味道 提示
应删除弃用的代码
144、Deprecated elements should have both the annotation and the Javadoc tag
坏味道 主要
弃用元素应有注解和doc标签
145、Descendant Token
坏味道 次要
检查其他令牌下的限制令牌
maximumMessage
error message when the maximum count is exceeded
maximumDepth
the maximum depth for descendant counts
limitedTokens
set of tokens with limited occurrences as descendants
maximumNumber
a maximum count for descendants
minimumMessage
error message when the maximum count is exceeded
minimumNumber
a minimum count for descendants
minimumDepth
the minimum depth for descendant counts
sumTokenCounts
whether the number of tokens found should be calculated from the sum of the individual token counts
默认值
false
146、Design For Extension
坏味道 次要
扩展设计
ignoredAnnotations
Annotations which allow the check to skip the method from validation.
默认值
Test,Before,After,BeforeClass,AfterClass
147、EJB interceptor exclusions should be declared as annotations
坏味道 阻断
EJB interceptor exclusions应该以注解的形式使用
148、Empty arrays and collections should be returned instead of null
坏味道 主要
应该返回空数组和集合,而不是null
149、Empty Block
坏味道 主要
Checks for empty blocks
tokens
blocks to check
默认值
LITERAL_WHILE,LITERAL_TRY,LITERAL_FINALLY,LITERAL_DO,LITERAL_IF,LITERAL_ELSE,LITERAL_FOR,INSTANCE_INIT,STATIC_INIT,LITERAL_SWITCH,LITERAL_SYNCHRONIZED
option
policy on block contents
默认值
stmt
150、Empty catch block
坏味道 主要
检查空的catch块。 有两个选项可以使验证更加精确(默认情况下,检查允许空的catch块和任何注释)
exceptionVariableName
Format of skipping exception''s variable name.
默认值
^$
commentFormat
Format of comment.
默认值
.*
151、Empty For Initializer Pad
坏味道 次要
检查初始化程序为空的填充; 那是空的是否需要一个空的初始化程序,或者禁止这样的空格。 示例:for(; i
option
policy on how to pad an empty for iterator
152、Empty For Iterator Pad
坏味道 次要
检查一个空的填充迭代器; 那就是空格是否需要一个空的迭代器,否则这样的空格是被禁止的。 示例:for(Iterator foo = very.long.line.iterator(); foo.hasNext();)
option
policy on how to pad an empty for iterator
153、Empty Line Separator
坏味道 主要
在标题,包,所有导入声明,字段,构造函数,方法,嵌套类,静态初始化器和实例初始化器之后检查空行分隔符
allowNoEmptyLineBetweenFields
Allow no empty line between fields
默认值
false
allowMultipleEmptyLines
Allows multiple empty lines between class members.
默认值
true
tokens
assignments to check
默认值
PACKAGE_DEF,IMPORT,CLASS_DEF,INTERFACE_DEF,ENUM_DEF,STATIC_INIT,INSTANCE_INIT,METHOD_DEF,CTOR_DEF,VARIABLE_DEF
allowMultipleEmptyLinesInsideClassMembers
Allow multiple empty lines inside class members
默认值
true
154、Empty Statement
坏味道 次要
检测空的语句(独立的';')。
155、Empty statements should be removed
坏味道 次要
移除空语句
156、Enumeration should not be implemented
坏味道 主要
不应实同Enumeration
157、Equality operators should not be used in "for" loop termination conditions
坏味道 严重
循环终止条件下不应使用平等运算符
158、Equals Avoid Null
坏味道 主要
159、Escaped Unicode characters should not be used
坏味道 主要
不应使用转义的Unicode字符
160、Exception classes should be immutable
坏味道 次要
异常类应该是不可变的
161、Exception handlers should preserve the original exceptions
坏味道 主要
异常处理程序应保留原始异常
162、Exception types should not be tested using "instanceof" in catch blocks
坏味道 主要
异常类型不应该在catch块中使用“instanceof”进行测试
163、Exceptions should not be thrown in finally blocks
坏味道 严重
异常不应该在finally块中抛出
164、Executable Statement Count
坏味道 主要
将可执行语句的数量限制为指定的限制(默认= 30)。
max
the maximum threshold allowed. Default is 30.
默认值
30
tokens
members to check
默认值
CTOR_DEF,METHOD_DEF,INSTANCE_INIT,STATIC_INIT
165、Execution of the Garbage Collector should be triggered only by the JVM
坏味道 严重
垃圾收集器的执行只能由JVM触发
166、Exit methods should not be called
坏味道 阻断
调用System.exit(int status)或Rutime.getRuntime()。exit(int status)调用关闭挂钩并关闭整个Java虚拟机。 调用Runtime.getRuntime()。halt(int)立即关闭,而不调用关闭挂钩,并跳过完成
167、Explicit Initialization
坏味道 主要
检查任何类或对象成员是否明确地初始化为其类型值的默认值(对于对象引用为空,数字类型为零,对于布尔为char为false)。
168、Expressions should not be too complex
坏味道 严重
表达式不应太复杂
max
Maximum number of allowed conditional operators in an expression
默认值
3
169、Extensions and implementations should not be redundant
坏味道 次要
扩展和实现不应该是多余的
170、Fall Through
坏味道 主要
171、Field names should comply with a naming convention
坏味道 次要
字段名称应符合命名约定
format
Regular expression used to check the field names against.
默认值
^[a-z][a-zA-Z0-9]*$
172、Fields in a "Serializable" class should either be transient or serializable
坏味道 严重
“Serializable”类中的字段应该是transient或可序列化的
173、Fields in non-serializable classes should not be "transient"
坏味道 次要
不可序列化类的字段不应该是“transient”
174、Fields should not be initialized to default values
坏味道 次要
不应将字段初始化为默认值
175、File Contents Holder
坏味道 次要
配置为TreeWalker子模块时,保留当前的全局访问文件内容。 例如,过滤器可以通过此模块访问当前文件内容
176、File Length
坏味道 主要
如果源文件变得很长,那么很难理解。 因此,长类通常应该重构到专注于特定任务的几个单独的类中
fileExtensions
file type extension of files to process
max
maximum allowable number of lines. Default is 2000.
177、File Tab Character
坏味道 次要
检查源代码中没有制表符('\ t')
fileExtensions
file type extension of files to process
eachLine
whether to report on each line containing a tab, or just the first instance. Default is false.
178、Files should contain an empty new line at the end
坏味道 次要
文件最后应该包含一个空的新行
179、Files should contain only one top-level class or interface each
坏味道 主要
文件应该只包含一个顶级类或接口
180、Files should not be empty
坏味道 次要
删除空文件
181、Files should not have too many lines of code
坏味道 主要
源码文件代码行数检查
182、Modifiers should be declared in the correct order
坏味道 次要
Java语言规范建议按以下顺序列出修饰符:
1. Annotations
2. public
3. protected
4. private
5. abstract
6. static
7. final
8. transient
9. volatile
10. synchronized
11. native
12. strictfp
183、Sections of code should not be "commented out"
坏味道 主要
不要有注掉的代码,影响可读性,可以删除
184、Strings should not be concatenated using '+' in a loop
坏味道 次要
用StringBuilder代替String拼接
185、String function use should be optimized for single characters
坏味道 主要
字符串方法操作中单字符建议优先用单引号
186、Unused local variables should be removed
坏味道 次要
如果一个局部变量被声明但未被使用,那么它是死代码,应该被删除。 这样做会提高可维护性,因为开发人员不会想知道使用什么变量
187、The diamond operator ("<>") should be used
坏味道 次要
Java 7引入了操作符(<>)来减少泛型代码的冗长度。
List
188、Useless imports should be removed
坏味道 次要
不要导入没有用到的导入
189、Source files should not have any duplicated blocks
坏味道 主要
源文件不应有任何重复块
190、Only static class initializers should be used
坏味道 主要
静态代码块应加,static标识
191、Generic exceptions should never be thrown
坏味道 主要
通用异常如Error, RuntimeException, Throwable, and Exception不应抛出,应定义和抛出一个专门的异常,而不是使用通用异常
192、Method names should comply with a naming convention
坏味道 次要
方法名称应符合命名约定
默认规则:^[a-z][a-zA-Z0-9]*$
193、Synchronized classes Vector, Hashtable, Stack and StringBuffer should not be used
坏味道 主要
Java API的早期类,例如Vector,Hashtable和StringBuffer已被同步,使其成为线程安全的。 不幸的是,即使从单个线程使用这些集合,同步也会对性能产生很大的负面影响
194、Standard outputs should not be used directly to log anything
坏味道 主要
用日志记录代替标准输出
195、Local variable and method parameter names should comply with a naming convention
坏味道 次要
局部变量和方法参数名称应符合命名约定
默认:^[a-z][a-zA-Z0-9]*$
196、Local Variables should not be declared and then immediately returned or thrown
坏味道 次要
声明一个变量只是立即返回或抛出它是一个糟糕的做法
197、Instance methods should not write to "static" fields
坏味道 严重
静态属性更新需同步
198、Methods should not be empty
坏味道 严重
不要存在空方法
199、Utility classes should not have public constructors
坏味道 主要
帮助类不应该有公共构造函数,帮助类不宜实例化,且应该有一个如下的私有构造方法
private StringUtils() {
throw new IllegalStateException("Utility class");
}
200、Static non-final field names should comply with a naming convention
坏味道 次要
静态非最终字段名称应符合命名约定
默认:^[a-z][a-zA-Z0-9]*$
201、Methods returns should not be invariant
坏味道 阻断
方法返回值不应该是相同的值
202、Return of boolean expressions should not be wrapped into an "if-then-else" statement
坏味道 次要
可以根据boolean表达式就能返回的直接返回boolean表达式,不需要if-then-else语句
203、Try-with-resources should be used
坏味道 严重
Try-with-resources代替try-catch-finally
204、String literals should not be duplicated
坏味道 严重
重复的字符串文字会使重构过程容易出错,因为您必须确保更新所有
threshold
Number of times a literal must be duplicated to trigger an issue
默认值
3
205、
坏味道
可调整:
1、Abbreviation As Word In Name (默认 关闭)
坏味道 主要
检查验证标识符名称中的缩写(连续大写字母)长度,还允许执行骆驼案例命名
allowedAbbreviationLength 3
6、Annotation Location (默认 关闭)
坏味道 主要
注释位置
allowSamelineSingleParameterlessAnnotation
To allow single parameterless annotation to be located on the same line as target element.
默认值
true
allowSamelineParameterizedAnnotation
To allow parameterized annotation to be located on the same line as target element.
默认值
false
allowSamelineMultipleAnnotations
To allow annotation to be located on the same line as target element.
默认值
false
tokens
tokens to check
默认值
CLASS_DEF,INTERFACE_DEF,ENUM_DEF,METHOD_DEF,CTOR_DEF,VARIABLE_DEF
7、Annotation Use Style (默认 关闭)
坏味道 主要
trailingArrayComma
Defines the policy for trailing comma in arrays. Default is never.
closingParens
Defines the policy for ending parenthesis. Default is never.
elementStyle
Defines the annotation element styles. Default value is compact_no_array.
8、Artifact ids should follow a naming convention (默认 关闭)
坏味道 次要
共享命名约定允许团队有效协作。 当pom的artifactId与提供的正则表达式不匹配时,此规则引发了一个问题
regex
The regular expression the "artifactId" should match
默认值
[a-z][a-z-0-9]+
9、At-clause Order (默认 关闭)
坏味道 主要
检查从句顺序
tagOrder
allows to specify the order by tags.
默认值
@author,@version,@param,@return,@throws,@exception,@see,@since,@serial,@serialField,@serialData,@deprecated
target
allows to specify targets to check at-clauses.
10、Avoid Escaped Unicode Characters (默认 关闭)
坏味道 主要
避免转义的Unicode字符
allowIfAllCharactersEscaped
Allow if all characters in literal are escaped.
默认值
false
allowNonPrintableEscapes
Allow non-printable escapes.
默认值
false
allowByTailComment
Allow use escapes if trail comment is present.
默认值
false
allowEscapesForControlCharacters
Allow use escapes for non-printable(control) characters.
默认值
false
11、Avoid Nested Blocks (默认 关闭)
坏味道 主要
避免嵌套块
allowInSwitchCase
Allow nested blocks in case statements. Default is false.
12、Avoid Star Import (默认 关闭)
坏味道 次要
检查发现使用*符号的导入语句
excludes
packages where star imports are allowed. Note that this property is not recursive, subpackages of excluded packages are not automatically excluded.
allowStaticMemberImports
whether to allow starred static member imports like import static org.junit.Assert.*;
. Default is false.
默认值
false
allowClassImports
whether to allow starred class imports like import java.util.*;
. Default is false.
默认值
false
13、Boolean Expression Complexity (默认 关闭)
坏味道 主要
将嵌套布尔运算符(&&,||和^)限制为指定的深度(默认= 3)。
max
the maximum allowed number of boolean operations in one expression. Default is 3.
默认值
3
tokens
tokens to check. Default is LAND,BAND,LOR,BOR,BXOR.
默认值
LAND,BAND,LOR,BOR,BXOR
14、Branches should have sufficient coverage by tests (默认 关闭)
坏味道 主要
分支应有足够的测试覆盖
minimumBranchCoverageRatio
默认值
65
15、Catch Parameter Name (默认 关闭)
坏味道 主要
检查catch参数名是否符合format属性指定的格式
format
Specifies valid identifiers. Default is ^(e|t|ex|[a-z][a-z][a-zA-Z]+)$
默认值
^(e|t|ex|[a-z][a-z][a-zA-Z]+)$
16、Class Data Abstraction Coupling (默认 关闭)
坏味道 主要
度量衡量给定类中其他类的实例化数。
max
the maximum threshold allowed. Default is 7.
excludedClasses
User-configured class names to ignore.
excludeClassesRegexps
User-configured regular expressions to ignore classes
excludedPackages
User-configured packages to ignore
17、Class Fan Out Complexity (默认 关闭)
坏味道 主要
类的依赖类数量
max
the maximum threshold allowed. Default is 20.
excludedClasses
User-configured class names to ignore
excludeClassesRegexps
User-configured regular expressions to ignore classes
excludedPackages
User-configured packages to ignore
18、Class names should comply with a naming convention (开放)
坏味道 次要
类名应符合命名约定
format
Regular expression used to check the class names against.
默认值
^[A-Z][a-zA-Z0-9]*$
19、Classes from "sun.*" packages should not be used (开放)
坏味道 主要
不得使用“sun.*”软件包的类,sun类*或com.sun *包被视为实现细节,不属于Java API
Exclude
Comma separated list of Sun packages to be ignored by this rule. Example: com.sun.jna,sun.misc
20、Classes should not be coupled to too many other classes (Single Responsibility Principle) (默认 关闭)
坏味道 主要
类不应与太多其他类(单一责任原则)相耦合(依赖)
max
Maximum number of classes a single class is allowed to depend upon
默认值
20
21、Classes should not be too complex (默认 关闭)
坏味道 严重 废弃
类不应太复杂
max
Maximum complexity allowed.
默认值
200
22、Classes should not have too many "static" imports (默认 关闭)
坏味道 主要
静态导入类允许您使用其公共静态成员,而不必使用类名。 这可以很方便,但如果静态导入太多的类,你的代码可能会变得混乱,很难维护
threshold
The maximum number of static imports allowed
默认值
4
23、Classes should not have too many fields (默认 关闭)
坏味道 主要
类不应有太多字段
countNonpublicFields
Whether or not to include non-public fields in the count
默认值
true
maximumFieldThreshold
The maximum number of fields
默认值
20
24、Classes should not have too many methods (默认 关闭)
坏味道 主要
类不应该有太多方法
countNonpublicMethods
Whether or not to include non-public methods in the count.
默认值
true
maximumMethodThreshold
The maximum number of methods authorized in a class.
默认值
35
25、Close curly brace and the next "else", "catch" and "finally" keywords should be located on the same line (默认 关闭)
坏味道 次要
关闭大括号,下一个“else”,“catch”和“finally”关键字应位于同一行
26、Close curly brace and the next "else", "catch" and "finally" keywords should be on two different lines (默认 关闭)
坏味道 次要
关闭大括号和下一个“else”,“catch”和“finally”关键字应该在两个不同的行
29、Comments should not be located at the end of lines of code (默认 关闭)
坏味道 次要
注释不应位于代码行的末尾
legalTrailingCommentPattern
Description Pattern for text of trailing comments that are allowed. By default, comments containing only one word.
默认值
^\s*+[^\s]++$
30、Constant Name (默认 关闭)
坏味道 次要
检查常数名称是否符合指定的格式
applyToPackage
Controls whether to apply the check to package-private member
默认值
true
format
Regular expression
默认值
^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$
applyToPublic
Controls whether to apply the check to public member
默认值
true
applyToProtected
Controls whether to apply the check to protected member
默认值
true
applyToPrivate
Controls whether to apply the check to private member
默认值
true
31、Control flow statements "if", "for", "while", "switch" and "try" should not be nested too deeply (默认 关闭)
坏味道 严重
控制流程语句“if”,“for”,“while”,“switch”和“try”不能嵌套太深
max
Maximum allowed control flow statement nesting depth.
默认值
3
32、Custom Import Order (默认 关闭)
坏味道 主要
检查导入声明组按照用户指定的顺序显示。 如果有导入,但是在组态中未指定其组,则导入应放在导入列表的末尾。
thirdPartyPackageRegExp
RegExp for THIRDPARTY_PACKAGE group imports.
默认值
^$
separateLineBetweenGroups
Force empty line separator between import groups.
默认值
true
sortImportsInGroupAlphabetically
Force grouping alphabetically.
默认值
false
specialImportsRegExp
RegExp for SPECIAL_IMPORTS group imports.
默认值
^$
customImportOrderRules
List of order declaration customizing by user.
standardPackageRegExp
RegExp for STANDARD_JAVA_PACKAGE group imports.
默认值
java|javax
33、Cyclomatic Complexity (默认 关闭)
坏味道 主要
检查针对特定限制的方法的循环复杂性
switchBlockAsSingleDecisionPoint
whether to treat the whole switch block as a single decision point
默认值
false
max
the maximum threshold allowed.
默认值
10
tokens
tokens to check
默认值
LITERAL_WHILE,LITERAL_DO,LITERAL_FOR,LITERAL_IF,LITERAL_SWITCH,LITERAL_CASE,LITERAL_CATCH,QUESTION,LAND,LOR
34、Default Comes Last (默认 关闭)
坏味道 主要
检查在switch语句中的所有情况之后的默认值。
skipIfLastAndSharedWithCase
whether to allow default along with case if they are not last
默认值
false
35、Empty catch block (默认 关闭)
坏味道 主要
检查空的catch块。 有两个选项可以使验证更加精确(默认情况下,检查允许空的catch块和任何注释)
exceptionVariableName
Format of skipping exception''s variable name.
默认值
^$
commentFormat
Format of comment.
默认值
.*
36、Empty For Initializer Pad (默认 关闭)
坏味道 次要
检查初始化程序为空的填充; 那是空的是否需要一个空的初始化程序,或者禁止这样的空格。 示例:for(; i
option
policy on how to pad an empty for iterator
37、Empty For Iterator Pad
坏味道 次要
检查一个空的填充迭代器; 那就是空格是否需要一个空的迭代器,否则这样的空格是被禁止的。 示例:for(Iterator foo = very.long.line.iterator(); foo.hasNext();)
option
policy on how to pad an empty for iterator
38、Empty Line Separator (默认 关闭)
坏味道 主要
在标题,包,所有导入声明,字段,构造函数,方法,嵌套类,静态初始化器和实例初始化器之后检查空行分隔符
allowNoEmptyLineBetweenFields
Allow no empty line between fields
默认值
false
allowMultipleEmptyLines
Allows multiple empty lines between class members.
默认值
true
tokens
assignments to check
默认值
PACKAGE_DEF,IMPORT,CLASS_DEF,INTERFACE_DEF,ENUM_DEF,STATIC_INIT,INSTANCE_INIT,METHOD_DEF,CTOR_DEF,VARIABLE_DEF
allowMultipleEmptyLinesInsideClassMembers
Allow multiple empty lines inside class members
默认值
true
39、Executable Statement Count (默认 关闭)
坏味道 主要
将可执行语句的数量限制为指定的限制(默认= 30)。
max
the maximum threshold allowed. Default is 30.
默认值
30
tokens
members to check
默认值
CTOR_DEF,METHOD_DEF,INSTANCE_INIT,STATIC_INIT
40、Expressions should not be too complex (默认 关闭)
坏味道 严重
表达式不应太复杂
max
Maximum number of allowed conditional operators in an expression
默认值
3
42、File Length (默认 关闭)
坏味道 主要
如果源文件变得很长,那么很难理解。 因此,长类通常应该重构到专注于特定任务的几个单独的类中
fileExtensions
file type extension of files to process
max
maximum allowable number of lines. Default is 2000.
43、File Tab Character (默认 关闭)
坏味道 次要
检查源代码中没有制表符('\ t')
fileExtensions
file type extension of files to process
eachLine
whether to report on each line containing a tab, or just the first instance. Default is false.
44、Files should contain an empty new line at the end (默认 关闭)
坏味道 次要
文件最后应该包含一个空的新行
45、Add a private constructor to hide the implicit public one.
工具类不应该有默认或者公共的构造函数,也就是说这个类里可能方法都是static,那就不需要构造它的实例,因此应该给加一个private的构造函数,就不会报这个错了。
a class which only has private constructors should be final
例如上一个,加了private构造函数,又会出这个,把class设置成final即可。例:
public class Shape {
private Shape() {
/* set something here */
}
public static Shape makeShape(/* arglist */) {
System.out.println("here is the shape you ordered");
return (new Shape());
}
}
已调整:
1、String literals should not be duplicated (调整)
坏味道 严重
重复的字符串文字会使重构过程容易出错,因为您必须确保更新所有
threshold
Number of times a literal must be duplicated to trigger an issue
默认值
3 调整为 5
已关闭:
1、Utility classes should not have public constructors (关闭)
坏味道 主要
帮助类不应该有公共构造函数,帮助类不宜实例化,且应该有一个如下的私有构造方法
private StringUtils() {
throw new IllegalStateException("Utility class");
}
2、Methods returns should not be invariant (关闭)
坏味道 阻断
方法返回值不应该是相同的值
3、Return of boolean expressions should not be wrapped into an "if-then-else" statement (关闭)
坏味道 次要
可以根据boolean表达式就能返回的直接返回boolean表达式,不需要if-then-else语句
4、The diamond operator ("<>") should be used (关闭) jdk7+可用
坏味道 次要
Java 7引入了操作符(<>)来减少泛型代码的冗长度
5、Sections of code should not be "commented out" (关闭)
坏味道 主要
不要有注掉的代码,影响可读性,可以删除
6、Try-with-resources should be used (关闭) jdk7+可用
坏味道 严重
Try-with-resources代替try-catch-finally
7、Loops should not be infinite (关闭)
Bug 阻断
循环不应该是无限的
8、Credentials should not be hard-coded (关闭)
漏洞 阻断
凭证不应该硬编码
9、Anonymous inner classes containing only one method should become lambdas (关闭)
坏味道 主要
只有一个方法的匿名内部类应该变成lambdas
10、"throws" declarations should not be superfluous (关闭) 抛出运行时异常,有的框架接口即抛出此类异常
坏味道 次要
“抛出”声明不应该是多余的
11、IP addresses should not be hardcoded (关闭)
漏洞 次要
ip 地址不应该硬编码
12、"@Override" should be used on overriding and implementing methods (关闭)
坏味道 主要
重写的和实现在方法要加Override标注
13、An open curly brace should be located at the beginning of a line (关闭)
坏味道 次要
开放的大括号应位于一行的开头
14、Cognitive Complexity of methods should not be too high (关闭)
坏味道 严重
认知复杂度是衡量一种方法的控制流程难以理解的度量。 认知复杂性较高的方法难以维持。
Threshold
The maximum authorized complexity.
默认值