Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:114617
MIME-Version: 1.0
References: <CAOwTYAtc_xD6n=efmamkAy4ro4KqPKxq18-zXsNriY0280FY2g@mail.gmail.com>
In-Reply-To: <CAOwTYAtc_xD6n=efmamkAy4ro4KqPKxq18-zXsNriY0280FY2g@mail.gmail.com>
Date: Wed, 26 May 2021 14:57:27 +0200
Message-ID: <CABdc3WrwPZJcYnuaaPh6Q0Y60nED0ie3xOAQcZTNJiDtPqSY0g@mail.gmail.com>
To: Joe Watkins <krakjoe@php.net>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000f4d65105c33b2f18"
Subject: Re: [PHP-DEV] Consensus Gathering: is_initialized
From: michal.brzuchalski@gmail.com (=?UTF-8?Q?Micha=C5=82_Marcin_Brzuchalski?=)

--000000000000f4d65105c33b2f18
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

=C5=9Br., 26 maj 2021 o 12:14 Joe Watkins <krakjoe@php.net> napisa=C5=82(a)=
:

> Hi internals,
>
> In response to: https://bugs.php.net/bug.php?id=3D78480
>
> I implemented: https://github.com/php/php-src/pull/7029
>
> Not absolutely convinced that it's a good idea, I asked Nikita to review,
> and he's unconvinced also and suggested a discussion should be started.
>
> You can read both of our thoughts about it on the PR.
>
> What I'm looking for now, is reasons that we are wrong - good use cases
> that aren't born from code that is doing bad things ...
>

IMHO we don't need it, working with the typed properties since the
beginning
I thought only for a short period that I'd need this kind of feature but
shortly realized
that I'm doing it wrong if my code can reach such state,
given that all paths now lead to a known state,
object construction in all cases specifies all required properties
without having any doubts like uninitialized properties so this is doable.

If one really needs to check if a property is initialized or not I consider
it a dark magic
and such can reach up to the reflection mechanism to just check it.
IMHO it's not common (or at least should not be) to have such a need.

Cheers,
Micha=C5=82 Marcin Brzuchalski

--000000000000f4d65105c33b2f18--