Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:123667
Feedback-ID: i8414410d:Fastmail
User-Agent: Cyrus-JMAP/3.11.0-alpha0-522-ga39cca1d5-fm-20240610.002-ga39cca1d
Precedence: bulk
MIME-Version: 1.0
Message-ID: <8fc1969d-5f21-400d-a3ae-aee5d741a81d@app.fastmail.com>
In-Reply-To: 
 <CAP1Jc10nkYctLsDWXUi+S-QdZ-J8zrfnEsw7ZAAjsHx9u5JQuQ@mail.gmail.com>
References: 
 <CAOWwgpmbq5VRrZQvUXDsKiNK4r6+bFA4VxnjQ_U=h8T9r0o3DA@mail.gmail.com>
 <ab83af79-0669-47dd-a3cb-ab72327ae174@bastelstu.be>
 <CAP1Jc13JWVg99AULgzrGXcoCWJtJZGz+nr_VoyUocw8n5n5sfQ@mail.gmail.com>
 <1118bbcd-a7b4-47bf-bf35-1a36ab4628e1@bastelstu.be>
 <CAP1Jc11vNiazMRL8Y-4NcJhsEkg6kgFNKLymTGfEPYRv114oTA@mail.gmail.com>
 <d9798141-8957-4152-a1ae-f2abcfcc036c@app.fastmail.com>
 <CAP1Jc11R1P_c2vwRrSZ01X4cUBYvLukj9iYEYyqM+Hemriv3pw@mail.gmail.com>
 <b71effbb-e267-44bc-b11c-e6dbf4784a88@app.fastmail.com>
 <CAP1Jc10EagLDV5s9zPBz=xqK-eGg_byBe2Twne1Jin28S+zb3w@mail.gmail.com>
 <CAP1Jc10nkYctLsDWXUi+S-QdZ-J8zrfnEsw7ZAAjsHx9u5JQuQ@mail.gmail.com>
Date: Tue, 18 Jun 2024 19:44:55 +0000
To: "php internals" <internals@lists.php.net>
Subject: Re: [PHP-DEV] [RFC] Lazy Objects
Content-Type: text/plain
From: larry@garfieldtech.com ("Larry Garfield")

On Tue, Jun 18, 2024, at 5:45 PM, Arnaud Le Blanc wrote:
> Hi Larry,
>
> Following your feedback we propose to amend the API as follows:
>
> ```
> class ReflectionClass
> {
>     public function newLazyProxy(callable $factory, int $options): object {}
>
>     public function newLazyGhost(callable $initializer, int $options): object {}
>
>     public function resetAsLazyProxy(object $object, callable
> $factory, int $options): void {}
>
>     public function resetAsLazyGhost(object $object, callable
> $initializer, int $options): void {}
>
>     public function initialize(object $object): object {}
>
>     public function isInitialized(object $object): bool {}
>
>     // existing methods
> }
>
> class ReflectionProperty
> {
>     public function setRawValueWithoutInitialization(object $object,
> mixed $value): void {}
>
>     public function skipInitialization(object $object): void {}
>
>     // existing methods
> }
> ```
>
> Comments / rationale:
> - Adding methods on ReflectionClass instead of ReflectionObject is
> better from a performance point of view, as mentioned earlier
> - Keeping the word "Lazy" in method names is clearer, especially for
> "newLazyProxy" as a the "Proxy" pattern has many uses-cases that are
> not related to laziness. However we removed the word "Instance" to
> make the names shorter.
> - We have renamed "make" methods to "reset", following your feedback
> about the word "make". It should better convey the behavior of these
> methods, and clarify that it's modifying the object in-place as well
> as resetting its state
> - setRawValueWithoutInitialization() has the same behavior as
> setRawValue() (from the hooks RFC), except it doesn't trigger
> initialization
> - Renamed $initializer to $factory for proxy methods
>
> WDYT?
>
> Best Regards,
> Arnaud

Oh, that looks *so* much more self-explanatory and readable.  I love it.  Thanks!  (Looks like the RFC text hasn't been updated yet.)

--Larry Garfield