Question

每当我尝试运行代码时，我都会遇到越界错误。有谁知道它有什么问题吗？我似乎无法弄明白。

public class Swapper{

    /**
    This method swaps the first and second half of the given array.
    @param values an array
     */

    public void swapFirstAndSecondHalf(int[] values) {
        // your work here

        int[] first = new int[values.length/2];
        int[] second = new int[values.length/2];
        for(int i = 0; i < values.length / 2; i++) {
            second[i] = values[i];
        }
        for (int j = values.length / 2; j < values.length; j++) {
            first[j] = values[j];
        }
        for(int k = 0; k < values.length / 2; k++) {
            values[k] = first[k];
        }
        for(int l = values.length / 2; l < values.length; l++) {
            values[l] = second[l];
        }
    }

    // This method is used to check your work
    public int[] check(int[] values) {
        swapFirstAndSecondHalf(values);
        return values;
    }
}

Answer 1

int[] first = new int[values.length/2];

所以索引[0..values.length/2 - 1]对first有效。

for (int j=values.length/2; j<values.length; j++)
{
    first[j] = values[j];
}

因此，j的第一个值为values.length/2，它已经超出界限。

您需要练习调试，放置断点并在执行时跟踪代码。

Answer 2

您可以使用System.arraycopy()代替所有for循环。

public static void main(String[] args) throws Exception {
    int[] values = {1, 2, 3, 4, 5};
    values = swapFirstAndSecondHalf(values);
    System.out.println(Arrays.toString(values));

    values = new int[]{1, 2, 3, 4, 5, 6};
    values = swapFirstAndSecondHalf(values);
    System.out.println(Arrays.toString(values));
}

public static int[] swapFirstAndSecondHalf(int[] values) {
    boolean evenSize = values.length % 2 == 0;
    int half = values.length / 2;
    int[] swapper = new int[values.length];
    System.arraycopy(values, evenSize ? half : half + 1, swapper, 0, half);
    System.arraycopy(values, 0, swapper, evenSize ? half : half + 1, half);

    // The middle number stays the middle number
    if (!evenSize) {
        swapper[half] = values[half];
    }
    return swapper;
}

结果：

[4, 5, 3, 1, 2]
[4, 5, 6, 1, 2, 3]

如果您想要中间数字（对于奇数大小的数组）成为下半部分的一部分，那么swapFirstAndSecondHalf()将如下所示：

public static int[] swapFirstAndSecondHalf(int[] values) {
    boolean evenSize = values.length % 2 == 0;
    int half = values.length / 2;
    int[] swapper = new int[values.length];
    System.arraycopy(values, half, swapper, 0, evenSize ? half : half + 1);
    System.arraycopy(values, 0, swapper, evenSize ? half : half + 1, half);
    return swapper;
}

结果：

[4, 5, 3, 1, 2]
[4, 5, 6, 1, 2, 3]

Answer 3

分配新阵列是浪费空间。只需将两半原地交换：

public static void swapFirstAndSecondHalf(int[] values) {
    final int len = values.length / 2;
    final int offset = values.length - len;
    for (int i = 0; i < len; i++) {
        int temp = values[i];
        values[i] = values[offset + i];
        values[offset + i] = temp;
    }
}

代码允许奇数长度，并且只留下中心值。

完成一个交换整数数组的前半部分和后半部分的方法

3 个答案: