Path: csiph.com!usenet.pasdenom.info!news.albasani.net!news.mixmin.net!feeder.erje.net!eu.feeder.erje.net!newsfeed.xs4all.nl!newsfeed6.news.xs4all.nl!xs4all!post.news.xs4all.nl!not-for-mail
Date: Sat, 10 Nov 2012 17:30:14 -0500
From: Dave Angel <d@davea.name>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121011 Thunderbird/16.0.1
MIME-Version: 1.0
To: Jennie <marco.buttu@gmail.com>
Subject: Re: Method default argument whose type is the class not yet defined
References: <k7ma5c$qdk$1@speranza.aioe.org> <mailman.3541.1352579400.27098.python-list@python.org> <509EBE5A.5080405@gmail.com>
In-Reply-To: <509EBE5A.5080405@gmail.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Cc: python-list@python.org
Precedence: list
Reply-To: d@davea.name
Newsgroups: comp.lang.python
Message-ID: <mailman.3545.1352586645.27098.python-list@python.org>
Lines: 46
NNTP-Posting-Host: 2001:888:2000:d::a6
Xref: csiph.com comp.lang.python:33106

On 11/10/2012 03:51 PM, Jennie wrote:
> On 11/10/2012 09:29 PM, Terry Reedy wrote:
>
>> On 11/10/2012 2:33 PM, Jennie wrote:
>>>
>>> I propose three solutions. The first one:
>>>
>>>  >>> class Point:
>>> ...     def __init__(self, x=0, y=0):
>>> ...         self.x = x
>>> ...         self.y = y
>>> ...     def __sub__(self, other):
>>> ...         return Point(self.x - other.x, self.y - other.y)
>>> ...     def distance(self, point=None):
>>> ...         p = point if point else Point()
>>> ...         return math.sqrt((self - p).x ** 2 + (self - p).y ** 2)
>
>> What I do not like about this one is that it creates a new 0 point each
>> time one is needed. Two solutions:
>>
>> change Point() to point0 in the distance function and create
>> point0 = Point()
>> after the class.
>>
>> -or-
>> instead of p = line,
>> px,py = point.x, point.y if point else 0.0, 0.0
>
> Thanks, I like the second one :)
>
I like the first, once you fix the minor inefficiency in it;  add the
qualifier "is None"


...     def distance(self, point=None):
...         p = point if point is None else Point()
...         return math.sqrt((self - p).x ** 2 + (self - p).y ** 2)

The advantage it then has over the second one is that the whole class is
defined inside the class.

-- 

DaveA