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


    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 创建一个唯一的锁对象,使用 HashMapputIfAbsent 方法。如果给定的 ID 在 locks 映射中不存在对应的锁对象,就会创建一个新的,并存储在映射中。对于相同的 ID,后续调用 getLock 将返回之前创建的锁对象。

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

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

  • 0
    点赞
  • 1
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值