forループでif文をスキップするプログラム

Question

クライアントの詳細を関数で更新しようとしています。しかし、それは動作しません。私はプログラムをデバッグし、ifには絶対に入っていないことに気付きました。どうしてか分かりません？

これは、関数である：私はクライアントを更新しようとすると

def UpdateClient(self,id,cnp,name,adress): 
    for i in range (len(self.clients.get_all())): 
     if self.clients.get_all()[i].get_id==id: 
      self.clients.get_all()[i].set_name(name) 
      self.clients.get_all()[i].set_cnp(cnp) 
      self.clients.get_all()[i].set_adress(adress) 

、私はid、name、cnp、およびadressを与えるが、私は何も変更されていないリストを印刷するとき。デバッガでifに入ることはできません。なぜなら、それらは決して等しくないからです。どうして？

Answer 1

checkをget_id == idとしていますが、それは機能です。また、

if self.clients.get_all()[i].get_id()==id: 
#         ^making it a function call 

これを行うには良い方法は、別の変数にself.clients.get_all()を格納することである：あなたのifは次のようにする必要があります。 get_idがget_id()と同じではないことを

def UpdateClient(self,id,cnp,name,adress): 
    clients = self.clients.get_all() 
    for i in range (len(clients)): 
     if clients[i].get_id()==id: 
      clients[i].set_name(name) 
      clients[i].set_cnp(cnp) 
      clients[i].set_adress(adress) 

Answer 2

注：したがって、あなたのコードは次のようになります。前者はメソッドを参照し、後者は実際にメソッドを呼び出します。

さらに、慣用的なPythonを書くことでコードを単純化することを検討してください。例：

for client in self.clients.get_all(): 
     if client.get_id() == id: 
      client.set_name(name) 
...