Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:119513
MIME-Version: 1.0
References: <PH8PR10MB63372F72B718DF413DBA13C0BFD89@PH8PR10MB6337.namprd10.prod.outlook.com>
 <CALKiJKpTYt5-BjEnv0W=JXVzR+PTvv-AKhE6Qx=6Gq-L8dsJyQ@mail.gmail.com> <PH8PR10MB633777DE6218F94F3B08BC6ABFD99@PH8PR10MB6337.namprd10.prod.outlook.com>
In-Reply-To: <PH8PR10MB633777DE6218F94F3B08BC6ABFD99@PH8PR10MB6337.namprd10.prod.outlook.com>
Date: Thu, 9 Feb 2023 16:50:23 +0000
Message-ID: <CALKiJKrQSe+s1Hi1CbsvYCZ_i9q5_ezxOdU3B60tZo66NYDOTA@mail.gmail.com>
To: "internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="00000000000001cb0f05f4472e75"
Subject: Re: [PHP-DEV] RFC proposal: values getter in BackedEnum
From: rowan.collins@gmail.com (Rowan Tommins)

--00000000000001cb0f05f4472e75
Content-Type: text/plain; charset="UTF-8"

On Thu, 9 Feb 2023 at 16:18, Sergii Shymko <sergey@shymko.net> wrote:

>
> IMO, other variations of array_column() returning a map don't seem to be
> useful in practice.
> Particularly, what is knowing the case name good for? Do you propose to
> resolve it to the case instance like so?
> $case = constant(ExampleEnum::class . '::' . $caseName);
> My understanding is that backed enums serve the purpose of doing the
> mapping.
>


The same argument can be made for your validation examples: the enum is
"doing the mapping" when you use tryFrom.

My opinion on this probably comes from my general opinion of backed enums -
to me, the opaque case object is primary, and the "value" is just an
arbitrary piece of data attached. I would have preferred to be able to
attach any number of such properties, e.g.
Options::verbose->longArgumentName, Options::verbose->shortArgumentName.

Regards,
-- 
Rowan Tommins
[IMSoP]

--00000000000001cb0f05f4472e75--