Mutable fields should not be "public static"
可变字段不应该是public static
不合规
public interface MyInterface {
public static String [] strings; // Noncompliant
}
public class A {
public static String [] strings1 = {"first","second"}; // Noncompliant
public static String [] strings2 = {"first","second"}; // Noncompliant
public static List<String> strings3 = new ArrayList<>(); // Noncompliant
// ...
}合规
private static final String [] COLUMN_NAMES = new String[]{"date","customerNumber","customerName",
"account","emailAdress","mobilePhoneNumber","emailStatus"};
public static List<String> getColumnNames() {
return Collections.unmodifiableList(Arrays.asList(COLUMN_NAMES));
}问题分析
请注意，使可变字段（例如数组）为final将阻止重新分配变量，但这样做不会影响数组内部状态的可变性
可以改变的类型如果非常有必要声明为 ‘public static’ 可以通过上述 Collections.unmodifiableList 方式实现。
"public static" fields should be constant
-
public static 属性应是常量
合规代码
public class Greeter {
public static Foo foo = new Foo();
public static String realmOplate = "this is name";
...
}
不合规代码
public class Greeter {
public static final Foo FOO = new Foo();
...
}
private static String realmOplate = "this is name";
public static String getRealmOplate() {
return realmOplate;
}
public static void setRealmOplate(String realm) {
realmOplate = realm;
}分析
在大多数情况下，public static 在多个对象之间共享状态的一种期望。但是使用这种方法，任何对象都可以在共享状态下做任何想做的事情，例如将其设置为null。
只有设置为了final时才会防止共享值被修改。
Class variable fields should not have public accessibility
-
类中的成员属性不应该共有访问。
不合格代码
public class MyClass {
public static final int SOME_CONSTANT = 0; // Compliant - constants are not checked
public String firstName; // Noncompliant
}
合规代码
public class MyClass {
public static final int SOME_CONSTANT = 0; // Compliant - constants are not checked
private String firstName; // Compliant
public String getFirstName() {
return firstName;
}
public void setFirstName(String firstName) {
this.firstName = firstName;
}
}
问题分析
公共类变量字段不遵守封装原理，并且具有三个主要缺点：
无法添加其他行为，例如验证。
内部表示形式已公开，以后不能更改。
成员值可能会在代码中的任何位置发生更改，并且可能不符合程序员的假设。
通过使用私有属性和访问器方法（设置和获取），可以防止未经授权的修改。
Invoke method(s) only conditionally
不合规代码
logger.log(Level.DEBUG, "Something went wrong: " + message);
// Noncompliant; string concatenation performed even when log level too high to show DEBUG messages合规代码
logger.log(Level.DEBUG, "Something went wrong:{} "，message)
分析
日志在输出的情况下支持一个字符串输出，没有必要通过+生成多个字符串，浪费性能。
"toString()" and "clone()" methods should not return null
不合规代码
public String toString () {
if (this.collection.isEmpty()) {
return null; // Noncompliant
} else {
// ...
@Override
public Object clone() {
try {
return (ContractQuery) super.clone();
} catch (CloneNotSupportedException e) {
}
return null;
}合规
public String toString () {
if (this.collection.isEmpty()) {
return "";
} else {
// ...
@Override
public Object clone() throws CloneNotSupportedException {
return (ContractQuery) super.clone();
}
分析
当toString clone 返回Null时，容易产生空指针问题。
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
不合规代码
public final static String HOME= "home";
合规代码
public static final String HOME = "home";
分析
应根据约定俗称的规范进行编码，代码不仅仅是给机器用，但却是给人看的。写给机器的能用的不叫本事，写出人人都看的懂的才是能耐。
"@Deprecated" code should not be used
所以声明过期的方法，类都有与之对应的代替方式。
反射中常用的
//newInstance() 已过期
ContractDto contract = contractClass.newInstance();
//代替方式
contractClass.getDeclaredConstructor().newInstance()Parentheses should be removed from a single lambda input parameter when its type is inferred
lambda 表达式中一个输入参数没有必要加括号
不合规代码
(x) -> x * 2
合规代码
x -> x * 2
总结
代码编程长路漫漫，吾将上下而求索。使用sonar改进代码，漂亮的代码令人向往，犹如一位漂亮精致的姑娘，让人目不转睛，爱不释手。
做个有追求的程序员还是有必要的。
头条号
微信号
来源：oschina
链接：https://my.oschina.net/u/3247419/blog/3195204