2

编辑

请注意，我注意到在 __init__ 之外定义的实例属性 attribute_name可能是重复的，我基本同意（我没有遇到这个，因为我不知道要搜索 pylint）。但是，我想保持这个问题是开放的，因为我希望能够使用相同的方法重新初始化我的类。上一个问题的普遍共识是从脚本中返回每个参数loadData，然后将其解析为 self 对象。这很好，但是，我仍然需要在另一个方法中再次这样做才能重新初始化我的类实例，这似乎仍然是额外的工作，只是为了提高一点可读性。也许这个问题就是我的例子。在现实生活中，大约有 30 个参数被loadData例程，这就是为什么我不愿意在两个不同的位置解析它们的原因。

如果这里的普遍共识是返回参数是要走的路，那么我们可以继续并关闭这个问题作为重复；但是，与此同时，我想等一下，看看其他人是否有任何想法/一个很好的解释。

原来的

这是一个“最佳实践”的问题。我最近一直在学习 python（部分是为了学习新的东西，部分是为了远离 MATLAB）。在 python 中工作时，我创建了一个结构如下的类：

class exampleClass:
    """
    This is an example class to demonstrate my question to stack exchange
    """

    def __init__( self, fileName ):
        exampleClass.loadData( self, fileName )

    def loadData( self, fileName ):
        """
        This function reads the data specified in the fileName into the 
        current instance of exampleClass.
        :param fileName: The file that the data is to be loaded from
        """

        with open(fileName,'r') as sumFile:
            self.name = sumFile.readLine().strip(' \n\r\t')

现在这对我来说很有意义。我有一个初始化类，它通过调用填充函数来填充类的当前实例。我还有人口功能，如果出于某种原因需要，我可以重新初始化此类的给定实例（例如，如果该类占用大量内存，而不是创建我只想创建的单独的类实例有一个我覆盖的实例。

但是，当我将此代码放入我的 IDE (pycharm) 时，它会引发警告，说明实例属性是在__init__. 现在显然这不会影响代码的运行，一切正常，但我想知道在这种情况下是否有任何理由要注意警告。我可以做一些事情，在调用 init 方法之前将所有属性初始化为某个默认值，loadData method但这对我来说似乎是不必要的工作，并且会减慢执行速度（尽管只是很小的一部分）。我基本上也可以拥有 loadData 方法的两个副本，一个在__init__方法中，一个作为实际方法，但这似乎又是不必要的额外工作。

总的来说，我的问题是在这种情况下最好的做法是什么。是否有任何理由我应该以我在上一段中提到的一种方式重构代码，或者这只是一个带有过于广泛的代码检查警告的 IDE 实例。我显然可以看到一些需要考虑此警告的情况，但根据我目前的经验，在这种情况下它看起来不是问题。

4

1 回答 1

2

我认为最好预先定义所有属性，即使您稍后要重新定义它们。当我阅读您的代码时，我希望能够看到您的数据结构。如果某个方法中隐藏的某些属性仅在某些情况下才被定义，则会使代码更难理解。

如果给属性赋予最终值不方便或不可能，我建议至少将其初始化为None. 这向读者表明该对象包含该属性，即使它稍后被重新定义。

class exampleClass:
    """
    This is an example class to demonstrate my question to stack exchange
    """

    def __init__( self, fileName ):
        # Note: this will be modified when a file is loaded
        self.name = None

        exampleClass.loadData( self, fileName )

另一种选择是loadData返回值而不是设置它，因此您的 init 可能如下所示：

def __init__(self, fileName):
    self.name = self.loadData(fileName)

我倾向于认为第二种方法更好，但任何一种方法都可以。关键是，让你的类和对象尽可能容易理解。

于 2015-11-16T16:41:13.403 回答