1

运行此代码会在该行给我一个数组越界异常:

int sum = array[k]+array[l]; //sum of l and k

...应该是一个简单的修复,但我不知道是什么原因造成的,因为我正在使用 array.length 来绑定循环。任何人都可以帮忙吗?

PS 作为记录,这段代码应该在一个 int 数组中搜索与目标 int 相等的 int 对或单个 int。它仅使用 println 工作,但我试图将加起来到目标的数字放入向量中。

public Vector<Vector<Integer>> subsetSum(int[] array, int target) {
    //creates vectors, adds inner vector to another vector
    outer = new Vector<Vector<Integer>>();
    inner = new Vector<Integer>();
    outer.add(inner);

    for (int k = 0; k <= array.length; k++) {
        for (int l = 0; l <= array.length; l++) {
            int sum = array[k]+array[l]; //sum of l and k
            int i = 0;

            if (sum == target) {

                inner.add(i, array[l]);
                inner.add(i, array[k]);

                i++;

                //prints combination
                System.out.println(array[l]+"+"+array[k]+"="+target);
            }
            if (k == target) {
                inner.add(i, array[k]);
                i++;
                //prints if int equals target
                System.out.println(k+"="+target);
            }
            if (l == target) {
                inner.add(i, array[l]);
                i++;
                //prints if int equals target
                System.out.println(l+"="+target);
            }
        }
    }
    //return combinations that add up to target in vector form
    System.out.println(outer);
    return outer;
}
4

3 回答 3

7

您需要在 for 循环中使用“<”而不是“<=”。

由于数组中的第一个位置是 0,所以最后一个位置是 length-1。发生的情况是,当您到达最后一次迭代时,索引已经超出了数组的范围。

例如,如果您有一个数组:

array = [0,1,2,3] 最后一次迭代将是 array[4],即数组的长度,超出范围。

于 2011-10-19T20:57:18.530 回答
2

<= 应该替换为 <

于 2011-10-19T20:58:22.257 回答
0

将循环更改为:

for (int k = 0; k < array.length; k++)

for (int l = 0; l < array.length; l++)

由于数组是基于 0 的,因此您希望比长度小 1。

于 2011-10-19T20:58:51.647 回答