Java数组删除会带来意想不到的结果吗？

Question

以下代码是程序的一部分，该程序采用数组ar的值（变量z除外）并将其复制到另一个名为ar2的数组中。 结果应为除负数2（19、1、17、17）以外的所有数字，但当前结果为19 1 17 17 -2 19 1 17 17 -2 19 1 17 17 -2 19 1 17 17 -2。

public class Second_tiny {

    public static void main(String[] args) {
        int[] ar = { 19, 1, 17, 17, -2 };
        int z = ar[0];

        for (int i = 0; i < (ar.length); i++) {
            if (z > ar[i]) {
                z = ar[i];

            }
        }
        // second pass
        int[] ar2 = new int[ar.length];
        int zero = 0;
        for (int x = 0; x < (ar.length); x++) {
            if (ar[x] == z) {
                continue; // If it is equal to z go back to the loop again

            }
            ar2[zero++] = ar[x];

            for (int i = 0; i < ar.length; i++) {
                System.out.println(ar[i]);
            }
            /*
             * //2nd pass copy all items except smallest one to 2nd array int[] ar2= new int[ar.length-1]; int curIndex = 0; for (i=0; i<ar.length; i++) { if (ar[i]==z) continue; ar2[curIndex++] =
             * ar[i]; }
             */

        }

    }
}    

Answer 1

Java 8方式：

        int[] ar = { 19, 1, 17, 17, -2 };
        int min = Arrays.stream(ar).min().getAsInt();
        int[] ar2 = Arrays.stream(ar).filter(s -> s!=min).toArray();
        System.out.println(Arrays.toString(ar2));

Answer 2

您在此块中将原始阵列打印4次：

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

该块应该在循环之外，并且应该引用ar2而不是ar 。

for (int x = 0; x < (ar.length); x++) {
    if (ar[x] == z) {
        continue; // If it is equal to z go back to the loop again
    }

    ar2[zero++] = ar[x];
}

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

这将为您提供以下结果：

19
1
17
17
0

最后一个0出现是因为0是int的默认值。 您的ar2 5个元素，对于最后一个元素，永远不会替换默认值。

Answer 3

您可以使用Math.min(int, int)来确定最小值。 您的第二个数组应该小一个元素。 我建议您避免删除多个值。 您可以使用Arrays.toString(int[])来打印第二个array 。 就像是，

int[] ar = { 19, 1, 17, 17, -2 };
int z = ar[0];
for (int i = 1; i < ar.length; i++) {
    z = Math.min(z, ar[i]);
}
// second pass
int[] ar2 = new int[ar.length - 1];
boolean first = true;
for (int x = 0; x < ar.length; x++) {
    if (ar[x] == z && first) {
        first = false;
        continue; // If it is equal to z go back to the loop again
    }
    int y = x - (!first ? 1 : 0);
    ar2[y] = ar[x];
}
System.out.println(Arrays.toString(ar2));

输出是

[19, 1, 17, 17]

Answer 4

您的代码错误是对的，您给了一个额外的for循环。我对for循环做了注释。使用以下代码，您将获得所需的输出。

公共类HelloWorld {

    public static void main(String[] args) {
        int[] ar = { 19, 1, 17, 17, -2 };
        int z = ar[0];

        for (int i = 0; i < (ar.length); i++) {
            if (z > ar[i]) {
                z = ar[i];

            }
        }
        // second pass
        int[] ar2 = new int[ar.length];
        int zero = 0,i=0;

        for (int x = 0; x < (ar.length); x++) {
            if (ar[x] == z) {
                continue; // If it is equal to z go back to the loop again

            }
            ar2[zero++] = ar[x];

          //  for (int i = 0; i < ar.length; i++) {
                System.out.println(ar[i]);
                i++;
          //  }
            /*
             * //2nd pass copy all items except smallest one to 2nd array int[] ar2= new int[ar.length-1]; int curIndex = 0; for (i=0; i<ar.length; i++) { if (ar[i]==z) continue; ar2[curIndex++] =
             * ar[i]; }
             */

        }

    }
}    

输出为：

19

1

17

17