ConcurrentHashMap foreach循环问题

发布于 2024-12-06 17:57:29 字数 679 浏览 0 评论 0原文

我有一个名为 users 的并发哈希图。我有用户对象，其中有一些不是 id 的整数键。我想找到具有给定 id 的用户。因此，我检查 hashmap 的所有元素并返回用户对象（如果存在）。这是我的代码：

    for(User u : users.values()) {
        logger.error("u.getId() : " + u.getId());
        logger.error("id : " + id );
        if( u.getId() == id ) {
            logger.error("match");
            return u;
        }
    }
    logger.error("Not found: id:" + id);
    for(User u : users.values()) {
        logger.error(u.getPos() + ". user: " + u.getId());
    }

但是，即使我的 u.getId() 和 id 是相同的，我也无法在日志中看到“匹配”。

213 匹配但不能输入下面的 if 语句。这是我的日志：

在此处输入图像描述

您对此有何看法？

原文

I have a concurrenthashmap called users. I have user objects in it with some integer keys that is not id. I want to find the user with a given id. Therefore, I check all elements of hashmap and return the user object if it is present. Here is my code :

    for(User u : users.values()) {
        logger.error("u.getId() : " + u.getId());
        logger.error("id : " + id );
        if( u.getId() == id ) {
            logger.error("match");
            return u;
        }
    }
    logger.error("Not found: id:" + id);
    for(User u : users.values()) {
        logger.error(u.getPos() + ". user: " + u.getId());
    }

However even tough my u.getId() and id are the same I cannot see "match" at my logs.

213 matches but It can not enter the following if statement.
here are my logs:

enter image description here

What do you think about it?

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

德意的啸 2024-12-13 17:57:29

User.getId() 方法返回什么类型以及什么类型的 id 变量？如果不是原始类型，则需要使用 equals() 而不是 ==。

顺便说一句，像 FindBugs 这样的优秀静态代码分析器可以找到此类错误。

回复收藏 0 原文

最初的梦 2024-12-13 17:57:29

您尚未显示所涉及的类型，但 id 或 getId() 是否有可能是 Integer 而不是 int ？如果是这样，您将比较引用，因此您应该只用来

if (u.getId().equals(id))

比较Integer对象内的值。如果 getId 可能返回 null，请小心...

You haven't shown the types involved, but is it possible that id or getId() is an Integer instead of an int? If so, you'll be comparing references, so you should just use