Findbugs不检测由用户定义的类公开内部表示

Question

[INFO] path.getInsertDate() may expose internal representation by returning Ttt.insertDate path.Ttt] At Ttt.java:[line 119]

public class Test {
    public String test;
}

public class ExposingTest { 
    private Test test;

    //No warning here.
    public Test getTest() {
         return test;
    } 
}

Answer 1

Dr Findbugs为预定义的类集生成此警告。

FindBugs的目的是警告开发人员代码的潜在问题。Findbugs不检查不变性。

可变类本身并不是一个bug或问题。可变类可能是一个有效的设计选择。

public class Person {
    private Department dep;

    public void setDepartment(Department dep) {
        this.dep = dep;
    }
}

public class Department { 
    private String name;
    public void setName(String name) {
        this.name = name;
    }
}

static boolean mutableSignature(String sig) {
    return sig.equals("Ljava/util/Hashtable;") || 
            sig.equals("Ljava/util/Date;") ||
            sig.equals("Ljava/sql/Date;") ||
            sig.equals("Ljava/sql/Timestamp;") ||
            sig.charAt(0) == '[';
}

因此，Findbugs将为返回以下内容的方法生成这种类型的警告（内部表示暴露）：

• 哈希表；
• 日期(java.util.date、java.sql.date、java.sql.timestamp)；
• 数组

Findbugs甚至不为列表和集合生成此警告。