Java中泛型类继承非泛型类是否规范？含代码优化与类型选择疑问

原始代码 #

public abstract class Course<T extends Student> {
    private String name;
    private int months;
    private int capacity;
    private static List<Student> students = new ArrayList<>();

    public Course(String name, int months, int capacity) {
        this.name = name;
        this.months= months;
        this.capacity = capacity;
    }

    public int getCapacity() {
        return capacity;
    }

    public List<Student> getStudents() {
        return Collections.unmodifiableList(students);
    }

    protected void addStudent(Student student){
        students.add(student);
    }

    public abstract void enroll(T student);
}

public class Math extends Course<Student> {

    public Math(String name, int months, int capacity) {
        super(name, months, capacity);
    }

    @Override
    public void enroll(Student student) {
        System.out.println(student.getClass());
        if (student instanceof MathBackground) {
            if (this.getCapacity() <= this.getStudents().size()) {
                System.out.println("The course is already full. We will add you to the waitlist");
            } else {
                this.addStudent(student);
                System.out.println("The student " + student.getName() + " has been enrolled");
            }
        } else{
            System.out.println("Can not enroll the student " + student.getName() + " due to the background");
        }
    }
}

疑问解答 #

1. 让泛型Course类被非泛型子类继承是否为不良实践？ #

不算不良实践，但要看场景是否合理。当前Math extends Course<Student>相当于把泛型约束放宽到所有Student子类，反而浪费了父类泛型的编译期类型检查能力。如果子类的注册逻辑只针对特定类型的学生（比如Math只收MathBackground学生），更合理的做法是把泛型参数指定为对应类型，这样能在编译阶段就拦截不符合的输入，比运行时用instanceof更安全可靠。

2. 如何优化Math课程，使其仅允许实现MathBackground接口的学生注册？ #

从编译期约束和逻辑正确性两个层面优化：

• 修复static成员问题：Course类里的students是static的，这会导致所有Course子类实例共享同一个学生列表（比如Math和Economics的学生混在一起），必须去掉static修饰符，改为实例成员：

  private List<Student> students = new ArrayList<>();
  

• 编译期类型约束：将Math类的泛型参数改为MathBackground，这样enroll方法的参数直接限定为MathBackground类型，无需再做instanceof判断：

  public class Math extends Course<MathBackground> {
  
      public Math(String name, int months, int capacity) {
          super(name, months, capacity);
      }
  
      @Override
      public void enroll(MathBackground student) {
          if (getCapacity() <= getStudents().size()) {
              System.out.println("The course is already full. We will add you to the waitlist");
          } else {
              addStudent(student);
              System.out.println("The student " + student.getName() + " has been enrolled");
          }
      }
  }
  

这样既保证了只有符合背景的学生能注册，又避免了运行时类型判断的冗余。

3. 声明对象时，使用父类类型Course mathClass = new Math()还是子类类型Math mathClass = new Math()更合适？ #

分场景选择：

• 如果代码只用到Course抽象类定义的通用方法（比如enroll、getCapacity），优先用Course mathClass = new Math()。这符合面向抽象编程的原则，降低代码耦合，后续如果需要替换成其他Course子类（比如把Math换成AdvancedMath），调用代码几乎不用改动。
• 如果代码需要调用Math类特有的方法（比如后续给Math新增了getMathScoreAverage()这类方法），必须用Math mathClass = new Math()，因为父类类型无法访问子类特有的成员。

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