Java课程管理系统循环中变量未初始化报错问题求助

我正在使用Java开发课程管理系统，需检查用户是否已选某门课程。以下是我的代码：

boolean duplicateCheck;

for (int i=0; i<enrolledCourses.size(); i++) {
    if (enrolledCourses.get(i) == searchResult.get(Integer.parseInt(userInput)-1)) {
        duplicateCheck = true;
        break;
    } else {
        duplicateCheck = false;
    }
}

if (duplicateCheck) {
    System.out.println("You've already enrolled in this course!");
} else {
...
}

但在第二个if语句处出现报错：

The local variable duplicateCheck may not have been initialized

1. 局部变量未初始化：Java语法强制要求局部变量在被使用前必须完成赋值。如果enrolledCourses是空集合，循环体根本不会执行，duplicateCheck就始终处于未赋值状态，编译器会直接抛出该错误。
2. 逻辑冗余：当前代码在循环的else分支中重复赋值false，这完全没必要——只要默认状态是“未选这门课”，仅在找到匹配课程时修改状态即可。
3. 对象比较风险：用==比较课程对象时，实际比较的是内存引用地址，而非课程的实际内容（比如课程ID、名称），可能导致误判。

方案1：基础修复（解决报错+优化逻辑） #

先给duplicateCheck赋初始值，再简化循环逻辑：

// 初始化为false，默认用户未选该课程
boolean duplicateCheck = false;
// 提前取出目标课程，避免循环中重复调用方法
Course targetCourse = searchResult.get(Integer.parseInt(userInput)-1);

for (int i=0; i<enrolledCourses.size(); i++) {
    // 对象比较用equals()，需确保Course类正确重写了equals方法
    if (enrolledCourses.get(i).equals(targetCourse)) {
        duplicateCheck = true;
        break;
    }
}

if (duplicateCheck) {
    System.out.println("You've already enrolled in this course!");
} else {
    // 执行选课逻辑
}

方案2：用增强for循环简化代码 #

增强for循环让代码更易读，避免索引操作：

boolean duplicateCheck = false;
Course targetCourse = searchResult.get(Integer.parseInt(userInput)-1);

for (Course course : enrolledCourses) {
    if (course.equals(targetCourse)) {
        duplicateCheck = true;
        break;
    }
}

方案3：Java 8+ 进阶简化（Stream API） #

用Stream的anyMatch方法可以一行完成检查，代码更简洁：

Course targetCourse = searchResult.get(Integer.parseInt(userInput)-1);
boolean duplicateCheck = enrolledCourses.stream()
                                        .anyMatch(course -> course.equals(targetCourse));

if (duplicateCheck) {
    System.out.println("You've already enrolled in this course!");
} else {
    // 执行选课逻辑
}

内容的提问来源于stack exchange，提问作者Xiaolong Guo