在生产的项目中看到类似的代码块，这个锁加的是不是没用呀？ 我写了 demo 还试了一下,大佬们看一下？

[kikione]

public class TestContronller {

public String method(HttpServletRequest request){
	
    String merchanId=request.getparamter('id');
    
	synchronized (merchanId){

    }
    
    return "  ";
}

}

[justRua]

synchronized 把锁加载对象上，关键是 request.getparamter('id')返回来的 String 对象是不是同一个，如果每次获得的对象是 new 出来的那是没用的，这种写法一言难尽，和 synchronized (Integer.valueOf(1))用的异曲同工的感觉，不是什么好写法，其他模块如果也这么写了可能就会有莫名其妙的锁竞争。

[ma836323493]

request.getparamter('id'), 应该是 new String() , 与常量池中的不等吧，锁起来应该有用

[6IbA2bj5ip3tK49j]

没用，需要 intern()

[lakehylia]

你还不如把 synchronized 加到方法上。。。public synchronized String method(HttpServletRequest request){}

[xiangyuecn]

这种逼写法就和 @Transactional 一样，交差拿钱了啥也不管，管你有没有效 （狗头

[wysnylc]

不要直接使用 intern()有性能问题会疯狂 GC,用 guava 的创建 Interners.newWeakInterner()一个弱引用的内部常量池
具体代码:
Interner<String> pool = Interners.newWeakInterner();

synchronized ( pool.intern("BizCode"+userId)){

//TODO:something

}

[Kirsk]

就算要用锁 也不是 syc 假装有锁