2

好的，所以我有一个方法需要接收一个充满 的数组ints，然后对照它的镜像检查它，看看它匹配的最大镜像是什么。所以例如我有 array [7, 1, 2, 9, 7, 2, 1]，它可以匹配的最大数组是 2，在[1, 2].

现在我把它分成3种方法。一个接受数组，另一个反转数组并返回它（mirrorArray）。第三个是计算匹配（groupCount）的数组的大小。这是我到目前为止所拥有的：

public int maxMirror(int[] nums) {
  int[] revArray = mirrorArray(nums);

  return groupCount(nums, revArray);
}

private int[] mirrorArray(int[] nums) {
  int[] newArray = new int[nums.length];

  for (int i = nums.length-1, j = 0; i >= 0; i--, j++) {
    newArray[j] = nums[i];
  }

  return newArray;
}

private int groupCount(int[] aFor, int[] bRev) {
  int maxCount = 0;
  int groupSize = 1;

  //get aFor value
  for (int i = 0; i < aFor.length; i++) {
    int[] tempA = Arrays.copyOfRange(aFor, 0, groupSize);

    //loop through bRev and check for matches
    for (int j = 0; j < bRev.length; j++) {
      int[] tempB = Arrays.copyOfRange(bRev, j, j+groupSize);

      if (Arrays.equals(tempA, tempB)) {
        maxCount = tempA.length;
      }
    }

    groupSize++;
  }
  return maxCount;
}

它在某处的第 3 种方法中失败（返回 1 而不是 2），我很困惑为什么我的循环没有返回我想要的。任何帮助将不胜感激。

4

4 回答 4

5

好吧我很好奇...

这是问题所在：

int[] tempA = Arrays.copyOfRange(aFor, 0, groupSize);

您总是将 tempB 与 aFor 长度的第一个子数组进行比较groupSize。将该行更改为

int[] tempA = Arrays.copyOfRange(aFor, i, i + groupSize);

它应该可以工作。

编辑让失败案例不断出现.. groupSize 的增量位置似乎存在问题

   while (groupSize < aFor.length) {
      //get aFor value
      for (int i = 0; i < aFor.length; i++) {
        int[] tempA = Arrays.copyOfRange(aFor, i, i + groupSize);

        //loop through bRev and check for matches
        for (int j = 0; j < bRev.length; j++) {
          int[] tempB = Arrays.copyOfRange(bRev, j, j+groupSize);

          if (Arrays.equals(tempA, tempB)) {
            maxCount = groupSize;
          }
        }
      }
      groupSize++;
  }

这不是最有效的，它可能是一个有趣的优化练习。一种起始方法是从 groupSize 开始aFor.length并递减。一旦maxCount被分配，您可以提前返回。

编辑 2

 int groupSize = aFor.length;
 while (groupSize >= 0) {
      //get aFor value
      for (int i = 0; i <= aFor.length - groupSize; i++) { // note this change
        int[] tempA = Arrays.copyOfRange(aFor, i, i + groupSize);

        //loop through bRev and check for matches
        for (int j = 0; j <= bRev.length - groupSize; j++) { // note this change
          int[] tempB = Arrays.copyOfRange(bRev, j, j+groupSize);

          if (Arrays.equals(tempA, tempB)) {
            return groupSize;
          }
        }
      }
      groupSize--;
  }
  return 1;
}

发生的事情是 Arrays.copyOfRange 用零填充超出范围的数字。我还添加了我之前提到的提前退出选项。可能还有更多的优化可以做

于 2013-05-21T12:49:49.700 回答
2

您之间的逻辑tempA和tempB有问题：

在第一个循环（原始）的每次迭代中，您选择 0->groupSize 值集并与反向数组中的所有相似大小序列进行精确比较，因此第一次迭代是

Orignal array (aFor) : [7, 1, 2, 9, 7, 2, 1]
Reverse array (bRev) : [1, 2, 7, 9, 2, 1, 7]
Iteration-1:
tempA=> [7]
tempB=> [1],[2],[7]...
maxCount = 1; (in first iteration, multiple single value matche)

Iteration-2:
tempA=> [7,1]
tempB=> [1,2],[2,7]...
maxCount = 1; (no match of [7,1], maxCount continues from first iteration)

同样，在所有其他迭代中，由于您的输入数据集，不会找到序列匹配。

现在，如果您将输入更改为 [7, 1, 2, 9, 7, 1, 7]，maxCount 将为 2

对于输入 [7, 1, 2, 9, 2, 1, 7]，maxCount 为 7

但这是由于您输入的性质和代码中的问题。

代码中的问题是外部循环（aFor循环）不会针对每个序列集进行评估，即在迭代 2 中，您只检查大小为 2 的第一组（[7,1]）和进一步的组（[1,2]，[ 2,9]..) 永远不会进行比较，因此由于之前的匹配，您总是会得到 maxCount=1。

于 2013-05-21T13:11:16.657 回答
1

我知道这似乎与问题无关，但我试图在不创建新数组（就地比较）的情况下进行测试，我希望这会有所帮助：

public static int getMaxMirrorSub(int[] arr) {
    for (int eqLength = arr.length; eqLength >= 0; eqLength--) {
        for (int arrayStart = 0; arrayStart < arr.length; arrayStart++) {
            for (int mirrorStart = arr.length - 1; mirrorStart >= eqLength - 1; mirrorStart--) {
                int indexArray = arrayStart, indexMirror = mirrorStart;
                while(indexArray < arr.length 
                        && indexMirror >= 0 
                        && arr[indexArray] == arr[indexMirror]){

                    indexArray++; indexMirror--;
                }

                if (indexArray - arrayStart == eqLength)
                    return eqLength;
            }
        }
    }
    return 0;
}
于 2013-05-21T15:26:29.923 回答
0 
public int maxMirror(int[] nums) {
  int[] reverse = null;
  int max = 0;
  for(int i = 0; i < nums.length; i++) {
     for(int k = 0; k < nums.length; k++) {
        if(i > k) { 
          } else {
            reverse = reverseSection(nums, i, k);
            if(searchFor(reverse, nums)) {
               if(reverse.length > max) { max = reverse.length; }
            }
         }
     }
  }

  return max;
}

public int[] reverseSection(int[] nums, int begin, int end) {
   int[] nArray = new int[end - begin + 1];
   int numbs = end - begin;
   for(int i = begin; i <= end; i++) {
      nArray[numbs] = nums[i];
      numbs--;
   }
   return nArray;
}

public boolean searchFor(int[] reverse, int[] nums) {
    int index = 0;
    boolean check = false;
        for(int i = 0; i < nums.length; i++) {
        if(nums[i] == reverse[index]) {
            index++;

            if(index >= reverse.length) {
                return true;
            }
        } else {
            index = 0;
        }
    }
    return false;
}
于 2016-04-19T16:59:58.337 回答