java - 将代码重构为只有 return 语句

Question

我正在尝试重构此代码以使其只有一个返回语句，但我似乎不知道如何重构。我正在搜索树中的特定节点。树中的每个节点都可以有 child 。任何方向将不胜感激，谢谢

private TreeNode getDefaultNode(List list, String supervisingGroupName){

    for (int i = 0; i < list.size(); i++){
        TreeNode tempNode = (TreeNode) list.get(i);
        OSGroup temp = (OSGroup) tempNode.getNodeBusinessObject();

        if (supervisingGroupName.equals(temp.getName())){
            return tempNode;
        }
        else {
            TreeNode node = getDefaultNode(tempNode.getChildren(), supervisingGroupName);
            if (node != null){
                return node;
            }
        }

    }

    return null;
}

Answer 1

您可以使用 break 关键字跳出您的 for 循环:

private TreeNode getDefaultNode(List list, String supervisingGroupName){
    TreeNode node = null;
    for (int i = 0; i < list.size(); i++){
        node = (TreeNode) list.get(i);
        OSGroup temp = (OSGroup) node.getNodeBusinessObject();

        if (supervisingGroupName.equals(temp.getName())){
            break;
        } else {
            node = getDefaultNode(tempNode.getChildren(), supervisingGroupName);
            if (node != null){
                break;
            }
        }

    }

    return node;
}

这样你就可以重构为一个单一的返回语句，虽然这对我来说似乎不再清楚了。重构前您的代码没有任何问题。