高并发系统中的锁定

Question

我有一个高度并发系统中的课程。该类的方法 getResolvedClassName() 可能会产生死锁。所以我按照以下方式设计它：

public class ClassUtils {

    private static ClassUtils classUtils;

    private transient Object object = new Object();

    private synchronized Object getObjectLock() {
        return object;
    }

    public void getResolvedClassName(Class<?> clazz) {
        synchronized (getObjectLock()) {
            //some job will be done here
        }       
    }

    public synchronized static ClassUtils getInstance() {
        if(classUtils == null) {
            classUtils = new ClassUtils();
        }
        return classUtils;
    }   
}

我这样做的方式正确吗？任何信息都会对我有帮助。

谢谢。

---

编辑：

public class ClassUtils {

    private static final ClassUtils classUtils = new ClassUtils();
    private ReentrantLock lock = new ReentrantLock();

    public void getResolvedClassName(Class<?> clazz) {
        lock.lock();
        //some job will be done here
        lock.unlock();      
    }

    public static ClassUtils getInstance() {        
        return classUtils;
    }   
}

Answer 1

有几点值得注意：

1. 我不认为 transient 关键字 意味着你认为它意味着什么。该关键字与同步无关，仅在序列化类时使用。您可能会将其与易失性混淆。顺便说一句，这里也不需要易失性。

2. 单例的延迟初始化可能是不必要的。为什么不直接执行private static Final ClassUtils classUtils = new ClassUtils();？那么你的 getInstance() 方法不需要同步，只需返回 classUtils; 它也是线程安全的。您还应该始终将单例实例声明为 final。

3. 不需要 getObjectLock() 的整个情况。您可以仅在 this 上进行同步（即将 getResolvedClassname 转换为 synchronized 方法），这样会更安全、更干净。

您还可以研究 java.util.concurrent.Lock 类，看看是否有比在 Object 上同步更合适的东西，现在这种方式被认为是糟糕的形式。

Answer 2

这个问题确实有点模糊，我不明白使用单例的目的以及为什么需要同步来完成某些工作。如果它不访问可变状态，则不需要同步。我只能说三个锁（ClassUtils.class、ClassUtils 实例和object）几乎肯定会增加不必要的复杂性。另外，正如 Justin 指出的，您应该将 object 设置为最终对象，然后就不需要同步来访问它。

Answer 3

您的问题有点普遍。但是，您可以考虑将该值初始化为不可变的。许多不可变的初始化值是线程安全的，并且不需要锁定。