Path: csiph.com!usenet.pasdenom.info!aioe.org!news.stack.nl!newsfeed.xs4all.nl!newsfeed1.news.xs4all.nl!xs4all!post.news.xs4all.nl!not-for-mail
Return-Path: <ian.g.kelly@gmail.com>
X-Original-To: python-list@python.org
Delivered-To: python-list@mail.python.org
X-Spam-Status: OK 0.115
X-Spam-Level: *
X-Spam-Evidence: '*H*': 0.78; '*S*': 0.01; 'defaults': 0.05; 'subject:code': 0.07; 'wrote:': 0.17; 'thu,': 0.17; 'feb': 0.19; 'header:In-Reply-To:1': 0.25; 'subject:please': 0.27; 'message- id:@mail.gmail.com': 0.27; "i'm": 0.29; 'to:addr:python-list': 0.33; 'skip:d 20': 0.34; 'received:google.com': 0.34; 'pm,': 0.35; 'received:209.85': 0.35; 'there': 0.35; 'being': 0.37; 'received:209': 0.37; 'subject:: ': 0.38; 'to:addr:python.org': 0.39; 'night': 0.62; 'you:': 0.75; '2013': 0.84; 'dumb': 0.84; 'to:name:python': 0.84
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=x-received:mime-version:in-reply-to:references:from:date:message-id :subject:to:content-type; bh=7DkAQZwzzdxi7w4hFPFItdjBVB4nrhDt4Xkq9ub3vsk=; b=LkHRzT9f1sJte5d1LjdCD3Ebz+L0LAUeqUVt1LRkJRmeQMpTqcC9QZ3TARBmU/6auS v49b4hz/wCLtchGNdt+3w00cIzcgNCSnvekD4KU+SgCx8LKsGK4Lb6VtziXaBVX3Cb1J P+E/2Olxt6uJeT4HFFL6OpXqDS0ssJa4K9/YSy2YjqNnC6S8AHxVAXptjiKu84atHow2 19ef73auTiXfJzoj3wmt0FtpntHziolijTY4XQnnztHgQBsbIXBbvbzCJK8ideR4XiPj o+KvpOFhnsoAT8F6RmK4oRhWJPUpX8hJBmPyCZOXBwBkDZlkXIb8G8ICyqI+/fx3aWUR FKSA==
X-Received: by 10.52.177.163 with SMTP id cr3mr2642398vdc.94.1362081541594; Thu, 28 Feb 2013 11:59:01 -0800 (PST)
MIME-Version: 1.0
In-Reply-To: <u8idnaoktKrcKbLMnZ2dnUVZ8uednZ2d@brightview.co.uk>
References: <u8idnaoktKrcKbLMnZ2dnUVZ8uednZ2d@brightview.co.uk>
From: Ian Kelly <ian.g.kelly@gmail.com>
Date: Thu, 28 Feb 2013 12:58:21 -0700
Subject: Re: suggestions for improving code fragment please
To: Python <python-list@python.org>
Content-Type: text/plain; charset=ISO-8859-1
X-BeenThere: python-list@python.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: General discussion list for the Python programming language <python-list.python.org>
List-Unsubscribe: <http://mail.python.org/mailman/options/python-list>, <mailto:python-list-request@python.org?subject=unsubscribe>
List-Archive: <http://mail.python.org/pipermail/python-list/>
List-Post: <mailto:python-list@python.org>
List-Help: <mailto:python-list-request@python.org?subject=help>
List-Subscribe: <http://mail.python.org/mailman/listinfo/python-list>, <mailto:python-list-request@python.org?subject=subscribe>
Newsgroups: comp.lang.python
Message-ID: <mailman.2671.1362081549.2939.python-list@python.org>
Lines: 22
NNTP-Posting-Host: 2001:888:2000:d::a6
X-Trace: 1362081549 news.xs4all.nl 6937 [2001:888:2000:d::a6]:35112
X-Complaints-To: abuse@xs4all.nl
Xref: csiph.com comp.lang.python:40167

On Thu, Feb 28, 2013 at 12:47 PM, The Night Tripper <jkn+gg@nicorp.co.uk> 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)

Perhaps this would work for you:

if arglist:
    defaults = [self.parm1, self.parm2, self.parm3, self.parm4]
    arglist = arglist[1:] + defaults[len(arglist)-1:]
    self.parm1, self.parm2, self.parm3, self.parm4 = arglist[:4]