我的類的構造函數不起作用

Question

我有以下ArrayIntList類，其構造函數定義如下。在最後一個構造函數中，我想要布爾型，如果爲true，則使用該特定元素實例化一個新對象。如果設置爲false，它應該實例化一個具有這麼多容量的新對象。請看我的意思是在這裏的客戶端代碼。它在布爾值爲true時有效。

類文件：

public class ArrayIntList { 
    private int[] elementData; // list of integers 
    private int size;   // current number of elements in the list 

    public static final int DEFAULT_CAPACITY = 100; 

    // post: constructs an empty list of default capacity 
    public ArrayIntList() { 
     this(DEFAULT_CAPACITY); 
    } 

    // pre : capacity >= 0 (throws IllegalArgumentException if not) 
    // post: constructs an empty list with the given capacity 
    public ArrayIntList(int capacity) { 
     if (capacity < 0) { 
      throw new IllegalArgumentException("capacity: " + capacity); 
     } 
     elementData = new int[capacity]; 
     size = 0; 
    } 

    //takes input list and adds to arrayIntList 
    public ArrayIntList(int[] elements) { 
     this(Math.max(DEFAULT_CAPACITY,elements.length*2)); 
     for (int n: elements){ 
     this.add(n); 
     } 
    } 

    //creates an arrayIntlist with data of element 
    public ArrayIntList(int element,boolean notCapacity) { 
     this(); 
     if (notCapacity) { 
     add(element); 
     } 
    //returns the totalCapacity NOT SIZE 
    public int getCapacity() { 
    return elementData.length; 
    } 
} 

客戶端代碼：

public class ArrayIntListExample { 
    public static void main(String[] args) { 
     // Create a new list and add some things to it. 
     ArrayIntList list = new ArrayIntList(); 

     //*** here is my question about ****// 
     ArrayIntList list1 = new ArrayIntList(2, false);//should give [] with capacity of two 
     ArrayIntList list2 = new ArrayIntList(2, true);//should give [2] 
     //*** ****************************** ****// 
     int[] array={2,3,4,5}; 
     ArrayIntList list3 = new ArrayIntList(array); 
     list.add(12); 
     list.add(3); 
     list.add(3); 
     System.out.println("list = " + list); 
     System.out.println("list1 = " + list1); 
     System.out.println("list2 = " + list2); 
     System.out.println("list2 = " + list3); 
     System.out.println("capacity of list1" + list1.getCapacity());//prints 100 but it must be 2 
    } 
} 

Answer 1

這應該工作，但是我覺得你的API是混亂..

public ArrayIntList(int element,boolean startElement) { 
    this(startElement ? 1 : element); 
    if (startElement) { 
    add(element); 
    } 
} 

我想你應該刪除此構造函數，而如果用戶想要一個包含特定元素的列表，則讓用戶執行new ArrayIntList(new int[] { 2 })。

Answer 2

要想讓它表現你想要的方式，我想你想傳遞element給構造與孤int參數：

public ArrayIntList(int element, boolean notCapacity) { 
    this(element); 
    if (notCapacity) { 
     add(element); 
    } 
} 

以前你只是調用this()，這與初始化數組默認容量。如果您手動或通過調試程序遍歷代碼，則可以看到發生這種情況。

還有其他問題，您的設計，但是。除了一個笨拙和令人困惑的接口/ API（通過你的構造函數）之外，數組的容量（即它所能容納的元素的總數量）和它的大小（即元素的數量數組）。

編輯

一個爲什麼你的API是混淆的原因是，你有以下情形：

------------------------------------------------------------------- 
Constructor   | int | boolean | Behavior 
-----------------------+-----+---------+--------------------------- 
(element)    | 2 | x | Array with capacity 2 
(element, notCapacity) | 2 | true | Array with one element (2) 
(element, notCapacity) | 2 | false | Array with capacity 2 
-----------------------+-----+---------+--------------------------- 

• 你必須做同樣的事情的兩個方面。
• 您有一個boolean參數被賦予一個負面的名稱。 「容量是錯誤的」比「不容錯誤的」容易。
• 在引入不一致和混淆的同時，您引入了具有有限值的特徵（使用任意值的一個元素初始化數組）。本質上，notCapacity服務的唯一一點是區分你想要的兩種行爲（初始化容量與初始化一個任意值的元素）。
• element有兩個，非常不同含義：容量與單個元素被添加到陣列。
• 將false更改爲true足以從構造函數中調用非常不同的行爲。
• 構造函數中的布爾值通常是不透明的，價值有限。從看到像new ArrayIntList(5, true)這樣的調用，意圖是否明顯？

Answer 3

如果按照通過代碼很清楚爲什麼行爲發生。

public ArrayIntList(int element,boolean notCapacity) { 
    this(); 
    if (notCapacity) { 
    add(element); 
    } 
} 

這個方法將創建的DEFAULT_CAPACITY一個新的數組，如果它的虛假它只是完成的方法，並返回沒有別的做。雖然，我強烈建議你重新考慮你的階級結構

public ArrayIntList(int element,boolean notCapacity) { 
    this(); 
    if (notCapacity) { 
    add(element); 
    } else { 
    elementData = new int[element]; 
    } 
} 

：

最簡單的解決辦法簡單地增加一個else語句是這樣。

Answer 4

讓我們看一下你的構造：

public ArrayIntList(int element,boolean notCapacity) { 
    this(); 
    if (notCapacity) { 
    add(element); 
    } 
} 

在這條線現在讓我們看看：

this(); 

這條線將調用構造函數不帶參數。該構造是這樣的：

public ArrayIntList() { 
    this(DEFAULT_CAPACITY); 
} 

因此，這將調用帶一個容量的構造，它會通過DEFAULT_CAPACITY給它。所以，加入一些評論原文：

public ArrayIntList(int element,boolean notCapacity) { 

    this(); // initializes this object with a capacity of 100 and no elements 

    if (notCapacity) { 
    add(element); // if notCapacity is true, add the element 
    } 
} 

正如你所看到的，無處你實際使用的「元素」變量，如果notCapacity是假的（這意味着它應該是容量）。

一個非常簡單的此修訂可能是：

public ArrayIntList(int element,boolean notCapacity) { 

    this(notCapacity ? DEFAULT_CAPACITY : element); 

    if (notCapacity) { 
    add(element); 
    } 
} 

不過，我認爲一個更好的設計是沒有這個構造函數的一切，而不是提供下列靜態方法：

public static ArrayIntList createWithElement(int element) { 
    ArrayIntList ret = new ArrayIntList(); 
    ret.add(element); 
    return ret; 
} 

然後調用者有一個乾淨清晰的方法來調用以創建一個具有1個元素的列表。