V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
kikione
V2EX  ›  程序员

这样加锁会导致什么问题?

  •  
  •   kikione · 66 天前 · 2172 次点击
    这是一个创建于 66 天前的主题,其中的信息可能已经有所发展或是发生改变。
    String lock = userId;
    synchronized(lock.intern()){
    }

    userId 是唯一的。

    这样加锁常量池的数据会很庞大,会被 GC 及时回收吗
    9 条回复    2021-11-21 20:21:26 +08:00
    sagaxu
        1
    sagaxu  
       66 天前 via Android
    gc 能回收,但 rootset 会很大,stw 更久
    fkdog
        2
    fkdog  
       66 天前
    你这有问题。
    你如何保证值相同的 userId 字符串对应的是同一个对象?
    fkdog
        3
    fkdog  
       66 天前
    Integer a = 999992;
    System.out.println(a.toString()==a.toString());

    你 run 一下就会发现这个结果在 jdk1.8 以上就是 false 。
    不是所有字符串都会建在常量池里
    kikione
        4
    kikione  
    OP
       66 天前
    @fkdog string intern() 函数 强制刷到常量池
    sagaxu
        5
    sagaxu  
       66 天前
    SachinBeyond
        6
    SachinBeyond  
       66 天前
    Constant expressions of type String are always "interned" so as to share unique
    instances, using the method String.intern.
    fkdog
        7
    fkdog  
       66 天前
    @kikione intern 能解决单机上边的问题,那么如果机器不只一台呢?
    moshiyeap100
        8
    moshiyeap100  
       66 天前
    这样加锁在单机上是没有问题的,但是在多台机器上就不行了,多台机器你可能需要一个分布式锁,可以通过 redis setnx 来解决这个问题。
    qaqLjj
        9
    qaqLjj  
       66 天前   ❤️ 1
    这样写代码不优雅,字符串常量池本来是用来存储编译时可以确定的字符串常量的,你这样写相当于运行时往常量池里放东西,建议使用 new Lock(userId); 代替
    关于   ·   帮助文档   ·   API   ·   FAQ   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   1074 人在线   最高记录 5497   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 23ms · UTC 20:24 · PVG 04:24 · LAX 12:24 · JFK 15:24
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.