函数直接返回了一个集合

public class Person {private Set<Course> courses;public Set<Course> getCourses() {return courses;}public void setCourses(final Set<Course> courses) {this.courses = courses;}
}

重构：让这个函数返回该集合的一个只读副本，并在这个类中提供add/remove 集合元素的函数

public class Person {private Set<Course> courses = new HashSet<>();public Set<Course> getCourses() {return Collections.unmodifiableSet(courses);}public void addCourse(Course course) {this.courses.add(course);}public void removeCourse(Course course) {this.courses.remove(course);}
}

动机
    取值函数不该返回集合自身，因为这样会让用户得以修改集合内容而集合拥有者却一无所知。
这也会对用户暴露过多对象内部数据结构的信息。如果一个取值函数确实需要返回多个值，它应该
避免用户直接操作对象内所保存的集合，并隐藏对象内与用户无关的数据结构。