另一个“只能迭代数组或 java.lang.Iterable 的实例”问题答案

【问题标题】：Yet another "Can only iterate over an array or an instance of java.lang.Iterable" issue另一个“只能迭代数组或 java.lang.Iterable 的实例”问题
【发布时间】：2014-05-23 22:02:07
【问题描述】：

我有这段代码返回 java.lang.iterable 错误。我知道我在哪里犯了错误，但我不知道如何修复它。

代码如下：

public class ManagementServiceHandler implements ManagementService.Iface {
    private Map<Node, Integer> nodes;
    private int portCount = 1025;

    public ManagementServiceHandler() {
      nodes = new HashMap<Node, Integer>();
    }

    //register node
    public int RegisterNode(Node hostAndService) throws TException {
        portCount++;
        nodes.put(hostAndService, portCount);

        return portCount;            
    }

    //get service
    public Map<String, Integer> getProvidersForServices(String svcName, int port) throws TException {
        Map<String, Integer> result = new HashMap<>();

        for (Map.Entry<Node, Integer> pair : nodes.entrySet()) {
            for (String nodeService : pair.getKey().serviceName) {
                if (nodeService.equals(svcName)) {
                    result.put(pair.getKey().serviceName, pair.getValue());
                    break;
                }
            }
        }

        return result;
    }
}

问题在于第二种方法，在第二个 foreach 循环中。在pair.getKey() 之后，我不确定我应该在周期的正确部分放什么。我想要做的是循环映射nodes内的所有条目<Node, Integer>"，然后将serviceName的每个条目写入另一个映射result。

我知道 foreach 语句的正确部分代表我想要循环的元素集合，但在这种情况下这对我没有多大帮助，除了我编码错误的事实..

有什么提示吗？

【问题讨论】：

标签： java map foreach cycle

【解决方案1】：

显然你想要的只是赋值给一个局部变量。替换

for (String nodeService : pair.getKey().serviceName)

与

String nodeService = pair.getKey().serviceName;

【讨论】：

感谢您和 Nemo 接下来的回答，解决了问题。

【解决方案2】：

HashMap.Entry 只是一个元组，而不是一个可迭代的集合

这意味着您不需要第二个for-each，因为您可以直接访问Node.serviceName

String serviceName = pair.getKey().serviceName;

【讨论】：