Is there a more elegant way to do this?

[Kamilche]

'''
Is there a more elegant way of doing this?
I would like to have the arguments to pack
automatically taken from lst.
'''

def pack(self):
lst = ['id', 'parent', 'number', 'x', 'y', 'z', 'red', \
'green', 'blue', 'size', 'rotate', 'translucency']
return struct.pack('<IIIiiiBBBHHH', \
self.id, self.parent, self.number, \
self.x, self.y, self.z, \
self.red, self.green, self.blue, \
self.size, self.rotate, self.translucency)

[Peter Otten]

Kamilche wrote:

'''
Is there a more elegant way of doing this?
I would like to have the arguments to pack
automatically taken from lst.
'''

def pack(self):
lst = ['id', 'parent', 'number', 'x', 'y', 'z', 'red', \
'green', 'blue', 'size', 'rotate', 'translucency']
return struct.pack('<IIIiiiBBBHHH', \
self.id, self.parent, self.number, \
self.x, self.y, self.z, \
self.red, self.green, self.blue, \
self.size, self.rotate, self.translucency)

There is an alternative

struct.pack("<IIIiiiBBBHHH", *[getattr(self, name) for name in lst])

and you could even add the typecodes

lst = [("I", 'id'), ("I", 'parent'), ... ]
struct.pack("<" + "".join([t for (t, n) in lst]),
*[getattr(self, n) for (t, n) in lst])
but frankly, I prefer your original version.

Peter

[Shalabh Chaturvedi]

Kamilche wrote:

'''
Is there a more elegant way of doing this?
I would like to have the arguments to pack
automatically taken from lst.
'''

def pack(self):
lst = ['id', 'parent', 'number', 'x', 'y', 'z', 'red', \
'green', 'blue', 'size', 'rotate', 'translucency']
return struct.pack('<IIIiiiBBBHHH', \
self.id, self.parent, self.number, \
self.x, self.y, self.z, \
self.red, self.green, self.blue, \
self.size, self.rotate, self.translucency)

Note that you don't have to use backslashes when you're inside any kind
of brackets. Makes it look a little cleaner.

--
Shalabh

[Jeff Epler]

something like this:

def pack(self):
lst = ['id', 'parent', 'number', 'x', 'y', 'z', 'red', \
'green', 'blue', 'size', 'rotate', 'translucency']

values = [getattr(self, attr) for attr in lst]
return struct.pack('<IIIiiiBBBHHH', *values)

Jeff

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)

iD8DBQFA3fVyJd01MZaTXX0RAlgYAJ0UCE8OTGA4AqL/B0PxAzulzaWFgwCgppRS
BxN53Hjck5yxly91Eme1+4w=
=ea/I
-----END PGP SIGNATURE-----

[Kamilche]

Peter Otten <__*******@web.de> wrote in message news:<cb*************@news.t-online.com>...

struct.pack("<IIIiiiBBBHHH", *[getattr(self, name) for name in lst])

Thanks Peter! I knew there must be a way, but I don't comprehend list
comprehensions yet.

And I didn't know about the 'don't need backslashes when in a
bracket', thanks Shalabh.

:-)