Понимание ошибки: объект 'str' не является вызываемым

Question

Я просто работаю над примером, чтобы помочь мне узнать, как работает ООП на Python. Вот класс, я работаю с:

class account(object): 
    def __init__(self,holder,number,balance,credit_line=1500): 
     self.holder=holder 
     self.number=number 
     self.balance=balance 
     self.credit_line=credit_line 

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

    def withdraw(self,amount): 
     if (self.balance-amount < -self.credit_line): 
      #coverage insufficient 
      return False 
     else: 
      self.balance-=amount 
      return True 

    def balance(self): 
     return self.balance 

    def transfer(self,target,amount): 
     if (self.balance-amount < -self.credit_line): 
      #coverage insufficient 
      return False 
     else: 
      self.balance-=amount 
      target.balance+=amount 
      return True 

А вот программа драйвера я использую, чтобы проверить его:

import account 

john=account.account("John Doe","12345","1000.00") 
res=john.balance() 
print "%r" %res 
john.deposit(1500) 
res=john.balance() 
print "%r" %res 

Я получаю сообщение об ошибке при попытке запустить это:

Traceback (most recent call last): 
    File "banker.py", line 4, in <module> 
    res=john.balance() 
TypeError: 'str' object is not callable 

У кого-нибудь есть идеи, почему это так?

Answer 1

Даже если self.balance определяется как метод в своем классе, он заменяется строкой во время __init__ когда self.blance=balance. Поэтому, когда вы вызываете john.balance() в этом случае, где balance является строкой "1000.00", она возвращает довольно полезную ошибку TypeError: 'str' object is not callable.

Рекомендации:

1. использовать разные имена для метода и для атрибута.
2. Похоже, что баланс - это число. Зачем передавать его как строку?
3. Метод balance в настоящее время просто возвращает значение. На самом деле нет необходимости в методе в текущем примере. Но если вы хотите развить его, чтобы делать все возможное для каждого вызова, конечно, это путь.

Answer 2

Вы не можете дать одно и то же имя методу и атрибуту вашего класса. Вам не нужен метод balance(), и вообще python не использует методы getter и setter для доступа к атрибутам. Class_name.attribute_name возвращает значение Вашего атрибута

def balance(self): 
     return self.balance 

Answer 3

Вы маскируете атрибуты своего объекта.

self.balance=balance 

и

def balance(self): 

Python не различает self.balance число и self.balance функция. Независимо от того, что назначено последним, это тот, который придерживается. Дайте каждому атрибуту уникальное имя.

Answer 4

Вы переписали свою функцию balance с вашим атрибутом balance. Вы должны переименовать свой атрибут, например, self._balance.