Question

我正在尝试获取一个整数数组的链表，表示长度为N的位串的可能排列，例如N = 2

00 01 10 11

我成功地将代码表示为字符串，如下所示：

public static LinkedList<String> printBin(String soFar, int iterations) {
    if(iterations == 0) {
        LinkedList<String> ret = new LinkedList<String>();
        ret.add(soFar);
        return ret;
    }else {
        LinkedList<String> ret = new LinkedList<String>();
        ret.addAll(printBin(soFar + "0", iterations - 1));
        ret.addAll(printBin(soFar + "1", iterations - 1));
        return ret;
    }
}

但是我尝试将此代码转换为使用整数数组，因此N = 2我得

11 11 11 11

下面的SSCE：

public class Main {
    public static void main(String[] args){
        int numberOfBits = 2;
        LinkedList<int []> results = printBin(new int[numberOfBits], 0, numberOfBits);
        Iterator<int[]> i = results.iterator();
        while(i.hasNext()){
            int[] temp = i.next();
            for(int j = 0; j < temp.length; j++){
                System.out.print(temp[j]);
            }
            System.out.println("");
        }
    }


    public static LinkedList<int[]> printBin(int[] soFar, int counter, int iterations) {
        if(iterations == 0) {
            LinkedList<int[]> ret = new LinkedList<int[]>();
            ret.add(soFar);
            return ret;
        }else {
            LinkedList<int[]> ret = new LinkedList<int[]>();
            int[] soFar1 = soFar; soFar1[counter] = 0;
            int[] soFar2 = soFar; soFar2[counter] = 1;
            ret.addAll(printBin(soFar1, counter + 1, iterations - 1));
            ret.addAll(printBin(soFar2, counter + 1, iterations - 1));
            return ret;
    }
}

}

非常感谢任何帮助。

Answer 1

你做不到

    int[] soFar1 = soFar; soFar1[counter] = 0;
    int[] soFar2 = soFar; soFar2[counter] = 1;

它会导致sofar1和sofar2指向同一个数组。你需要创建一个新数组并将其复制到它。

int[] soFar1=(int[])soFar.clone(); soFar1[counter]=0;
int[] soFar2=(int[])soFar.clone(); soFar2[counter]=1;

Answer 2

您的代码的问题在于它不断在每个调用级别上重用相同的数组soFar实例，而不是复制。如果您将代码更改为

int[] soFar1 = (int[])soFar.clone(); soFar1[counter] = 0;
int[] soFar2 = (int[])soFar.clone(); soFar2[counter] = 1;

您的代码应该生成您想要的结果（demo on ideone）。

但是，最好将所有整数从0迭代到(1 << numberOfBits) - 1，并打印其二进制表示形式：

LinkedList<int[]> ret = new LinkedList<int[]>();
int endMask = 1 << numberOfBits;
for (int mask = 0 ; mask != endMask ; mask++) {
    int[] combo = new int[numberOfBits];
    for (int i = 0 ; i != numberOfBits ; i++) {
        combo[i] = ((mask & (1 << i)) != 0) ? 1 : 0;
    }
    ret.add(combo);
}

这是demo of iterative method on ideone。

Answer 3

建议：不要在参数中使用数组，使用返回值，自下而上

public static LinkedList<LinkedList<int>> printBin( int bits) {
    if(bits == 0) {
        LinkedList<LinkedList<int>> ret = new LinkedList<LinkedList<int>>();
        ret.add(new LinkedList<Int>());
        ret.get(0).add(0);
        ret.get(0).add(1);
        return ret;
    }else {
        LinkedList<LinkedList<int>> ret = printBin(bits-1);
        LinkedList<LinkedList<int>> ret2 = printBin(bits-1); // this should be deep copied
        for(LinkedList<int> bitstring : ret){
            bitstring.add(0); 
        }
        for(LinkedList<int> bitstring : ret2){
            bitstring.add(1); 
        }

        ret.addAll(ret2);
        return ret;
}

使用递归在Java中打印长度为N的位串的排列

3 个答案: