Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:112394
MIME-Version: 1.0
Content-Type: multipart/alternative; boundary="=_6026e522172b37bf73d9276bd8c1e8e7"
Date: Thu, 03 Dec 2020 10:51:18 +0100
To: =?UTF-8?Q?Micha=C5=82_Marcin_Brzuchalski?= <michal.brzuchalski@gmail.com>
Cc: PHP internals <internals@lists.php.net>
In-Reply-To: <CABdc3Wqei1ssqnWFS1eAafBVUgo3gWdqEUjG+rrXLb-AEQ_-PA@mail.gmail.com>
References: <91312fb6f300ab9fed5025fc009832c7dd0a15964dc828936e3dc2d0aabb2a1a@mahalux.com>
 <CABdc3Wqei1ssqnWFS1eAafBVUgo3gWdqEUjG+rrXLb-AEQ_-PA@mail.gmail.com>
Message-ID: <2166180e533b1a5aa2f95f29d44cef17435d6b951621b83ed045174cf2bbd9e1@mahalux.com>
Subject: Re: [PHP-DEV] Allow new Cl()->method()
From: vorismi3@fel.cvut.cz (=?UTF-8?Q?Michael_Vo=C5=99=C3=AD=C5=A1ek_-_=C4=8CVUT_FEL?=)

--=_6026e522172b37bf73d9276bd8c1e8e7
Content-Transfer-Encoding: 8bit
Content-Type: text/plain; charset=UTF-8; format=flowed

In my opinion: 

- the operator table docs should be updated to contain all operators (
see comment
https://www.php.net/manual/en/language.operators.precedence.php#121509 )


- "new" should have the highest absolute priority, ie. new
Cl()->method(); should be always interpreted as (new Cl())->method();,
new (Cl()->method()); is very unusual and parenthesis can be always used
to enforce the intended precedence. 

With kind regards / Mit freundlichen Grüßen / S přátelským pozdravem,

Michael Voříšek
ČVUT FEL

On 3 Dec 2020 07:45, Michał Marcin Brzuchalski wrote:

> Hi Michael, 
> 
> śr., 2 gru 2020 o 21:03 Michael Voříšek - ČVUT FEL <vorismi3@fel.cvut.cz> napisał(a): 
> 
>> Hi devs, 
>> 
>> currently new Cl() must be wrapped in parentheses if that object is
>> further used to call it's method like (new Cl())->method(). 
>> 
>> It there any reason why not to allow new Cl()->method() ? 
>> 
>> Based on https://www.php.net/manual/en/language.operators.precedence.php
>> - the "new" operator has already the highest precedence, thus there
>> should be no conflict.
> 
> I think that's because removing the parentheses may confuse what was the original intention of author. 
> 
> class CI {
> public function method() { echo 'foo'; }
> }
> class bar {
> public function __construct() {
> echo 'bar';
> }
> }
> function CI() {
> return new class {
> public function method() { return 'bar'; }
> };
> }
> (new CI())->method(); // foo
> new (CI()->method()); // bar 
> 
> Both ways are valid code in PHP8 with no error/warning, till PHP7.4 there was a 
> 
> Parse error: syntax error, unexpected '(' 
> 
> in the second object construction. 
> 
> Cheers, 
> Michał Marcin Brzuchalski
--=_6026e522172b37bf73d9276bd8c1e8e7--