java - for循环没有运行到最后？

Question

我正在做一项练习，需要一种方法来总结数据中所有电话号码的所有持续时间(即类似于来电显示)。我已经设法总结了持续时间，但并未打印所有数据。我一直试图弄清楚为什么会这样，但我无法理解它。任何帮助将不胜感激。下面是代码:

//initial code provided
     public static void main(String[] args) {
          String[] phoneNumbers = new String[100];
          int[] callDurations = new int[phoneNumbers.length];
          int size = 0;

          size = addCall(phoneNumbers, callDurations, size, "555-555-5555", 137);
          size = addCall(phoneNumbers, callDurations, size, "555-555-0000", 12);
          size = addCall(phoneNumbers, callDurations, size, "555-555-1234", 26);
          size = addCall(phoneNumbers, callDurations, size, "555-555-8888", 10);
          size = addCall(phoneNumbers, callDurations, size, "555-555-8888", 10);
          size = addCall(phoneNumbers, callDurations, size, "555-555-7777", 10);

       }


       public static int addCall(String[] phoneNumbers, int[] callDurations, int size, String newNumber, int newDuration) {
          if (size >= phoneNumbers.length) {
             System.out.println("Error adding " + newNumber + ": array capacity exceeded.");
          } else {
             phoneNumbers[size] = newNumber;
             callDurations[size] = newDuration;
             size++;
          }

          return size;
       }

//the portion of code that I'm trying to write

        public static void totalDurations(String[] phoneNumbers, int[] 
        callDurations, int size) {
             String[] copyNum = new String[phoneNumbers.length];  
             int[] copyDur = new int[phoneNumbers.length];  
             int newSize = size;  
             int pos = 1;  //counter for next available empty cell
             copyNum[0] = phoneNumbers[0];
             copyDur[0] = callDurations[0];

             for (int i = 0; i < newSize; i++){
               for (int j = 1; j < size; j++){
                 if (copyNum[i] != phoneNumbers[j]){
                   copyNum[i+pos] = phoneNumbers[j];
                   pos++;
                 }
                 else {
                   copyDur[i] += callDurations[j];
                   newSize = newSize -1;
                 }
               }          
             System.out.println(copyNum[i] + ":" + copyDur[i]+ "s");
             }
           }

我当前的输出

Total Durations:
555-555-5555:137s
555-555-0000:12s
555-555-1234:26s
555-555-8888:20s

Answer 1

这是一些非常丑陋的代码，您肯定需要在调试器中运行它，但我很清楚错误出在该行

newSize = newSize - 1;

这会减少外部 for 循环中使用的变量。如果您只完成了六个元素中的四个，则意味着为其中两个输入了 else 子句。也许您打算减少 size 来代替？

想想你想在这里做什么。使用调试器，以您意想不到的方式查看哪里出了问题。纠正它们。