文章目录
- 错误分析
- 先看sonar 给解决方案
- 最终解决方案
public void synchronizedMethod(Integer id) {
synchronized (id) {
// Code to be synchronized
}
}
错误分析
Synchronize on a new "Object" instead.
不应该在原始包装器(即整数、长整型、布尔值等)上进行同步,因为它们可以通过自动装箱创建,并且 Sonar 认为这是有风险的。这是有风险的,因为语句中使用的锁对象synchronized在所有要同步的线程中应该是恒定的。任何可能在程序逻辑中重复使用的动态对象很容易导致同步失败。因此,最佳实践是实例化 Object 并创建一个仅用于同步的特殊锁对象。
先看sonar 给解决方案
可以看到她让我们给一个 常量对象 ,但是明显不是我想要的,我想要的是每个不同ID 进入是异步的,只有同一个Id才是锁定的
最终解决方案
要使用 ID 作为锁,在 Java 中,你可以创建一个自定义的类来存储 ID,并将其作为监视器进行同步。以下是一个示例:
import java.util.HashMap;
import java.util.Map;
public class SynchronizationExample {
private Map<Integer, Object> locks = new ConcurrentHashMap<>();
public void synchronizedMethod(Integer id) {
synchronized (getLock(id)) {
// 需要同步的代码块
}
}
private Object getLock(Integer id) {
locks.putIfAbsent(id, new Object());
return locks.get(id);
}
}
在这个示例中,我们使用一个 HashMap
来存储与每个 ID 相关联的锁对象。synchronizedMethod
方法接受一个 ID 参数,并使用从 locks
映射中获取的相应锁对象来同步代码块。
getLock
方法确保为每个 ID 创建一个唯一的锁对象,使用 HashMap
的 putIfAbsent
方法。如果给定的 ID 在 locks
映射中不存在对应的锁对象,就会创建一个新的,并存储在映射中。对于相同的 ID,后续调用 getLock
将返回之前创建的锁对象。
通过将 ID 作为 locks
映射的键,你可以将特定的锁与每个 ID 关联起来,确保每个 ID 的同步而不是跨所有 ID 进行同步。
请记住,当使用自定义对象作为锁时,在访问和修改底层数据结构时要确保线程安全,以避免潜在的并发问题。