Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:122442
Precedence: bulk
MIME-Version: 1.0
References: <CALfTnfWTTQih-uyc4uF-p2V8_GOkaRq127ts6pTFikgFd=3T+g@mail.gmail.com>
In-Reply-To: <CALfTnfWTTQih-uyc4uF-p2V8_GOkaRq127ts6pTFikgFd=3T+g@mail.gmail.com>
Date: Tue, 20 Feb 2024 14:21:57 +0100
Message-ID: <CAPzBOBP4XyC6Yg8aH5TLKpbcq9U-CH7CfGVEQ68jx0f+KbYnyQ@mail.gmail.com>
Subject: Re: [PHP-DEV] ArrayAccess and array_key_exists function
To: =?UTF-8?Q?Ond=C5=99ej_Mirtes?= <ondrej@mirtes.cz>
Cc: PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
From: landers.robert@gmail.com (Robert Landers)

On Mon, Feb 19, 2024 at 1:35=E2=80=AFPM Ond=C5=99ej Mirtes <ondrej@mirtes.c=
z> wrote:
>
> Hello everyone,
> I was recently surprised that ArrayAccess does not work with array_key_ex=
ists() function.
>
> I am aware that ArrayAccess objects cannot be made to work with many arra=
y functions because it does not provide a way to iterate over its keys, you=
 can only ask for and set specific previously known keys.
>
> But it could be made to work with array_key_exists(), because it already =
works with isset() and there=E2=80=99s a direct method that could be used f=
or this - offsetExists().
>
> The use case I have in mind is that sometimes libraries that originally u=
se =E2=80=9Carray=E2=80=9D in return types want to switch to objects and th=
ey make these objects to implement ArrayAccess, for maximising the backward=
 compatibility with the old version. This mostly works, except for array_ke=
y_exists().
>
> Do you think we could add support for ArrayAccess objects in the 2nd para=
meter of array_key_exists() function?
>
> Thank you.
>
> Ond=C5=99ej Mirtes

I'd love to see all array_* functions accept iterators/objects instead
of only just arrays. Yes, I know the name says "array" but often we
just mean "do this operation on this thing that looks enough like an
array for now."

Robert Landers
Software Engineer
Utrecht NL