判断用户是否存在bug

[chanris]

org/opengoofy/index12306/biz/userservice/service/impl/UserLoginServiceImpl.java
代码片段

    @Override
    public Boolean hasUsername(String username) {
        boolean hasUsername = userRegisterCachePenetrationBloomFilter.contains(username);
        if (hasUsername) {
            StringRedisTemplate instance = (StringRedisTemplate) distributedCache.getInstance();
            return instance.opsForSet().isMember(USER_REGISTER_REUSE_SHARDING + hashShardingIdx(username), username);
        }
        return true;
    }

1. 当bloomFilter判断username为false，username一定不存在，则应该返回false；
2. 并且，hasUsername为true时，应返回 !instance.opsForSet().isMember(USER_REGISTER_REUSE_SHARDING + hashShardingIdx(username), username);，表示username已复用；
3. 此外，还有一种情况没有考虑，username不存在，但bloomFilter误判为true，此时isMember() 为 false，按 2. 的 逻辑为 true，实际应为 false；
   故感觉逻辑不正确，若我理解错误，请多指教。

[YiLaiL]

我也感觉逻辑怪怪的，想了半天没想明白，兄弟现在懂了吗

[Chinese-Coding]

这个函数的函数名起的不对 应该叫做 isUsernameAvailable, 作用是判断用户名是否存在, 这样才符合函数的逻辑.