伙伴匹配bug:Long类型id比较bug

伙伴匹配bug:Long类型id比较bug

问题代码:Long类型id直接 “==” 判断值,结果项目测试一直达不到预期。

// 只有管理员或者队伍的创建者可以修改
if (oldTeam.getUserId() != loginUser.getId() && !userService.isAdmin(loginUser)) {
	throw new BusinessException(ErrorCode.NO_AUTH);
 }

解决:

// 只有管理员或者队伍的创建者才可以修改
if (oldTeam.getUserId().longValue() != loginUser.getId().longValue() && !userService.isAdmin(loginUser)){		throw new BusinessException(ErrorCode.NO_AUTH);
 }

Long源码:

private static class LongCache {
	private LongCache(){}
	static final Long cache[] = new Long[-(-128) + 127 + 1];
	static {
    	for(int i = 0; i < cache.length; i++)
        	cache[i] = new Long(i - 128);
        }
}
public static Long valueOf(long l) {
	final int offset = 128;
	if (l >= -128 && l <= 127) { // will cache
		return LongCache.cache[(int)l + offset];
	}
	return new Long(l);
}
public boolean equals(Object obj) {
        if (obj instanceof Long) {
            return value == ((Long)obj).longValue();
        }
        return false;
    }

和Integer差不多。
顺便看了下Long的equals()方法,最后发现直接调用equals()应该也能解决这个问题。

评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包

打赏作者

-62

你的鼓励将是我创作的最大动力

¥1 ¥2 ¥4 ¥6 ¥10 ¥20
扫码支付:¥1
获取中
扫码支付

您的余额不足,请更换扫码支付或充值

打赏作者

实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

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

余额充值