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

gc 能回收，但 rootset 会很大，stw 更久

你这有问题。
你如何保证值相同的 userId 字符串对应的是同一个对象？

Integer a = 999992;
System.out.println(a.toString()==a.toString());

你 run 一下就会发现这个结果在 jdk1.8 以上就是 false 。
不是所有字符串都会建在常量池里

@fkdog string intern() 函数 强制刷到常量池

@fkdog String.intern 保证一定是同一个对象

https://docs.oracle.com/en/java/javase/11/docs/api/java.base/java/lang/String.html#intern()

Constant expressions of type String are always "interned" so as to share unique
instances, using the method String.intern.

@kikione intern 能解决单机上边的问题，那么如果机器不只一台呢？

这样加锁在单机上是没有问题的，但是在多台机器上就不行了，多台机器你可能需要一个分布式锁，可以通过 redis setnx 来解决这个问题。

这样写代码不优雅，字符串常量池本来是用来存储编译时可以确定的字符串常量的，你这样写相当于运行时往常量池里放东西，建议使用 new Lock(userId); 代替