On 2013-02-28 19:47, The Night Tripper wrote: > Hi there > I'm being very dumb ... how can I simplify this fragment? > > > if arglist: > arglist.pop(0) > if arglist: > self.myparm1 = arglist.pop(0) > if arglist: > self.myparm2 = arglist.pop(0) > if arglist: > self.myparm3 = arglist.pop(0) > if arglist: > self.parm4 = arglist.pop(0)
If they're arbitrarily named attributes of the "self", you could do something like for attr in ("myparm1", "myparm2", "myparm3", ...): if arglist: setattr(self, attr, arglist.pop(0)) else: break -tkc -- http://mail.python.org/mailman/listinfo/python-list