python - 从函数返回列表和整数的最佳方法是什么？

Question

关闭。这个问题是 opinion-based 。目前不接受答案。

---

想要改进这个问题吗？更新问题，以便 editing this post 可以用事实和引文来回答它。 .

已关闭 3 年前。

Improve this question

我有一个读取 csv 的函数给我一个列表。然后，我找到该列表的最大值，然后从函数返回该列表和最大值。我想知道执行此操作的最佳方法，以使我的代码更高效且更具可读性。我对 python 还比较陌生，我正在尝试使用最佳实践来构建我的技能。目前，我有:

def getLayerList(layersLoc):
    import csv
    layerList = []
    reader = csv.reader(open(layerLoc))
    for list in reader:
        for layer in list:
            layerList.append(int(layer))
    numLayers = max(layerList)
    return layerList, numLayers

我可以将图层列表和 numLayers 称为该函数的 [0],[1] 实例。我担心的是，如果我返回越来越多不同类型的数据，将很难跟踪它们并且很难读取它们。我应该使用类吗？更Pythonic 的方法是什么？

Answer 1

从性能角度来看，我首先要避免的是在函数本身中调用import。在顶层进行导入将避免重复检查 sys.modules 来查看模块是否已导入。

接下来，在迭代后调用 max 会再次有效地迭代该列表。我会跟踪循环内的最大值，以避免对列表进行第二次迭代。

最后，我将使用 with 语句打开文件，这样您就知道该文件在阅读器的生​​命周期内保持打开状态，这实际上可以在函数外部处理

# imports go up here unless you have a very specific reason not to
import csv

# naming should be snake_case
def get_layer_list(my_reader):
    layer_list = []
    num_layers = 0

    # don't shadow builtin names like list
    for lst in my_reader:
        for layer in lst:
            val = int(layer)

            # do that check here
            if val > num_layers:
                num_layers = val

            layer_list.append(val)

    # now, num_layers has already been checked
    return layer_list, num_layers


with open('somefile.csv') as fh:
    reader = csv.reader(fh)

    # this is how you'd call the function
    layers, nlayers = get_layer_list(reader)

# do something with layers and nlayers

否则，返回中的多个值是完全Pythonic的。当元组解包通常就足够时，unPythonic 就是对返回值进行索引。

# instead of this
x = return_two_values()
a = x[0]
b = x[1]

# do this
a, b = return_two_values()