suggestions for improving code fragment please

Tim Chase python.list at tim.thechases.com
Thu Feb 28 15:37:15 EST 2013


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






More information about the Python-list mailing list