java - Java 同步轮询

Question

这是我曾经看过的一些代码。你能看出它有什么问题吗？

[更新]

public class ResourceManager1
{
    private final String mutex = "";
    Object resource = null;

    public Object getResource()
    {
        synchronized (mutex)
        {
            if (resource == null)
            {
                resource = new Object();
            }
        }

        return resource;
    }
}

public class ResourceManager2
{
    private final String mutex = "";
    Object resource = null;

    public Object getResource()
    {
        synchronized (mutex)
        {
            if (resource == null)
            {
                resource = new Object();
            }
        }

        return resource;
    }
}

Answer 1

永远不要在字符串上同步，尤其是被实习的字符串文字。你基本上只有一个锁。

通常，除非外部可见性的目的正是为了锁定目的，否则永远不要同步任何可能在您的类之外可见的引用（包括“this”） 。我通常使用private final仅为锁定目的而创建的变量。

Answer 2

您使用相同的 String 作为两个类的互斥锁，因此一次只能使用一个同步块，这似乎不是代码的意图。

Answer 3

互斥锁不是最终的，资源不是私有的。

此外，您需要一个返回资源的 getResource 方法，但我想这只是一个错字。