我正在尝试编写一个小程序,在数组中打印出不同的数字.例如,如果用户输入1,1,3,5,7,4,3,程序将只打印1,3,5,7,4.

我在函数checkDuplicate中的else上遇到错误.

到目前为止,这是我的代码：

import javax.swing.JOptionPane;

public static void main(String[] args) {
    int[] array = new int[10];
    for (int i=0; i<array.length;i++) {
        array[i] = Integer.parseInt(JOptionPane.showInputDialog("Please enter"
                                  + "an integer:"));
    }
    checkDuplicate (array);
}

public static int checkDuplicate(int array []) {
    for (int i = 0; i < array.length; i++) {
        boolean found = false;
        for (int j = 0; j < i; j++)
            if (array[i] == array[j]) {
                found = true;
                break;
            }
        if (!found)
            System.out.println(array[i]);
    }
    return 1;
}
}

解决方法:

首先,“else if”语句是不正确的,因为你没有为if提供任何条件(如果你想要if,你需要写“if(condition)…”).

其次,你不能在内部循环中决定是否应该打印一个值：代码的工作方式你为每个值数组[j]写一个值数组[i],它不同于array [i]！

第三：内部循环只需要从0到外部索引i-1：对于每个元素,您只需要决定它是否是第一次出现(即,如果在任何先前的索引处发生相同的值).如果是,打印出来,如果没有,请忽略它.

CheckDuplicate()的正确实现将是：

public static void checkDuplicate(int array []) {
  for (int i = 0; i < array.length; i++) {
    boolean found = false;
    for (int j = 0; j < i; j++)
      if (array[i] == array[j]) {
        found = true;
        break;
      }
    if (!found)
      System.out.println(array[i]);
  }
}

但是当然,对于更大的阵列来说,某种Set会更有效…

编辑：当然,mmyers(参见注释)是正确的,因为CheckDuplicate()不返回任何值,它应该返回类型void(而不是int).我在上面的代码中纠正了这一点……