线安全单身，锁

Question

我有一个Singleton课程，这是线程保险箱，我需要锁定它的方法吗？

        private static volatile JsonWriter instance;
private static final Object mutex = new Object();

ReentrantReadWriteLock lock = new ReentrantReadWriteLock();

private JsonWriter() {
}

public static JsonWriter getInstance() {
    JsonWriter result = instance;
    if (result == null) {
        synchronized (mutex) {
            result = instance;
            if (result == null) {
                instance = result = new JsonWriter();
            }
        }
    }
    return result;
}

我需要锁定这样的每种方法以确保线程安全吗？

    public void write(String filePath, ArrayNode content) throws IOException {
    lock.writeLock().lock();
    File file = new File(MASTER_DIR + "/" + filePath);
    mapper.writerWithDefaultPrettyPrinter().writeValue(Files.newOutputStream(file.toPath()), content);

}

Answer 1

最佳性能和线程安全的单身实施是William Pugh Singleton。您不需要同步块和reentrantreadWritelock。

威廉·普格（William Pugh）的实施可确保多线程安全性和最佳性能，因为它可以避免渴望创造。实际上，只有在嵌套类是由类加载器加载时，即在实际使用嵌套类时，才会创建静态成员实例。在以下实现中，只有在调用getInstance（）方法时，才能发生这种情况。实际上，相反，该模型实际上允许使用封闭的类而不会引起急切的实例化。这意味着无需初始化实例而无需安全地使用封闭类提供的任何其他方法；只有getInstance（）方法才会引起它。

您的实施可能会像这样：

class JsonWriter {

    private JsonWriter() {}

    /* ... other methods ... */

    public static JsonWriter getInstance() {
        return SingletonHelper.INSTANCE;
    }

    private static class SingletonHelper {
        private static final JsonWriter INSTANCE = new JsonWriter();
    }
}

特别感谢@Holger，他在评论中以重要的澄清和评论做出了贡献。

Answer 2

1

如果要使用同步：

private static volatile JsonWriter instance;   

private JsonWriter() {
}

public synchronized static JsonWriter getInstance() {
    JsonWriter result = instance;
    if (result == null) {
        instance = new JsonWriter();
    }
    return result;
}

3或可以使用锁：

private static volatile JsonWriter instance;
private static final Object mutex = new Object();

ReentrantReadWriteLock lock = new ReentrantReadWriteLock();

private JsonWriter() {}

public static JsonWriter getInstance() {
    lock.writeLock().lock();
    JsonWriter result = instance;
    if (result == null) {
       instance = result = new JsonWriter();
      }
    lock.writeLock().unlock();
    return result;
}