2

我想在构造函数中使用类的 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
    }
}

输出：

订购 8 x Souvlaki

异常捕获：数字必须是正数

线程“主”java.lang.IllegalArgumentException 中的异常：MyClass.main(MyClass.java:40) 的 MyClass.setString(MyClass.java:23) 的字符串不能为空。(MyClass.java:10)

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

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

4

3 回答 3

4

正如评论所暗示的那样，这可能存在问题。特别是，您可能想看看构造函数中可覆盖的方法调用有什么问题？. 底线大致是：有人可能会set...以意想不到的方式覆盖方法，并引用类的其他（未初始化的）字段，这可能会导致各种错误。

专用验证方法可能是一种选择。但是这些可能会被多次调用，即使没有必要的验证。

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

“设计和文件继承或禁止它”

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

于 2015-03-15T11:29:28.137 回答
0

checkInvariant()您可以在类中创建一个方法来验证所有字段，而不是在构造函数中进行验证。

class MyClass {
    private int num;
    private String value;

    public void checkInvariants() {
        assertNotEmpty(value, "String value cannot be empty");
        assertPositive(num, "Number num should be non-negative");
    }
}

然后在其他地方，您可以将此类的实例作为参数传递，首先调用此方法以确保不变量成立：

class SomeOtherClass {
    public void doSomethingWithMyClass(MyClass myClass) {
        myClass.checkInvariants();
        // Proceed with work.
    }
}
于 2015-03-15T09:58:21.917 回答
0

您的变量可以在类中的任何位置访问，因此无需使用 mutator 方法来初始化您的变量。

如果您想对输入参数进行一些验证，请使用另一种方法来执行所需的所有验证。

在构造函数中调用验证方法。

于 2015-03-15T10:45:10.737 回答