sonar 错误 提示 Synchronize on a new “Object“ instead.使用Id 作为锁

文章目录

  • 错误分析
  • 先看sonar 给解决方案
  • 最终解决方案


    public void synchronizedMethod(Integer id) {
        synchronized (id) {
            // Code to be synchronized
        }
    }

错误分析

Synchronize on a new "Object" instead.
不应该在原始包装器(即整数、长整型、布尔值等)上进行同步,因为它们可以通过自动装箱创建,并且 Sonar 认为这是有风险的。这是有风险的,因为语句中使用的锁对象synchronized在所有要同步的线程中应该是恒定的。任何可能在程序逻辑中重复使用的动态对象很容易导致同步失败。因此,最佳实践是实例化 Object 并创建一个仅用于同步的特殊锁对象。

先看sonar 给解决方案

sonar 错误 提示 Synchronize on a new “Object“ instead.使用Id 作为锁_第1张图片

可以看到她让我们给一个 常量对象 ,但是明显不是我想要的,我想要的是每个不同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 创建一个唯一的锁对象,使用 HashMapputIfAbsent 方法。如果给定的 ID 在 locks 映射中不存在对应的锁对象,就会创建一个新的,并存储在映射中。对于相同的 ID,后续调用 getLock 将返回之前创建的锁对象。

通过将 ID 作为 locks 映射的键,你可以将特定的锁与每个 ID 关联起来,确保每个 ID 的同步而不是跨所有 ID 进行同步。

请记住,当使用自定义对象作为锁时,在访问和修改底层数据结构时要确保线程安全,以避免潜在的并发问题。

你可能感兴趣的:(日常问题记录,并发编程,python,开发语言)