Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:107090
MIME-Version: 1.0
References: <CABdc3Wpit02=BqvVE85ciw9qo3UYc2LBatOy23yFk5z+quZ2rw@mail.gmail.com>
 <CAKuGYLT7sGn43yXAoYe9fxA70hL9Wrp+P1kDWpUoA--GeR4KnQ@mail.gmail.com> <358fd32a-4d61-7947-293b-8a309e4e31ff@php.net>
In-Reply-To: <358fd32a-4d61-7947-293b-8a309e4e31ff@php.net>
Date: Sat, 14 Sep 2019 11:21:32 +0200
Message-ID: <CABdc3WqyM=zT5GMiCJkzy7xsVs0ZZTKT1top6PtubQjqhcCtdA@mail.gmail.com>
To: PHP Internals List <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="00000000000041795105927fe63a"
Subject: Re: [PHP-DEV] [RFC] Object Initializer
From: michal@brzuchalski.com (=?UTF-8?Q?Micha=C5=82_Brzuchalski?=)

--00000000000041795105927fe63a
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi Sebastian,

sob., 14 wrz 2019 o 11:05 Sebastian Bergmann <sebastian@php.net> napisa=C5=
=82(a):

> Am 13.09.2019 um 15:23 schrieb Matthew Brown:
> > Though this is truly a stylistic complaint, I think it will lead to
> > harder-to-analyse code.
>
> Fully agreed, and not just harder-to-analyse for a static analysis tool
> but also for humans that read the code. -1 from me.
>
>
Could you explain why additional noise caused by constantly repeating
instance variable name and arrow in front of property name and assignment
part
is easier to read? Just wondering why you see object initializer as harder
to read which I've view as easier to read.

Example - easy to read:
function createCustomerDTO(): Customer
{
    $customer =3D new Customer();
    $customer->id =3D Uuid::uuid4;
    $customer->name =3D "John Doe";
    $customer->address =3D "Customer Street 16";
    $customer->city =3D "London";
    $customer->country =3D "GB";
    $customer->phoneNumber =3D PhoneNumber::fromString("+1555 010 020");
    $customer->birthDate =3D new DateTimeImmutable("1983-01-01");
    $customer->email =3D Email::fromString("john.doe@example.com");

    return $customer;
}

Example - hard to read: Why?
function createCustomerDTO(): Customer
{
    return new Customer {
        id =3D Uuid::uuid4,
        name =3D "John Doe",
        address =3D "Customer Street 16",
        city =3D "London",
        country =3D "GB",
        phoneNumber =3D PhoneNumber::fromString("+1555 010 020"),
        birthDate =3D new DateTimeImmutable("1983-01-01"),
        email =3D Email::fromString("john.doe@example.com"),
    };
}

Thanks in advance,
Micha=C5=82 Brzuchalski

--00000000000041795105927fe63a--