Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:120855
MIME-Version: 1.0
References: <CAOfzkkxrQOje274RMUW8kv13vBmtNSNd_D9Bq_SJeDEBVA8i7Q@mail.gmail.com>
 <CAJpLVh1hhd=zxSz+T4eZdd7eBD-40muU81eFwhmh3e0n7VGwyQ@mail.gmail.com>
 <8055b366-32f1-4d2c-85d0-0cf665e6c8d3@app.fastmail.com> <CABAhh3FjjwyZjrpzrES6E1kbRz_czg64-CRxtKw-jkY5JzGYeA@mail.gmail.com>
 <CAJpLVh0sCG-cZ3qubNnHVTCQc9T8MdKHDdZLo5F=P0kcOAiGRQ@mail.gmail.com> <CAPzBOBMsrO8i=C1VumGVxU8zfGdp+m6REaq6XWzYEvuMYLXwyw@mail.gmail.com>
In-Reply-To: <CAPzBOBMsrO8i=C1VumGVxU8zfGdp+m6REaq6XWzYEvuMYLXwyw@mail.gmail.com>
Date: Thu, 20 Jul 2023 13:01:43 -0700
Message-ID: <CAMrTa2H2d48DPRv3rsNVYKoc5Av_RA2X2ZO1CBW3+2xQO3eDvQ@mail.gmail.com>
To: Robert Landers <landers.robert@gmail.com>
Cc: =?UTF-8?Q?Olle_H=C3=A4rstedt?= <olleharstedt@gmail.com>, 
	someniatko <someniatko@gmail.com>, php internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000bcd0ba0600f09e26"
Subject: Re: [PHP-DEV] pipes, scalar objects and on?
From: jordan.ledoux@gmail.com (Jordan LeDoux)

--000000000000bcd0ba0600f09e26
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Tue, Jul 18, 2023 at 8:05=E2=80=AFAM Robert Landers <landers.robert@gmai=
l.com>
wrote:

> On Tue, Jul 18, 2023 at 3:18=E2=80=AFPM Olle H=C3=A4rstedt <olleharstedt@=
gmail.com>
> wrote:
> >
> > 2023-07-18 14:48 GMT+02:00, someniatko <someniatko@gmail.com>:
> > > I am glad this topic arose! I was also planning to write on this topi=
c
> to
> > > the internals mailing list, but have abandoned this idea, because I
> feel it
> > > might be inconvenient for the real active PHP developers on the list =
to
> > > receive too many emails from the people which don't actively
> participate in
> > > the development itself.
> > >
> > > My interest in the pipe operator might seem a little non-standard -
> > > basically what I'd really want to see is a **nullable** pipe operator=
!
> > >
> > > There is a popular library github.com/schmittjoh/php-option, which
> has 250
> > > MILLION installations. Basically what it provides is a class-wrapper
> of a
> > > value of lack thereof: it's either `Some<Type>` or `None`. I also
> maintain
> > > a similar library
> https://packagist.org/packages/someniatko/result-type
> > > which fixes some shortcomings of the original one related to the stat=
ic
> > > analysis, but this is another story. Basically what the stats tell us
> is
> > > that such stuff is popular among the PHP community.
> > >
> > > In my eyes, it is actually semantically equivalent to the nullable PH=
P
> > > types: `?Type`. And some operations provided by the lib, are actually
> > > covered by PHP itself, which has multiple null-friendly operators:
> > > `$option->getOrElse($defaultVal)` --> `$nullable ?? $defaultVal`
> > > `$option->isEmpty()` --> `$nullable =3D=3D=3D null`
> > > `$option->getOrThrow(new \Exception('blah'))` --> `$nullable ?? throw
> new
> > > \Exception('blah')`
> > >
> > > I'd like to use the arguably "more idiomatic" native PHP nullables,
> rather
> > > than a foreign-feeling userspace construct, if they were more
> convenient.
> > >
> > > But there is a very important operation, `map()`, which is
> unfortunately
> > > not covered by the native PHP, which is `Option::map()`, and here is =
a
> > > real-world example:
> > > ```
> > > return $repository->getById($idFromHttpRequest)
> > >     ->map($serializer->serializeToJson(...)) // only executes when th=
e
> > > Option is Some, not None
> > >     ->map(fn (string $json) =3D> new Response(status: 200, content:
> $json))
> > >     ->getOrElse(new Response(status: 404));
> > > ```
> >
> > Ehm, wouldn't that be the same as a Pipe class that's configured to
> > stop on null?
> >
> > public function getThing($id) {
> >   return new Pipe(
> >     $this->getData(...),
> >     $this->serializeData(...),
> >     $this->mapToResponse(...)
> >   )
> >   ->stopOnEmpty()
> >   ->from($id)
> >   ->run();
> > }
> >
> > Wait, are you using map() for arrays or not? Looks like not.
> >
> > Olle
> >
> > --
> > PHP Internals - PHP Runtime Development Mailing List
> > To unsubscribe, visit: https://www.php.net/unsub.php
> >
>
> I might be venting, but, I wish the operators RFC had passed...
>

I don't think I will re-propose it with the current set of voters tbh. The
main people who contribute to the Zend folder (the core engine/compiler
parts) were highly skeptical of it, with at least two telling me flat out
that they would vote no even on a hypothetical perfect feature design. Very
few contributors actually touch large parts of that area of the engine, so
most voters tend to take the opinions of those 5-6 voters very seriously on
such proposals. As I don't think it's possible for me to convince those 5-6
people, I don't think it would be a good idea to re-propose, because even
if I did get a 2/3 vote on it, I feel like the people most responsible for
helping me maintain the feature would not be very happy/willing to do so,
and that would probably harm the PHP project in other ways.

Jordan

--000000000000bcd0ba0600f09e26--