如何重构代码以避免在Java循环中出现标签？

Question

如何重构代码以删除标签及其需要？ 我知道带有标签的循环是“禁止的”。 但是我找不到没有标签的方法来重写它。

有任何想法吗 ？ 谢谢

private List<myList> sortLinks(SegmentType s, Set<myList> LinkSet) {
        List<myList> LinkList = new LinkedList<myList>();

        String dep = s.getDep().toString();
        mainLoop: for (int index = 0; !LinkSet.isEmpty(); index++) {

            for (Iterator<myList> iterator = LinkSet.iterator(); iterator.hasNext(); ) {
                myList link = iterator.next();
                if (link.getLegDep().toString().equals(dep)) {
                    iterator.remove();
                    link.setLine(s.getLineCode());
                    link.setNb(s.getNb());
                    link.setSuff(s.getSuff());
                    link.setIndex(index);
                    linkList.add(link);

                    dep = link.getDest().toString();
                    continue mainLoop;
                }
            }

            return Collections.emptyList();
        }
        return linkList;
    }

Answer 1

您可能需要一个boolean 。 我不完全理解您的代码试图完成什么，但是似乎内部循环正在试图找到某种“依赖关系”？ 如果是这样，我将其命名为boolean depedencyFound ; 如果我真的猜错了，可以称之为其他名称。 无论如何，是这样的：

private List<myList> sortLinks(SegmentType s, Set<myList> LinkSet) {
    List<myList> LinkList = new LinkedList<myList>();

    String dep = s.getDep().toString();
    for (int index = 0; !LinkSet.isEmpty(); index++) {

        boolean dependencyFound = false;
        for (Iterator<myList> iterator = LinkSet.iterator(); iterator.hasNext(); ) {
            myList link = iterator.next();
            if (link.getLegDep().toString().equals(dep)) {
                iterator.remove();
                link.setLine(s.getLineCode());
                link.setNb(s.getNb());
                link.setSuff(s.getSuff());
                link.setIndex(index);
                linkList.add(link);

                dep = link.getDest().toString();
                dependencyFound = true;
                break;
            }
        }

        if (!dependencyFound) {
            return Collections.emptyList();
        }
    }
    return linkList;
}

如果将变量命名为正确，它将使发生的事情很清楚。 在这里，您需要查找依赖项； 如果找不到，则返回一个空列表； 如果这样做，那么您将再次遍历主循环。

PS完成后，可以很容易地将整个内部循环带入一个返回boolean的方法，尽管看起来还需要返回一个具有新值dep的String 。 但是，如果您找到一种执行此操作的方法，它将进一步清理您的代码。

Answer 2

实际上，心律失常的答案是正确的。 您可以使用布尔值继续执行以下操作：

private List<myList> sortLinks(SegmentType s, Set<myList> LinkSet) {
    List<myList> LinkList = new LinkedList<myList>();

    String dep = s.getDep().toString();
    for (int index = 0; !LinkSet.isEmpty(); index++) {

        for (Iterator<myList> iterator = LinkSet.iterator(); iterator.hasNext(); ) {
            myList link = iterator.next();
            boolean toContinue = false;
            if (link.getLegDep().toString().equals(dep)) {
                iterator.remove();
                link.setLine(s.getLineCode());
                link.setNb(s.getNb());
                link.setSuff(s.getSuff());
                link.setIndex(index);
                linkList.add(link);

                dep = link.getDest().toString();
                toContinue = true;
                break; // this breaks inner loop
            }
        }
        if(toContinue) {
            continue; // this continues main loop without executing the 'return'
        }
        return Collections.emptyList();
    }
    return linkList;
}

Answer 3

由于continue在内部循环的末尾，因此只需使用break即可开始主循环的下一次迭代。