Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:114610
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 12:30:25 +0200
Message-ID: <CADyq6sKZoC0GWE__=ODENUmOr+wrzCiNr7ZBuRgurpPacSU+VA@mail.gmail.com>
To: Joe Watkins <krakjoe@php.net>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="0000000000001eb63505c339228a"
Subject: Re: [PHP-DEV] Consensus Gathering: is_initialized
From: ocramius@gmail.com (Marco Pivetta)

--0000000000001eb63505c339228a
Content-Type: text/plain; charset="UTF-8"

On Wed, May 26, 2021 at 12:14 PM Joe Watkins <krakjoe@php.net> wrote:

> Hi internals,
>
> In response to: https://bugs.php.net/bug.php?id=78480
>
> 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 ...
>
> Equally if you think this is really bad even if the reason has already been
> mentioned, please make noise.
>


What's the problem with good 'ol `(new ReflectionProperty(Foo::class,
'bar'))->isInitialized($baz)` ?

If it's instantiation time/performance, I'm fairly sure that typical
reflection instantiation caching works just fine (and a new API would need
a lot of meaningful benchmarks to be warranted).

Marco Pivetta

http://twitter.com/Ocramius

http://ocramius.github.com/

--0000000000001eb63505c339228a--