递归,在每一步导出信息

时间:2015-03-10 23:28:33

标签: java arrays recursion linked-list combinations

我正在处理输出字符串数组的所有组合的代码。 我可以使用doCombine方法打印数组,但是当将该数组添加到LinkedList时,它会添加仅包含D的数组。如何将每个数组传递到我的列表?

public LinkedList<String[]> allCombinations = new LinkedList<String[]>();

public static void main(String[] args) {
    String[] arr = {"A", "B", "C", "D"};

    Combinations c = new Combinations();

    c.combine(arr, 0);
    c.combine(arr, 1);
    c.combine(arr, 2);
    c.combine(arr, 3);
    c.combine(arr, 4);
    c.printList();

}

private void combine(String[] arr, int r) {
    String[] res = new String[r];
    doCombine(arr, res, 0, 0, r);
}

private void doCombine(String[] arr, String[] res, int currIndex, int level, int r) {
    if (level == r) {

        printArray(res);
        //Here it doesn't work
        allCombinations.add(res);
        return;
    }
    for (int i = currIndex; i < arr.length; i++) {
        res[level] = arr[i];
        doCombine(arr, res, i + 1, level + 1, r);

        if (i < arr.length - 1 && arr[i].equals(arr[i + 1])) {
            i++;
        }
    }
}

private void printArray(String[] res) {

    for (int i = 0; i < res.length; i++) {
        System.out.print(res[i] + " ");
    }
    System.out.println();
}

public void printList() {
    for (int i = 0; i < allCombinations.size(); i++) {
        printArray(allCombinations.get(i));
    }

}

1 个答案:

答案 0 :(得分:1)

我假设allCombinationsList<String[]>(LinkedList)的实现。

我认为您的错误是当您执行allCombinations.add(res)时,您在res中添加了对象List的引用,但此对象在add之后被修改

因此,当您在res之前打印add时,会打印当前值,但是,在列表中,您有对同一对象res的N引用,该对象最终为一个'D'系列(当你打印列表时,你打印N次res的最后一个值)

添加时必须克隆列表的当前值,它将起作用:

private void doCombine(String[] arr, String[] res, int currIndex, int level, int r) {
    if (level == r) {

        printArray(res);
        //Here it doesn't work
        allCombinations.add(res.clone());
        return;
    }
    for (int i = currIndex; i < arr.length; i++) {
        res[level] = arr[i];
        doCombine(arr, res, i + 1, level + 1, r);

        if (i < arr.length - 1 && arr[i].equals(arr[i + 1])) {
            i++;
        }
    }
}