0

目前我有这个代码：

public final class Tutor {
private String name;
private final Set<Student> tutees;
public Tutor(String name, Student[] students){
    this.name = name;
     tutees = new HashSet<Student>();
     for (int i = 0; i<students.length; i++)
         tutees.add(students[i]);
}

我正在尝试重写它（只是在纸上），以便它制作/添加学生的防御性副本，而不是直接将它们添加到哈希集中，并且想知道以下代码是否会这样做：

public final class Tutor {
private String name;
private final Set<Student> tutees;
public Tutor(String name, Student[] students){
    this.name = name;
     tutees = new HashSet<Student>();
     for (int i = 0; i<students.length; i++)
         tutees.add(students[i](students.getName(), students.getCourse());
}

如果需要，学生的代码：

public class Student {
private String name;
private String course;
public Student(String name, String course){
     this.name = name;
     this.course = course;
}
public String getName() { return name; }
public String getCourse() { return course; }
public void setName(String name) {
     this.name = name;
}
public void setCourse(String course){
     this.course = course;
 }
}

谢谢

4

2 回答 2

2

你做得对，但有一些错误，因为你是在纸上写的。如果你将它重写到程序中，它不会编译，由于这一行

tutees.add(students[i](students.getName(), students.getCourse());

需要替换为

tutees.add(new Student(students[i].getName(), students[i].getCourse());

请注意，您正在添加新的Student，但字段是由现有引用初始化的，这导致浅拷贝对象不同但正在共享内容。但是，String类是immutable意味着修改字符串的每个方法都会创建具有应用修改的新字符串，而旧字符串保持不变。所以即使原始学生和它的副本共享内容，字符串修改也不会相互影响，因此我们可以说它起到了防御性复制的作用。

Student original = new Student("name", "course");
Student copy = new Student(original.getName(), original.getCourse());
// does not change the name of the copy
String modifiedName = copy.getName().replaceAll("a", "b");

这是一个真正的防御性复制（深复制）的示例：

Student deepCopy = new Student(
        new String(original.getName()), 
        new String(original.getCourse())
);

出于效率原因，如果您知道您正在使用的类immutable，只需复制它们的引用即可。

于 2017-05-24T15:07:32.867 回答
1

您已经发现将可变学生放入 aSet是一个坏主意的问题。您不想在某个集合中更改某些内容，因为它违反了集合的合同。

创建副本可以解决症状，但不能解决根本问题。问题是您的 Student 类是可变的。如果你让你的 Student 类不可变，你就不需要担心复制，它会显着减少出错的可能性。

public class Student {
    private String name;
    private String course;
    public Student(String name, String course){
        this.name = name;
        this.course = course;
    }
    public String getName() { return name; }
    public String getCourse() { return course; }
}

如果学生更改姓名 - 这种情况多久发生一次？在您的系统中，您可能根本不需要对其建模 - 或更改课程，您只需创建一个新学生并删除旧的不正确的学生。

于 2017-05-24T15:26:03.237 回答