Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:63680
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain garfieldtech.com from 66.111.4.26 cause and error)
Message-ID: <508C9B4A.5090801@garfieldtech.com>
Date: Sat, 27 Oct 2012 21:41:14 -0500
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121011 Thunderbird/16.0.1
MIME-Version: 1.0
To: internals@lists.php.net
References: <508A695F.8080308@zerocue.com>
In-Reply-To: <508A695F.8080308@zerocue.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] [RFC] Property Accessors v1.2 : Extra shorthand declaration
From: larry@garfieldtech.com (Larry Garfield)

On 10/26/2012 05:43 AM, Clint Priest wrote:
> I'm opening up several new threads to get discussion going on the 
> remaining "being debated" categories referenced in this 1.1 -> 1.2 
> change spec:
> https://wiki.php.net/rfc/propertygetsetsyntax-as-implemented/change-requests 
>
>
> ------------------------------------------------------------------------
> *Extra shorthand declaration*
>
> Allow extra short declaration of an accessor:
>
> class TimePeriod {
>     public DateTime $date;
> }
>
> /* Would be equivalent to this */
>
> class TimePeriod {
>     public $date {
>         get() { return $this->date; }
>         set(DateTime $value) { $this->date = $value;}
>     }
> }
>
> ------------------------------------------------------------------------
> Thoughts?
>

My knee-jerk response is that I'd love to be able to do public DateTime 
$date; it's nicely self-documenting and gives me very simple type safety 
(and therefore a clear and useful error thrown in my face, rather than a 
mystery bug later).  That said, I think that goes back to my mental 
model question in my previous post, and I can't fully endorse this 
without knowing my mental model.

--Larry Gafield