suggestions for improving code fragment please

[Dave Angel]

On 02/28/2013 03:37 PM, Tim Chase wrote:
> 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
>
>
>
Or something like (untested):

     for name, value in zip(["myparm1", "myparm2", "myparm3"], arglist):
          setattr(self, name, value)
     arglist = []     #if you care how it ends up


-- 
DaveA