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沒有問題。