10

我有一个公共构造函数，它需要一个参数（int age）来创建一个对象。我想检查传递的参数是否合法，例如年龄不能为负。如果它非法，则不要创建对象/实例。如果合法，没问题。

我只能想到一种方法来做到这一点 -

将构造函数设为私有。创建一个带有参数 (int age) 的静态方法来执行所有检查，如果您传递一个非法值，则返回 null。如果你传递一个合法的值，那么创建一个对象并返回它的引用。还有其他方法吗？也许来自构造函数本身？

编辑： 我想到了上述方法的一个问题。由于显而易见的原因，工厂方法/对象创建者方法只能是静态方法。如果工厂方法必须访问成员变量（进行一些检查）来创建对象会发生什么？然后，我们将被迫将该成员变量设为静态。这可能并非在所有情况下都可以。

是否有意义 ？

4

4 回答 4

9

还有其他方法吗？也许来自构造函数本身？

是的。我建议Exception从构造函数中抛出一个

public class Person
{


    int age;
    public Person(int age) throws Exception
    {
       if(age <= 0)
       {

          throw new Exception("Age is not allowed");
       }
       // Do some stuffs
       this.age = age;
    }

}

编辑：

您也可以IllegalArgumentException按照 Till Helge Helwig 的建议使用

public class Person
{


    int age;
    public Person(int age) throws IllegalArgumentException
    {
       if(age <= 0)
       {

          throw new IllegalArgumentException("Age is not allowed");
       }
       // Do some stuffs
       this.age = age;
    }

}
于 2013-01-18T07:56:22.840 回答
6

考虑这个例子，这是 java.util.HashMap 实现

public HashMap(int initialCapacity, float loadFactor) {
    if (initialCapacity < 0)
        throw new IllegalArgumentException("Illegal initial capacity: " +
                                           initialCapacity);
    if (initialCapacity > MAXIMUM_CAPACITY)
        initialCapacity = MAXIMUM_CAPACITY;
    if (loadFactor <= 0 || Float.isNaN(loadFactor))
        throw new IllegalArgumentException("Illegal load factor: " +
                                           loadFactor);

    // Find a power of 2 >= initialCapacity
    int capacity = 1;
    while (capacity < initialCapacity)
        capacity <<= 1;

    this.loadFactor = loadFactor;
    threshold = (int)(capacity * loadFactor);
    table = new Entry[capacity];
    init();
}

更多内容Effective Java 2nd Edition, Item 38: Check parameters for validity由 Joshua Bloch 撰写，他也是上述代码的作者

于 2013-01-18T08:19:50.960 回答
3

为此目的使用静态工厂会更好。因为从构造函数中抛出异常并不是一个好主意。

public class Person
{     
    public static Person newPerson(int age) /* throws SomeException -- if you want */ {
        if (age <= 0 || age >= 150) {
           return null; // or throw an Exception - it is how you want   
        }
        return new Person(age);
    }

    private Person(int age) {
        // assign age to field value
    }
}
于 2013-01-18T08:05:17.437 回答
1

Exception如果参数非法，则抛出。

public Test(int age) throws IllegalArgumentException {
    if(age<0)
        throw new IllegalArgumentException(...);
    this.age = age;
}
于 2013-01-18T07:59:37.260 回答