Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:107059
MIME-Version: 1.0
References: <CABdc3Wpit02=BqvVE85ciw9qo3UYc2LBatOy23yFk5z+quZ2rw@mail.gmail.com>
 <CANS-=peR=rcZeHcRvZFxJPDsA402MKv_BxZ=ftU=Zo8Nad4CdQ@mail.gmail.com>
 <CABdc3WravK6Xgqd9zcskdmVAxO+ewekXB=HsSc56+HVfcWDa+g@mail.gmail.com>
 <74C03BB1-28DF-4694-A91D-2516C545F8B2@gmail.com> <3CA484B0-BCA3-4A2B-B2A6-8B845F9C462B@newclarity.net>
In-Reply-To: <3CA484B0-BCA3-4A2B-B2A6-8B845F9C462B@newclarity.net>
Date: Fri, 13 Sep 2019 10:51:39 +0200
Message-ID: <CANZOY=AMc6rj1s-GLXUr1X72LHBxpROzOV5=_AT-wfuDOdSFAg@mail.gmail.com>
To: PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="0000000000007dc9fe05926b5d4a"
Subject: Re: [PHP-DEV] [RFC] Object Initializer
From: arnold.adaniels.nl@gmail.com (Arnold Daniels)

--0000000000007dc9fe05926b5d4a
Content-Type: text/plain; charset="UTF-8"

This can easily be done in a number of ways, like the suggested helper
function. The same can be said for many other features that were
implemented recently, like array unpacking. This feature is easy to
implement and will make the code for data objects much more readable, with
additional benefits for static analysis.

Comments:

I don't like how it works for anonymous classes. It's more difficult to
implement since the compiler doesn't know the meaning of the (first)
bracket. It's doesn't make the code more readable, for the same reason. I
think it's better to not support this syntax with anonymous classes.

The examples do not show how constructor arguments are passed. I'm assuming
it's

    $customer = new Customer("foo") {
        name = "John"
    };

About the idea of letting `{ foo = 10 }` create an `stdClass` object (not
in the RFC); While not used much since it has no effect, it's perfectly
okay to put your code in brackets eg `{ { { $foo = 10; } } }`. As such, I
don't think it's a good idea to allow `new stdClass` to be omitted.

--0000000000007dc9fe05926b5d4a--