【问题标题】:Eliminating Out of Bounds error in Java sorting comparison消除 Java 排序比较中的越界错误
【发布时间】:2013-03-09 16:28:02
【问题描述】:

我目前正在实现 InsertionSort 和 MergeSort 来比较它们的操作,我之前已经实现了 MergeSort,但我正在尝试我当前的算法教科书中的版本。我在 cmets 标记的行(第 58 行)上遇到了一个数组越界错误,一旦它越过它,我假设也可能在第 60 行。这是书中 T 的算法,但我找不到我的错误来自哪里。代码包含简单的主要测试方法,包含完整代码,因此它是可复制和可运行的。

package algsortcomp;


public class AlgSortComp {


public static void InsertionSort(int[] data){
    //Sorts given array by Insertion Sort
    //Input: an array data of ints
    //Output: array data sorted in nondecreasing 
    int v, j;
    for (int i = 1; i<data.length; i++){
        v = data[i];
        j = i - 1;
        while (j >= 0 && data[j] > v){
            data[j+1] = data[j--];
        }
        data[j+1]=v;
    }
}

public static void MergeSort(int[] data){
    //Sorts array data by recursive mergesort
    //Input: an array data of ints
    //Output: array data sorted in nondecreasing order
    int dlen = data.length;
    int mid = dlen/2;

    int[] b = new int[mid];
    int[] c = new int[dlen-mid];

    if (dlen > 1){
       System.arraycopy(data, 0, b, 0, mid-1);
       System.arraycopy(data, mid, c, 0, dlen-mid);
       MergeSort(b);
       MergeSort(c);
       Merge(b, c, data);
   }         
}

public static void Merge(int[] b, int[] c, int[] data){
    //Merges two sorted arrays into one sorted array
    //Input: Arrays b and c both sorted
    //Output: Sorted array data of elements b and c.
    int p = b.length, q = c.length, dlen = data.length;
    int i=0, j=0, k=0;
    while (i<p && j<q){
        if (b[i] <= c[j])
            data[k] = b[i++];
        else
            data[k]=c[j++];
        k++;
    }
    if (i == p)
        System.arraycopy(c, j, data, k, dlen-1); //<---currently getting Array out of bounds here, tried replacing with
    else                                         //dlen-k and error is gone but array returned is all [0,0,...,0]
        System.arraycopy(b, i, data, k, dlen-1);
}



public static void main(String[] args) {
    int [] test = {3, 5, 23, 56, 1, 12, 32};
    String arraystr = java.util.Arrays.toString(test);
    System.out.println(arraystr);

    //InsertionSort(test);
    MergeSort(test);
    System.out.println(java.util.Arrays.toString(test));
}

}

班上还没有人起床,所以我请各位谦虚的人多多注意一下,以发现我的错误。提前致谢。

编辑:我也忘记了如何仅通过 Arrays.toString() 调用 java.util.Arrays.toString()。我尝试导入 java.util.Arrays 但这没有用,而且我承认已经有一段时间没有接触 java/arrays 了。

【问题讨论】:

  • 欢迎来到 Stack Overflow!要求人们发现代码中的错误并不是特别有效。您应该使用调试器(或添加打印语句)来隔离问题,方法是跟踪程序的进度,并将其与您期望发生的情况进行比较。一旦两者发生分歧,你就发现了你的问题。 (然后如果有必要,你应该构造一个minimal test-case。)
  • 为什么不用像eclipse这样的IDE来调试代码?
  • 我只有 netbeans 并且从未使用过调试器功能(如果有的话)。 netbeans 是否也具有此功能,还是我需要 eclipse?
  • 是的,Netbeans 也有调试功能。只需使用 Google 进行少量研发,您就会成功!

标签: java arrays sorting mergesort


【解决方案1】:

System.arraycopy 将要复制的元素数作为最后一个参数。在这里,您要从 c 中复制剩余项目的数量,应该是 q-j。同样,下一个案例的 p-i。

你是import java.util.Arrayspackage行之后吗?

另外,将 System.arraycopy(data, 0, b, 0, mid-1); 更改为

System.arraycopy(data, 0, b, 0, mid);

【讨论】:

  • 当我这样做时,我得到与尝试 dlen-k 相同的结果,数组最终为 [0, 0, 0, 0, 0, 0, 32]
  • 将此System.arraycopy(data, 0, b, 0, mid-1); 设为`System.arraycopy(data, 0, b, 0, mid);`
  • 也改为 p-i 仍然得到 [0, 0, 0, 0, 0, 0, 32] 作为最终数组。
  • [1, 3, 5, 12, 23, 32, 56]!!谢谢你,那个 mid-1 就是这样做的。谢谢。
  • 是的,我做到了,就在你这么说之前:P +1 不管那是什么价值。
猜你喜欢
  • 2014-06-03
  • 2014-07-06
  • 2016-04-12
  • 2022-01-13
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2013-10-13
  • 2013-04-02
相关资源
最近更新 更多