Question

所以我花了相当多的时间来理解我的代码有什么问题。我有一个示例程序，我比较我的，这是有效的。我的代码结构不同（这是我的教授所要求的一种方法），而不是示例（使用两种方法）。我应该创建一个递归的，分而治之的解决方案来计算int数组中的反转。

我很遗憾为什么示例程序在整个递归过程中保持对输入数组的操作，而我的则没有。我知道Java是按值传递的，所以我很困惑为什么这个例子有效。任何帮助我理解这些解决方案的差异将不胜感激！谢谢！

包含两种方法的示例代码 - merge和invCounter：

public static long merge(int[] arr, int[] left, int[] right) {
  int i = 0, j = 0, count = 0;
  while (i < left.length || j < right.length) {
     if (i == left.length) {
        arr[i+j] = right[j];
        j++;
     } else if (j == right.length) {
        arr[i+j] = left[i];
        i++;
     } else if (left[i] <= right[j]) {
        arr[i+j] = left[i];
        i++;
     } else {
        arr[i+j] = right[j];
        count += left.length-i;
        j++;
     }
  }
  return count;
}

//the recursive function
public static long invCounter(int[] arr) {
  int sum = 0;

  if (arr.length < 2)
     return 0;

  int m = (arr.length + 1) / 2;
  int left[] = Arrays.copyOfRange(arr, 0, m);
  int right[] = Arrays.copyOfRange(arr, m, arr.length);
  sum += invCounter(left);
  sum += invCounter(right);
  sum += merge(arr, left, right);
  return sum;
}

我的单方法实施（尝试）：

public static int invCounter(int ranking[]) {
  int sum = 0;
  int result[] = new int[ranking.length];
  int resIndx = 0;

  if (ranking.length < 2) {
     return 0; //base case
  }

  //divide
  int left[] = Arrays.copyOfRange(ranking, 0, ranking.length/2);
  int right[] = Arrays.copyOfRange(ranking, ranking.length/2,                    
   ranking.length);
  sum += invCounter(left);
  sum += invCounter(right);

  int i = 0, j = 0;
  while (i < left.length || j < right.length) {
     if (i == left.length) {
        //i empty, just add j
        result[resIndx++] = right[j++];
     }
     else if (j == right.length) {
        //j empty, just add i
        result[resIndx++] = left[i++];
     }
     else if (right[j] < left[i]) {
        //inversion 
        result[resIndx++] = right[j++];
        sum += left.length - i;
     }
     else {
        //no inversion
        result[resIndx++] = left[i++];
     }
  }

  ranking = Arrays.copyOf(result, result.length);

  return sum;
}

为什么示例程序能够通过递归来维护更新的数组而不是我的？

更新（2015年10月22日）：所以我发现如果我用result替换ranking并且只是直接修改这个数组，我就能得到正确的结果。我现在的问题是为什么我不能使用结果数组临时存储结果，然后将它们复制到最后的排名（参数）数组中？在我看来，它会像在前面放置值那样做同样的事情，但如果我在最后更改它，则不会反映ranking的更改。

Answer 1

您的方法不会修改排名参数，而是创建一个新的int数组（结果），然后您可以对其进行处理。尝试直接在排名数组上设置值，而不是在结果数组上设置值，或者只是将结果变量设置为排名。

public static int invCounter(int ranking[]) {
  int sum = 0;
  int result[] = ranking;
//other code...

编辑：或者您可以复制它的内容，但不能复制Arrays.copyOf，因为它首先创建一个新数组然后复制到它中。使用System.arrayCopy代替复制到EXISTING数组：

System.arrayCopy(result, 0, rankings, 0, result.length();

Java数组操作和递归

1 个答案: