Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:92522
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain pthreads.org from 209.85.161.176 cause and error)
MIME-Version: 1.0
In-Reply-To: <CAL=_i_=V-pZcaTQcrt7mJ6SkJk+BkOa+4G4wXkZxo7Ph8rJzrQ@mail.gmail.com>
References: <CAL=_i_==wsq5j2Jf1QdusxaCBU8COX3eJvZY+puD8zH1zCrTXw@mail.gmail.com>
	<CAF+90c_3Sj7zqJaQy6u=3RQOxh+Y7uZ7Eo-pW2m=Ty95npfjfA@mail.gmail.com>
	<CAF1yvAS0nq_imYHN7OuxVLhWhf-qP9mW7a1V3cpyuxDBPoXGog@mail.gmail.com>
	<CAL=_i_=V-pZcaTQcrt7mJ6SkJk+BkOa+4G4wXkZxo7Ph8rJzrQ@mail.gmail.com>
Date: Wed, 20 Apr 2016 07:36:43 +0100
Message-ID: <CAL=_i_=aiADx9Z6XzDVdLO_Ra_iD4RmxFmUjk1Ty-8MZOcxaSw@mail.gmail.com>
To: Jesse Schalken <me@jesseschalken.com>
Cc: Nikita Popov <nikita.ppv@gmail.com>, PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a1142926234120c0530e4d1bb
Subject: Re: [PHP-DEV] RFC: Anonymous Class Lexical Scope
From: pthreads@pthreads.org (Joe Watkins)

--001a1142926234120c0530e4d1bb
Content-Type: text/plain; charset=UTF-8

Oh, I fixed it ... I think ... carry on ...

Cheers
Joe

On Wed, Apr 20, 2016 at 7:01 AM, Joe Watkins <pthreads@pthreads.org> wrote:

> Can we just halt this conversation for a few days.
>
> That implementation is totally and utterly wrong, it's so obvious this
> morning ... I'm almost embarrassed.
>
> I must say that of all the suggestions made so far for syntax, I still
> prefer use().
>
> While the idea to allow symbols and expressions is very cool, it will
> create so much confusion that an anonymous class can do things like set
> objects as member properties but other classes can't.
>
> The syntax also looks absolutely terrible when it comes to refs:
>
> class {
>     public $thing = &$this->thing;
> }
>
> This is almost literally *backwards*.
>
> All of the solutions suggested so far also create rather a lot of
> repetition for anything that isn't example code.
>
> Anyway, the implementation is broken, let's come back to this discussion
> later on, please.
>
> Cheers
> Joe
>
> On Wed, Apr 20, 2016 at 4:54 AM, Jesse Schalken <me@jesseschalken.com>
> wrote:
>
>> I'm not sure it matters, but there is some precedent for this in
>> JavaScript/ES6:
>>
>> function foo(bar) {
>>     return new class {
>>         myBar = bar;
>>
>>         getBar() {
>>             return this.myBar;
>>         }
>>     }();
>> }
>>
>> console.log(foo('hello').getBar()); // "hello"
>>
>>
>> (You can actually reference outer symbols anywhere in the definition of a
>> JS class, so I could have used "bar" directly in "getBar" without going
>> through a property.)
>>
>>
>> On Wed, Apr 20, 2016 at 12:18 AM, Nikita Popov <nikita.ppv@gmail.com>
>> wrote:
>>
>>> On Tue, Apr 19, 2016 at 3:31 PM, Joe Watkins <pthreads@pthreads.org>
>>> wrote:
>>>
>>> > Morning Internals,
>>> >
>>> >     Please review the following RFC:
>>> >
>>> >     https://wiki.php.net/rfc/lexical-anon
>>> >
>>> >     A look at the patch from those of you that do that would be good :)
>>> >
>>>
>>> Hey Joe,
>>>
>>> The syntax and semantics proposed in this RFC don't sit quite well with
>>> me.
>>> Especially the fact that a use($foo) on the class is then used as
>>> $this->foo in methods is non-intuitive to me, as it differs from how the
>>> same syntax behaves on closures. I'd like to suggest an alternative
>>> syntax:
>>>
>>> $foo = 42;
>>> return new class {
>>>     private $bar = $foo;
>>>     public function getBar() { return $this->bar; }
>>> }
>>>
>>> That is, allow properties inside the anonymous class to be initialized
>>> based on values from the surrounding scope. This is more explicit
>>> (clearly
>>> shows that a property is being created), it allows explicit control over
>>> the visibility and, depending on implementation, might be more flexible
>>> with regards to the values it accepts. It probably doesn't make sense to
>>> restrict this to specific expressions, so all of
>>>
>>> return new class {
>>>     private $a = $var;
>>>     private $b = $obj->prop;
>>>     private $d = $obj->prop ?? 'default';
>>>     // ...
>>> }
>>>
>>> could be fine.
>>>
>>> Thanks,
>>> Nikita
>>>
>>
>>
>

--001a1142926234120c0530e4d1bb--