在单个 Java 方法中有两个 for 循环

Having two for loop in a single Java Method

提问人:Agbako 提问时间:3/24/2022 最后编辑:markspaceAgbako 更新时间:3/24/2022 访问量:66

问:

我有一个在下面的代码中显示的方法,我有一个哈希图缓冲区和队列,两者都有几个列表。我想搜索特定消息以在稳定时将其设置为 true;消息可以位于缓冲区或队列中。将两个 for 循环混为一谈是否可以在单个方法中?或者有没有更好的方法来表达我的意图。

因此,如果消息在缓冲区中,则将其设置为 true,如果不是,则检查队列;如果未找到,则返回 false

public static boolean SetMessageToStable(int event_actor, Message msg) {
        if (mBuffer.get(event_actor) != null) {
            for (int i = 0; i < mBuffer.get(event_actor).size(); i++) { // iterate the list of event actor to find matching message
                if (mBuffer.get(event_actor).get(i).equals(msg)) {
                    mBuffer.get(event_actor).get(i).setMsg_stable(true); // set stable if found
                    return true;
                }
            }
        }
        if (DQ.get(event_actor) != null) {
            for (int i = 0; i < DQ.get(event_actor).size(); i++) {  // search the event actor's DQ if the message has not been found
                if (DQ.get(event_actor).get(i).equals(msg)) {
                    DQ.get(event_actor).get(i).setMsg_stable(true); // set stable if found
                    return true;
                }
            }
        }
        return false;
    }
Java 性能 for 循环 方法 传递

评论

2赞 Scott Hunter 3/24/2022
您可以根据需要在方法中放置任意数量的 for 循环。
1赞 Turo 3/24/2022
由于它的工作代码,这个问题更适合 codereview-site。并且还有改进的余地,特别是在空间上,当两者返回时mBuffer.get(event_actor)DQ.get(event_actor)List<Message>
0赞 Ryan 3/24/2022
如果第一个循环返回 true,则第二个循环永远不会发生。这是你想要的吗?
0赞 Agbako 3/24/2022
@Ryan,是的,如果在第一个循环中找到消息,则第二个循环不应发生,反之亦然

答:

1赞 John Stone 3/24/2022 #1

将两个 for 循环混为一谈是否可以在一种方法中?

确定。

有没有更好的方法来表达我的意图?

这两个循环是相同的,但正在计算的特定映射除外(DQ 与 mBuffer)。您可以轻松地做到这一点:

public static boolean SetMessageToStable(int event_actor, Message msg) {
    return SetMessageToStableInternal(mBuffer, event_actor, msg)
        || SetMessageToStableInternal(DQ, event_actor, msg);
}

private static boolean SetMessageToStableInternal(
        Map<Integer, List<Message>> map,
        int event_actor,
        Message msg
) {
    List<Message> msgList = map.get(event_actor);
    if (msgList != null) {
        for (int i = 0; i < msgList.size(); i++) {
            Message m = msgList.get(i);
            if (m.equals(msg)) {
                m.setMsg_stable(true);
                return true;
            }
        }
    }
    return false;
}

评论

0赞 Agbako 3/24/2022
@约翰·斯通(John Stone),现在很有意义。我真的很感谢这个帮助
0赞 Agbako 3/24/2022
@John Stone,如果方法返回类型不是布尔值。返回一个 ||b 将不适用。我的权利吗?