Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:63290
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.214.170 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <9570D903A3BECE4092E924C2985CE485612B4353@MBX202.domain.local>
References: <9570D903A3BECE4092E924C2985CE485612B3B48@MBX202.domain.local>
 <5073328D.5000002@gmail.com> <50735165.8010703@aaronholmes.net>
 <CAEPJdu=pRSDRD68t3o7XVbWRo-FvA07Wr+bzM_bwOXkwfiMxng@mail.gmail.com>
 <CACxA8bkF0EHDs3As=kMcg-BahRfBK6A5i7B9TdRc_SweCsv23A@mail.gmail.com> <9570D903A3BECE4092E924C2985CE485612B4353@MBX202.domain.local>
Date: Mon, 8 Oct 2012 20:08:24 -0700
Message-ID: <CACxA8b=wbHN_mRomH25Jr+i9BeTzj=VTGfPe_T8igjzCsMNn_Q@mail.gmail.com>
To: Clint Priest <cpriest@zerocue.com>
Cc: "internals@lists.php.net" <internals@lists.php.net>, Aaron Holmes <aaron@aaronholmes.net>, 
	Benjamin Eberlei <kontakt@beberlei.de>
Content-Type: multipart/alternative; boundary=f46d044516ddf1ac1e04cb97a7f5
Subject: Re: [PHP-DEV] [RFC] Propety Accessors v1.1
From: tbprogrammer@gmail.com (Jazzer Dane)

--f46d044516ddf1ac1e04cb97a7f5
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

While I understand your concern with set being the only keyword using (),
and even agree it's a bit problematic, I see a big problem with using
$value.

Even if $value internally makes sense due to something along the lines of *
__setHours($value)* {} being equal to *set {}*, I think using $value
without it ever being defined in the developer's code is not at all a good
idea.
If I see $value in the code, I'll logically look for where it was defined,
and when I don't see it anywhere else in the code, things are going to very
quickly get confusing.
Our best option to combat this confusion is, in my eyes, putting a note in
the documentation. That's not enough.

A similar alternative to using $value that I'd argue would be much more
sensible would be to, as Denis mentioned, use either a magic constant or a
superglobal.

As I mentioned previously, I would rather go with the set($value) {} syntax=
.

Now, back to the part where I agree with you - the inconsistency wherein
set has () that denote it is a method but get, isset, and unset do not. I
see this inconsistency as something problematic enough to warrant a
solution.

We could go with the following:
public $Hours {
  get() { ... }
  set($value) { ... }
  isset() { ... }
  unset() { ... }
}

Yes, we now have a little bit more meat on the syntax, but in this case, I
don't think that it's all that bad. Here's two reasons why:
1) Adding parenthesis denotes that they are all functions - which they are!
If anything, adding parenthesis to all of them makes the implementation *
more* sensible.
2) It's *only* two more characters per function. On top of that, in my
opinion, this syntax is not "ugly". In fact, as I just mentioned - this
implementation is arguably *more* consistent with the rest of PHP.

On Mon, Oct 8, 2012 at 6:10 PM, Clint Priest <cpriest@zerocue.com> wrote:

> Seems a fair amount of people would like it with a definable parameter
> name, though the original RFC I based mine off of is more than 4 years ol=
d
> (mine is over a year old already).
>
> The $value is precisely chosen because it is exactly the way C# operates
> and the original author thought to keep it the same as another well-known
> language (why re-invent new syntax for no reason).
>
> That being said, javascript does indeed allow it, my concern then would b=
e
> would we have the parameterized syntax only for set() and not get, isset =
or
> unset?
>
> If we do have them for all of them, it's a lot of extra characters with n=
o
> real need.
>
> I definitely favor set($value) over a magic $Hours for the $Hours
> property, but I personally see no problem with the $value, it's not magic
> it's a locally defined variable.
>
> Internally, this:
>    public $Hours {
>       get { ... }
>       set { ... }
>    }
>
> Is implemented as standard functions, while they are hidden through
> reflection, these functions exist (as a result of the above example):
>
> public __getHours() { ... }
> public __setHours($value) { ... }
>
> Lastly, with regards to JavaScript style getters/setters, I don't think
> I've ever cared what the variable name was, I typically just do something
> like:
>
> set blah(x) { ... } <-- x is fairly irrelevant and similarly the use of
> $value is fairly irrelevant.   Thoughts?
>
> > -----Original Message-----
> > From: Jazzer Dane [mailto:tbprogrammer@gmail.com]
> > Sent: Monday, October 08, 2012 5:32 PM
> > To: Benjamin Eberlei
> > Cc: Aaron Holmes; internals@lists.php.net
> > Subject: Re: [PHP-DEV] [RFC] Propety Accessors v1.1
> >
> > I agree.
> > It's more consistent than the $Hours solution and we don't have to add
> another superglobal or magic constant, which is quite nice. The
> > typehinting is a big plus as well.
> >
> > On Mon, Oct 8, 2012 at 3:26 PM, Benjamin Eberlei <kontakt@beberlei.de
> >wrote:
> >
> > > The set() one is really nice with the typehints.
> > >
> > > On Tue, Oct 9, 2012 at 12:19 AM, Aaron Holmes <aaron@aaronholmes.net>
> > > wrote:
> > >
> > > > On 10/8/12 1:07 PM, Denis Portnov wrote:
> > > >
> > > >> 08.10.2012 15:52, Clint Priest =D0=BF=D0=B8=D1=88=D0=B5=D1=82:
> > > >>
> > > >>>      public $Hours {
> > > >>>          get { return $this->Seconds / 3600; }
> > > >>>          set { $this->Seconds =3D $value; }
> > > >>>          isset<http://www.php.net/isset**>  { return isset<
> > > >>> http://www.php.net/isset**>($this->Seconds); }
> > > >>>          unset<http://www.php.net/unset**>  { unset<
> > > >>> http://www.php.net/unset**>($this->Seconds); }
> > > >>>      }
> > > >>>
> > > >>
> > > >>
> > > >> Hi Clint,
> > > >>
> > > >> I've noticed some magic variable '$value' is introduced. And excep=
t
> > > >> for superglobals I guess there is no such thing in PHP, so it look=
s
> > > >> bit puzzling to me. I'd suggest on of the following:
> > > >>
> > > >>
> > > >> - setter resambles setter method, wich also allows typehinting
> > > >>     public $Hours {
> > > >>         set ($value) { $this->Seconds =3D $value * 3600; }
> > > >>     }
> > > >>
> > > >>     public $Hours {
> > > >>         set (DateTime $dateTime) { $this->Seconds =3D
> > > >> $dateTime->getTimestamp(); }
> > > >>     }
> > > >>
> > > >>  This seems like the cleanest method, in my opinion. Javascript
> > > >> does
> > > this
> > > > for object prototypes:
> > > > http://ejohn.org/blog/**javascript-getters-and-**setters/<
> > > http://ejohn.org/blog/javascript-getters-and-setters/>
> > > >
> > > >
> > > >>
> > > >> What do you think?
> > > >>
> > > >> Thanks
> > > >> Denis
> > > >>
> > > >>
> > > >
> > > > --
> > > > PHP Internals - PHP Runtime Development Mailing List To unsubscribe=
,
> > > > visit: http://www.php.net/unsub.php
> > > >
> > > >
> > >
>

--f46d044516ddf1ac1e04cb97a7f5--