OOP - 初始化方法不改变类属性答案

【问题标题】：OOP - init method not changing class attributeOOP - 初始化方法不改变类属性
【发布时间】：2021-08-28 20:50:26
【问题描述】：

我正在尝试在CheckingAccount 类中使用transfer_to_saving 方法。但是，每当我创建SavingAccount 对象时，self.has_saving = True 不会将类属性更改为True。所以，每当我尝试转账时，它都会打印出Must create a saving account。

class CheckingAccount(Account):
    balance = 0

    def __init__(self, account_number, pin):
        super().__init__(account_number)
        self.SavingAccount = SavingAccount
        self.pin = pin

    def deposit(self, amount):
        old_bal = self.balance
        self.balance += amount
        print(f'Previous Balance: ${old_bal}\nDeposit amount: ${amount}\nNew Balance: ${self.balance}')

    def withdraw(self, pin, amount):
        if pin == self.pin:
            self.balance -= print('Insufficient funds') if amount > self.balance else amount
        else:
            print('Invalid PIN')

    def transfer_to_saving(self, amount):
        if self.SavingAccount.has_saving is False:
            print('Must create a saving account')
        elif amount > self.balance:
            print('Insufficient funds')
        else:
            self.SavingAccount.balance += amount
            self.balance -= amount


class SavingAccount(Account):
    balance = 0
    has_saving = False

    def __init__(self, account_number):
        super().__init__(account_number)
        self.CheckingAccount = CheckingAccount
        self.has_saving = True

    def deposit(self, amount):
        self.balance += amount

我这样做对吗？ init 方法不应该改变类属性吗？

---更新---

我想要完成的目标是查明用户是否已经创建了储蓄账户。我有额外的用户类，我没有包括在内，因为它有点矫枉过正。但是，目标是防止用户在没有储蓄账户的情况下将资金从支票转为储蓄。

【问题讨论】：

你想做的可能不是你想做的。如果不知道为什么要拥有一个与已经代表储蓄账户的类或该类的实例相关联的 has_saving 标志变量，这很难说。正如其中一个答案所指出的那样，您可能会滥用对类名的引用并将该名称的实例变量分配给 self.同名变量。
是的，这就是我正在做的，滥用对类名的引用。我很感激反馈。我正在做一些改变。

标签： python class init class-attributes

【解决方案1】：

您实际上并没有使用此行创建SavingAccount 的实例：

        self.SavingAccount = SavingAccount

您正在将self.SavingAccount 属性分配给下面定义的SavingAccount 类。

你需要调用SavingAccount构造函数，像这样：

        self.saving_account = SavingAccount(account_number)

请注意，Python 约定是使用lower_snake_case 表示属性/变量，使用UpperCamelCase 表示类名。

你在 SavingAccount 构造函数的这一行做同样的事情：

        self.CheckingAccount = CheckingAccount

我不确定这里的目标是什么，但如果您希望每个 SavingAccount 都持有对 CheckingAccount 的引用，反之亦然，这样做可能更简洁：

class CheckingAccount(Account):

    def __init__(self, account_number, pin):
        super().__init__(account_number)
        self.saving_account = SavingAccount(account_number, self)
        self.pin = pin

class SavingAccount(Account):
    def __init__(self, account_number, checking_account):
        super().__init__(account_number)
        self.checking_account = checking_account

这样，每当您创建一个CheckingAccount 时，您都会得到一个相应的SavingAccount，并且它们将各自持有对彼此的引用。我认为这在概念上仍然有点奇怪，因为帐号是相同的，所以最好像这样单独创建它们：

class CheckingAccount(Account):
    def __init__(self, account_number, pin):
        super().__init__(account_number)
        self.saving_account = None  # to be assigned later
        self.pin = pin

class SavingAccount(Account):
    def __init__(self, account_number):
        super().__init__(account_number)
        self.checking_account = None  # to be assigned later


checking_account_number = 123
checking = CheckingAccount(checking_account_number)
saving_account_number = 456
saving = SavingAccount(saving_account_number)
checking.saving_account = saving
saving.checking_account = checking

最后，SavingAccount 的has_saving 属性根本不需要。检查SavingAccount 是否更简洁的方法是使用isinstance：

    def transfer_to_saving(self, amount):
        if not isinstance(self.saving_account, SavingAccount):
            print('Must create a saving account')
        elif amount > self.balance:
            print('Insufficient funds')
        else:
            self.saving_account.balance += amount
            self.balance -= amount

【讨论】：

非常感谢您的详尽解释（以及您提供的所有额外好处）！
self.saving_account = None # to be assigned later 这正是我希望做的。再次感谢您。

【解决方案2】：

当您执行self.has_saving = True 时，您正在建立一个作为实例一部分的实例变量。您没有修改类变量。要修改类变量，您需要使用类名而不是 self. 引用。

【讨论】：