0

我不明白为什么代码不将唯一元素复制到另一个数组。这是我的代码。我虽然==是复制元素，但我得到了一个错误，所以我改用了=。

 public static void main(String[] args) {
    // TODO Auto-generated method stub
    int[] B = new int[15];
    B[2] = 2;
    B[3] = 3;
    B[4] = 4;
    B[5] = 5;
    B[6] = 6;
    B[7] = 7;
    B[8] = 8;
    B[9] = 9;
    B[10] = 10;

    int[] check = new int[B.length];
    for (int i = 0; i < B.length; i++) {
        for (int x = 0; x < check.length; x++) {
            if (B[i] != check[x]) {
                check[x] = B[i];
            }
        }
    }

    for (int i = 0; i < check.length; i++) {
        System.out.println(check[i]);
    }
}
4

3 回答 3

2

您将所有内容循环两次，而应该只循环一次。

目前它看起来像这样：

for (int i = 0; i < B.length; i++) {
    for (int x = 0; x < B.length; x++) {
        if (B[i] != check[x]) {
            check[x] = B[i];
        }
    }
}

这意味着when i = 0 , then x=0, 1, 2, 3, 4, etc.. .那么when i = 1, x=0,1,2,3.... , etc.

所以最后一次运行将在i=14哪里B[i] = 0。
所以对于每个check[x]，它都不等于0。

你想要做的是在一条线上处理它。i因此，x您可以i像这样使用外循环而不是 2 个变量。这意味着您只是在比较B[1] to check[1] and B[2] to check[2]，依此类推。

像这样：

for (int i = 0; i < B.length; i++) {
    if (B[i] != check[i]) {
        check[i] = B[i];
    }
}
于 2013-04-06T14:08:26.917 回答
1

==用于相等测试，=用于赋值，

顺便说一句，用于System.arraycopy(xxx)复制数组。

public int[] findUnique(int[] data) {
    int[] unique = new int[data.length];
    int x = 0;
    for (int i = 0; i < data.length; i++) {
        boolean uni = true;
        for (int j = i + 1; j < data.length; j++) {
            if (data[i] == data[j]) {
                uni = false;
                break;
            }
        }
        if (uni) {
            unique[x++] = data[i];
        }
    }
    return unique;
}
于 2013-04-06T14:00:01.380 回答
0

System.arrayCopy()更快

于 2013-04-06T14:01:04.947 回答