关于 synchronized 锁定对象出现的问题

[fronted]

代码如下，add 方法中有一个 synchronized 代码块，锁定的是 uid 字符串，代码中的逻辑也很简单，先查询没有则新增，有则不做操作。但是这个方法会出现重复新增的情况，也就是没锁住，多个线程进来了。发生这种情况的数据量也很少，大概几千条数据就一两条出现这种情况.大家有什么看法吗？

补充一下：出现上述并发情况下打印过锁定对象的内存地址值，结果发现地址值也是一样的。

public class StringPoolUtil {
    private static final Interner<String> POOL = Interners.newWeakInterner();

    public static Interner<String> getPool(){
        return POOL;
    }
}

public void add(String uid){
    synchronized (StringPoolUtil.getPool().intern(uid)) {
    // 查询数据
    // 没有数据则新增
    }
}

[GThui]

肯定锁对象不安全了。可能 uid 或者 StringPoolUtil 方法可能和 gc 也有关系

[fronted]

@GThui 打印过锁定对象的地址值，都是一样的

[dqzcwxb]

项目部署了几个

[fronted]

@dqzcwxb 单项目哈

[lovedoing]

Interners 里面的 String 也是会被 gc 的

[Hurriance]

「没有数据则新增」这里可以打个日志看看吗？可能是其他线程真的也没查到数据？

[codehz]

虽然不是很懂 interner ，但是稍微查看了下文档，这应该要用 newStrongInterner 才对吧。。。

[fronted]

@Hurriance 是其他线程没有查到数据，但是锁定的同一个对象，没释放锁其他线程不知道怎么的也进来了

[fronted]

@codehz 有道理，但是我这里出现的这个问题没有发生 gc 这种情况

[zilongzixue]

直接上 redisson 分布式锁好了，一步到位

[sprit]

有无可能不是 SYNC 的问题，是 MySQL 的问题[dog]

[liudaolunhuibl]

看一下“StringPoolUtil.getPool().intern(uid)”这个呢？

为什么不直接 uid.intern(）呢？

[liudaolunhuibl]

jdk7 之后可以通过-XX:StringTableSize 参数来修改 StringTable 的大小，先试试改为用 String 的 intern 吧，你这个本来也是 String 对象

[liudaolunhuibl]

也可以换成 Interners.newStrongInterner()

[exqibao]

会不会是 synchronized 块已经结束了，但事务还没有提交。

[corningsun]

数据库加唯一索引吧？

[suStudent]

弱引用：弱引用也是用来描述那些非必须对象，但是它的强度比软引用更弱一些，被弱引用关联的对象只 能生存到下一次垃圾收集发生为止。当垃圾收集器开始工作，无论当前内存是否足够，都会回收掉只 被弱引用关联的对象。

怀疑多次加锁时，中间经历过一次垃圾回收。

[fronted]

@suStudent 应该不至于 我在进入 synchronized 打印过锁定的对象地址，值都是一样的

[night98]

楼上说的都挺有道理，我补充一条吧，sync 应该包裹 add 方法而不是在 add 方法体内，否则会存在极小的时间差即 return 后事务注解执行 commit 方法前进入方法

[suStudent]

@fronted 对应地址值？ hashCode()? hashCode 可能相等。网上的例子"重地"和"通话" hashCode 值是一样的