我应该使用Java字符串池进行基于唯一客户ID的同步吗？

Question

public class Key {
    private String id;

    public Key(String id) {
        this.id = id;
    }

    public String getId() {
        return id;
    }

    @Override
    public int hashCode() {
        final int prime = 31;
        int result = 1;
        result = prime * result + ( (id == null) ? 0 : id.hashCode());
        return result;
    }

    @Override
    public boolean equals(Object obj) {
        if (this == obj) return true;
        if (obj == null) return false;
        if (getClass() != obj.getClass()) return false;
        Key other = (Key)obj;
        if (id == null) {
            if (other.id != null) return false;
        } else if (!id.equals(other.id)) return false;
        return true;
    }
}

Interner<Key> myIdInterner = Interners.newWeakInterner();

public void processApi1(String clientUniqueId, RequestType1 request) {
    synchronized(myIdInterner.intern(new Key(clientUniqueId))) {
        // code to process request
    }
}

public void processApi2(String clientUniqueId, RequestType2 request) {
    synchronized(myIdInterner.intern(new Key(clientUniqueId))) {
        // code to process request
    }
}

Answer 1

如果您的字符串足够唯一（例如，通过密码哈希¹)生成），那么在客户端ID上进行同步可能会起作用，只要您首先对它们调用string.intern()。由于ID是唯一的，您不太可能遇到与其他模块冲突的情况，除非您碰巧将ID传递给它们，并且它们遵循了锁定它们的不良做法。

这就是说，这可能是个坏主意。如果其他人锁定同一string实例，那么有一天可能会出现不必要的争用，除此之外，主要的问题是您必须intern()所有string对象，而且由于string intern表的本机实现、它的大小是固定的等原因，这通常会导致性能较差。如果您真的需要仅基于string进行锁定，那么最好使用Guava的Interners.NewWeakInterner()interner实现，这样可能会有更好的性能。将字符串包装在另一个类中，以避免在内置的string锁上发生冲突。关于这种方法的更多细节，请参阅本答案。

除此之外，通常还有另一个自然对象要锁定，比如会话对象中的锁等等。

这与这个问题很相似，因为这个问题的答案更充实。

¹...或者，至少有足够的位，使冲突的可能性足够小，并且如果您的客户端ID不是您的攻击面的一部分。