python - 在 __init__ 中定义一个类属性，而不是在另一个类方法中——python

Question

编辑

请注意，我注意到 Instance attribute attribute_name defined outside __init__可能是重复的，我基本同意(我没有想到这个，因为我不知道要搜索 pylint)。但是，我想保留这个问题，因为我希望能够使用相同的方法重新初始化我的类。上一个问题的普遍共识是从 loadData 脚本中返回每个参数，然后将其解析为 self 对象。这很好，但是，我仍然必须在另一个方法中再次执行此操作才能重新初始化我的类实例，这似乎仍然是额外的工作，只是为了增加一点可读性。也许问题是我的例子。在现实生活中，loadData 例程读取了大约 30 个参数，这就是为什么我不愿意在两个不同的位置解析它们。

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

---

原创

这是一个“最佳实践”问题。我最近一直在学习 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')

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

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

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

Answer 1

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

如果给属性赋予最终值不方便或不可能，我建议至少将其初始化为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)

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