Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:113165
MIME-Version: 1.0
References: <CAEsg9X2Zshqw1bHE7rNey6VotvKgs1i8+zJGSmFp-+JWPnJrww@mail.gmail.com>
 <CADyq6sJArRg1HDhKG0gYPJYZjyHtf+Cjiiv9OA+YhbvZz1dB5Q@mail.gmail.com>
In-Reply-To: <CADyq6sJArRg1HDhKG0gYPJYZjyHtf+Cjiiv9OA+YhbvZz1dB5Q@mail.gmail.com>
Date: Sat, 13 Feb 2021 13:22:02 -0300
Message-ID: <CAEsg9X3h04G-BPX=RsDa6dQZdUHbadsTq2vTP-VYosME+ifP7w@mail.gmail.com>
To: Marco Pivetta <ocramius@gmail.com>
Cc: PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000df111805bb3a2701"
Subject: Re: [PHP-DEV] Inline conditional that returns null if falsy
From: david.proweb@gmail.com (David Rodrigues)

--000000000000df111805bb3a2701
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Em s=C3=A1b., 13 de fev. de 2021 =C3=A0s 00:11, Marco Pivetta <ocramius@gma=
il.com>
escreveu:

> Hey David,
>
> Even upfront, this looks like a quite bad idea: either you have an empty
> collection, or you have a collection with elements in it, but `null` is a
> very different concept that isn't homogeneous with the resulting type.
>
> I've updated multiple projects in the past where `array|null` was used as
> return type for a collection-alike endpoint, removing `null` and replacin=
g
> it with an empty iterable instead.
>

This was just one of the examples I have, but I still think it is valid.
Creating an empty collection costs resources that could be avoided in
situations where null could be used (as is most of my real use cases).

But there are other situations where we can use different situations to use
this type of resource.

One of the examples I have is when I need to include a class in an HTML
element conditionally. There are probably better ways, but this is a quick
one for simple situations:

<div class=3D"user {{ $user =3D> "online" }}">...</div> vs.
<div class=3D"user {{ $user ? "online" : null }}">...</div>

It is much easier to read, and I believe that the cost for language would
be minimal, and perhaps even more optimized than the use of ?:.



Atenciosamente,
David Rodrigues


Em s=C3=A1b., 13 de fev. de 2021 =C3=A0s 00:11, Marco Pivetta <ocramius@gma=
il.com>
escreveu:

> Hey David,
>
> On Fri, Feb 12, 2021, 20:24 David Rodrigues <david.proweb@gmail.com>
> wrote:
>
>> Hello!
>>
>> It is just a suggestion to be discussed.
>>
>> A lot of places on my projects I have codes like:
>>
>> $companies =3D $user->companies->count()
>>     ? new Collection($user->companies)
>>     : null;
>>
>
> Even upfront, this looks like a quite bad idea: either you have an empty
> collection, or you have a collection with elements in it, but `null` is a
> very different concept that isn't homogeneous with the resulting type.
>
> I've updated multiple projects in the past where `array|null` was used as
> return type for a collection-alike endpoint, removing `null` and replacin=
g
> it with an empty iterable instead.
>
>>

--000000000000df111805bb3a2701--