31fps

I am no beginner in Python, but I’m definitely no guru. So I still have problems expressing some things. It’s like learning Spanish — you know enough of the language for others to understand you, but there is always an easier way to convey what you want to say. I know enough Python for the computer to understand me, but here is a situation that just came up that shows there’s always an easier, shorter way to do things.

def _get_custodian_clients(self):
  "Returns all the clients of the custodian"
  str = ''
  i = 1
  total_cli = self.custodian_total_clients
  for cli in self.clients.all():
    str += cli.name
    i += 1
    if i <= total_cli:
      str += ' | '
  return str


I knew there had to be an easier way to do it, but I just couldn’t figure it out. So I asked people that knew more than me, i.e., the guys on the #django IRC Channel. Two minutes later, and I have a nice, elegant, even readable solution.


def _get_custodian_clients(self):
  "Returns all the clients of the custodian"
  return ' | '.join([client.name for x in self.clients.all()])
custodian_clients = property(_get_custodian_clients)


Sometimes I am amazed by my stupidity.