由多个线程访问的单例类中的实例变量

Question

我有一个单人班：

public class School {
    private HashMap<String, String> students;

    private static School school;

    private School(){
        students = new HashMap<String, String>();   
    }

    public static School getInstance(){
       if(school == null){
           school = new School();
       }
       return school;
    }

    //Method to add student
    protected void addStudent(String id, String name){
          students.put(id,name);
    }
    //Method to remove student
    protected void removeStudent(String id){
          students.remove(id);
    }
}

在我的应用程序中，可能有多个线程使用这个school类来getinstance()，然后添加和删除student。为了使访问（特别是对students实例的访问）是线程安全的，我考虑对getInstanc()方法使用synchorized关键字，例如：

public synchronized static School getInstance(){
       if(school == null){
           school = new School();
       }
       return school;
    }

但是我认为我的微不足道的更改只能确保在多线程环境中只创建一个学校实例。我还需要做什么才能使多个线程访问students实例的线程安全。如有任何好的建议或意见，敬请接受，谢谢！

Answer 1

离开关于单例是否邪恶的讨论，让我们只考虑school类中的线程安全问题：

• 共享对象是“懒惰地”创建的--这需要同步，以避免使school的两个实例；您已经正确地识别并修复了此问题。但是，由于初始化School不需要太多时间，所以您不妨通过急切地初始化School=new School()，使getinstance（）成为一个简单的getter。
• 学校内部的哈希映射-对哈希映射的并发访问将导致异常。您需要在添加、删除和迭代学生的代码周围添加同步，以避免这些异常。
• 对单个学生的访问-一旦调用者获得student对象，他们就可以同时开始修改它。因此，student对象需要自己的并发保护。