Java的空参数时链构造

Question

比方说，我有多个构造一类，其中之一是一个拷贝构造函数（复制对象）：

public class Rectangle { 

    int width, height; 

    public Rectangle(int width, int height) { 
     this.width = width; 
     this.height = height; 
    } 

    public Rectangle(Rectangle source) { 
     this(source.width, source.height); 
    } 
} 

有没有什么办法可以让检查source是null在复制构造函数中，并且如果它是？IllegalArgumentException？因为其他构造函数调用已将作为我构造函数中的第一个语句。

Answer 1

你可以这样做：

public Rectangle(Rectangle source) { 
    this(checkNotNull(source, "Source cannot be null").width, source.height); 
} 

private static <T> T checkNotNull(T t, String msg) { 
    if (t == null) throw new IllegalArgumentException(msg); 
    return t; 
} 

我也乔恩斯基特同意，NullPointerException是不是在这种情况下，一个坏bevahiour。唯一的问题是，当你得到一个NPE时，可能有点难以确定哪个对象是null，这就是为什么更具体的消息可能有用。

也不能推倒重来，并使用标准java.util.Objects方法，如果你不打扰投掷NullPointerException代替：

public Rectangle(Rectangle source) { 
    this(Objects.requireNonNull(source, "Source cannot be null").width, source.height); 
} 

如果你的错误信息是昂贵的打造，你可以提供一个Supplier<String>相反，支付只在实际上是需要它消息的建设成本：

public Rectangle(Rectangle source) { 
    this(Objects.requireNonNull(source,() -> explainError(source)).width, source.height); 
} 

Answer 2

是的，你可以使用一个辅助方法，如果需要的话，这将引发异常，并返回原来的价值行吟诗人rwise ...你可以在你的构造函数调用中调用它，因为你允许方法调用作为参数评估的一部分。

// In a helper class 
public static <T> T checkNotNull(T value) { 
    if (value == null) { 
     throw new IllegalArgumentException(); 
    } 
    return value; 
} 

然后用它作为：

public Rectangle(Rectangle source) { 
    this(Helper.checkNotNull(source).width, source.height); 
} 

但是......我相信，NullPointerException是推荐的例外反正这里扔（有效的Java第二版为例），其现有的代码将会抛出。所以你很可能不要想对你现有的代码做任何改变。

如果你想为这样的检查的辅助方法，但很高兴它扔NullPointerException，我推荐使用番石榴和其Preconditions类，它有这个和其他有用的检查方法很多。

另请注意，Java 1.7引入了java.util.Objects它有requireNonNull，所以你甚至不需要第三方库。

Answer 3

一个文本诀窍是将构造函数的初始化移动到方法。然后，你可以有你才想要的任何代码：

public class Rectangle { 

    int width, height; 

    public Rectangle(int width, int height) { 
     init(width, height); 
    } 

    public Rectangle(Rectangle source) { 
     if (source == null) { 
      throw new IllegalArgumentException("source can't be null!"); 
     } 
     init(source.width, source.height); 
    } 

    private void init(int width, int height) { 
     this.width = width; 
     this.height = height; 
    } 
} 

Answer 4

你可以这样做

int width, height; 

public Rectangle(int width, int height) { 
    this.width = width; 
    this.height = height; 
} 

public Rectangle(Rectangle source) { 
    if(source != null) { 
     width = source.width; 
     height = source.height; 
    } 
} 

Answer 5

如果你确实想抛出一个IllegalArgumentException，我认为最干净的解决方案是使用一个静态方法，而不是一个构造函数：

public static Rectangle from(Rectangle source) { 
    if (source == null) { 
     throw new IllegalArgumentException("source can't be null!"); 
    } 
    return new Rectangle(source.width, source.height); 
} 

或者你可以只添加复制方法：

public Rectangle copy() { 
    return new Rectangle(this.width, this.height); 
} 

我更喜欢后者，因为它不需要关心自己可能为空的Rectangle。请注意，如果您将这与一个空对象一起使用，这将导致一个NPE，这可能进一步表明NPE没有问题。