Python 编码风格 - 多个返回语句[关闭]

对于同一任务，我编写了两个不同的函数。我想知道哪一个使用起来更优雅。

任务是检查一个pydot如果它看到请求的节点，则返回对象，如果是，则返回该节点和图形对象。 如果该节点不存在，则需要创建该节点。

为了获取节点的名称，我使用pydot物体get_nodes()功能。但是，如果尚未引入任何节点，则此函数将返回一个空列表。因此，在迭代这些值之前，我会进行空列表检查。

第一个变体（‘variant1’）很容易理解。长度检查之后，这是必要的，因为node.get_name()，它循环到节点名称，一旦找到正在搜索的节点，就会返回该节点和图。如果没有，它会调用一个函数来创建节点并更新图形。虽然这个函数很容易理解，但恕我直言，它并不优雅。它有两个“return”语句：

def variant1(node_name, graph):

    if len(graph.get_nodes()) > 0:

        for node in graph.get_nodes():
            if node_name == node.get_name():
                return node, graph

    return create_node(node_name, graph)

第二种变体相当复杂。一旦在图中找到节点，它就会中断并直接跳转到最后一行（“返回节点，图”）。此变体只有一个 return 语句。

def variant2(node_name, graph):

    if len(graph.get_nodes()) > 0:

        for node in graph.get_nodes():
            if node_name == node.get_name():
                break

        else:
            # the node doesnt exist. create it and update the graph
            node, graph = create_node(node_name, graph)

    else:
        # create the first node in the graph
        node, graph = create_node(node_name, graph)

    return node, graph

我的问题是，根据“The Zen of Python”、“PEP8”或“Google Python Style Guide”，我应该更喜欢使用哪一个？

我是这样写的：

def variant1a(node_name, graph):
    """Include a docstring, too!"""
    for node in graph.get_nodes():
        if node.get_name() == node_name:
            return node, graph
    return create_node(node_name, graph)

这意味着：

• 比较操作的顺序更易读；
• 只有一根线呼叫create_node (DRY!);
• 虽然有两个return行，两者都位于函数的底部；
• no for: else:（有些人不喜欢 - 我认为它通常很有用，但在这里没有必要）；和
• 没有不必要的长度检查（如果len(graph.get_nodes)为零，无论如何都会跳过循环）。

PEP-8 并不禁止多个return线条（事实上，风格指南中的一些示例确实有线条）。我在 Google 的风格指南中看不到对它的引用，但我没有展开所有部分！