java - 在构造函数中使用 set 方法来初始化类的字段是一种好习惯吗？

Question

我想在构造函数中使用类的 set 方法来检查要初始化的值，如果它们不符合我设置的约束则抛出异常。

代码示例:

public class MyClass {

    // Fields
    private int number;
    private String string;

    // Constructor
    public MyClass(int number, String string) {
        setNumber(number);
        setString(string);
    }

    // Set Methods
    public void setNumber(int number) {
        if (number<=0) {    // Certain constrain for number
            throw new IllegalArgumentException("Number must be positive");
        }
        this.number = number;
    }

    public void setString(String string) { // Certain constrain for string
        if (string.equals("")) {
            throw new IllegalArgumentException("string cannot be empty");
        } 
        this.string = string;
    }

    public String toString() {
        return String.format("Ordered %d x %s%n", number, string);
    }

    public static void main(String[] args) {
        MyClass obj = new MyClass(8, "Souvlaki");   // Everything allright
        System.out.println(obj);
        try {
            MyClass obj2 = new MyClass(-3, "Mousaka");  // Error in number argument
        } catch (IllegalArgumentException exception) {  // catch the exception
            System.out.printf("Exception Caught: Number must be positive%n%n");
        }
        MyClass obj2 = new MyClass(4, "");  // Error in string argument
        // Allow the exception to end program execution
    }
}

输出:

Ordered 8 x Souvlaki

Exception Caught: Number must be positive

Exception in thread "main" java.lang.IllegalArgumentException: string cannot be empty at MyClass.setString(MyClass.java:23) at MyClass.(MyClass.java:10) at MyClass.main(MyClass.java:40)

输出正是我想要的。创建的第一个对象使用适当的值进行初始化。调用 toString() 方法隐式证明了这一点。第二个和第三个对象由于初始化错误而抛出异常。捕获第一个异常是为了让程序继续执行。第二个异常没有被捕获，以便输出打印出异常的错误信息。

所以一切似乎都是正确的，但这是一种好的编程技术还是它隐藏了一些错误？

Answer 1

正如评论所暗示的那样，这可能存在问题。特别是，您可能想看看 What's wrong with overridable method calls in constructors? .底线大致是:有人可能会以意想不到的方式覆盖 set... 方法，并引用该类的其他(未初始化的)字段，这可能会导致各种错误。

专用验证方法可能是一种选择。但是这些可能会被多次调用，即使在不需要验证的情况下也是如此。

您可以通过将 set... 方法设为 final 来缓解大部分问题。无论如何，这是一个很好的做法。正如 Joshua Bloch 在他的书“Effective Java”第 17 项中所说:

“为继承而设计和记录，否则禁止”

这意味着您应该使每个方法final，除非您明确希望允许它被覆盖，并且记录它应该如何被覆盖(或者，或者，使整个类 final)。