我的循环中的逻辑不正确

Question

public static void main(String[] args) {
        Integer[] courses1 = {1,2,3};
        Integer[] courses2 = {0,1};

        Integer[] longestArr;
        Integer[] shortestArr;
        ArrayList commonCourses = new ArrayList();

        if ( courses1.length > courses2.length)
        {
            longestArr = courses1; shortestArr = courses2;
        }    
        else
        {    
            longestArr = courses2; shortestArr = courses1;
        }

        for ( int i : longestArr)
        {
            for ( int j : shortestArr)
            {
                if (i == j);
                   commonCourses.add(i); 
            }
        } 
        Collections.sort(commonCourses);
        System.out.println(commonCourses.size());


}

无论 course1 和 course2 的值是什么，当它应该只包含两个数组中的元素时，arrayList commonCourses 的大小总是比两个数组中的元素总数大一倍。

我有 2 个问题，为什么将 2 个数组中的元素添加到 arrayList，为什么 arrayList 的大小总是比元素总数大一？

如果您想知道为什么我一开始就声明了课程 1 和课程 2，这是我在 Eclipse 上测试的来自 TalentBuddy.co 的问题。我用不同的起始条件进行了测试，总是发生同样的事情。

我的新瘦身解决方案

public static void main(String[] args) {
        Integer[] courses1 = {1};
        Integer[] courses2 = {0};

        ArrayList <Integer>commonCourses = new ArrayList<Integer>();

        for ( int i : courses1)
        {
            for ( int j : courses2)
            {
                if (i == j)
                   commonCourses.add(i); 
            }
        } 
        Collections.sort(commonCourses);
        System.out.println(commonCourses.size());     
}

Answer 1

你的问题是这一行的分号：

if (i == j);

if 子句到此结束，始终执行以下行。

Answer 2

if 条件末尾的分号是您的问题。有关为什么会发生这种情况的详细说明，请参阅this StackOverflow post. 简而言之，分号导致编译器在运行时忽略条件。

for ( int j : shortestArr)
{
    if (i == j);
    //         ^  Remove this semi-colon
    commonCourses.add(i); 
}

使用花括号来格式化 if 语句有助于防止这种情况发生：

if(condition){
    doSomething();
}

Answer 3

 for ( int j : shortestArr)
 {
      if (i == j);
              // ^ REMOVE THIS SEMICOLON
           commonCourses.add(i); 
 }

问题是多余的分号。您的代码块现在在功能上与此相同：

 for ( int j : shortestArr)
 {
      if (i == j)
      {
      }

      commonCourses.add(i); 
 }

Answer 4

if 条件后面有一个误用的分号。

if (i == j);

Answer 5

我从未在一家公司工作过，如果：

if (...) {
   ...
}

而且原因很明显。

让我展示最抽象的解决方案：

• 课程是一组唯一的课程编号
• 普通课程是一个交叉集
• 在 java 中，使用抽象接口（如 Set）作为变量，但可以使用不同的实现类来实现该集合

• 想要一个有序集合，TreeSet就是这样一个实现

  Set<Integer> courses1 = new HashSet<>();
  Collections.addAll(courses1, 1, 2, 3);
  
  SortedSet<Integer> courses2 = new HashSet<>();
  Collections.addAll(courses2, 0, 1);
  
  SortedSet<Integer> commonCourses = new TreeSet<>(courses1);
  commonCourses.retainAll(courses2);