Re: suggestions for improving code fragment please

Discussion in 'Python' started by Dave Angel, Feb 28, 2013.

  1. Dave Angel

    Dave Angel Guest

    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
    Dave Angel, Feb 28, 2013
    #1
    1. Advertising

Want to reply to this thread or ask your own question?

It takes just 2 minutes to sign up (and it's free!). Just click the sign up button to choose a username and then you can ask your own questions on the forum.
Similar Threads
  1. Rick Johnson
    Replies:
    0
    Views:
    93
    Rick Johnson
    Feb 28, 2013
  2. Joel Goldstick
    Replies:
    0
    Views:
    100
    Joel Goldstick
    Feb 28, 2013
  3. Ian Kelly
    Replies:
    0
    Views:
    92
    Ian Kelly
    Feb 28, 2013
  4. Tim Chase
    Replies:
    0
    Views:
    94
    Tim Chase
    Feb 28, 2013
  5. MRAB
    Replies:
    0
    Views:
    90
Loading...

Share This Page