Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:127052
Precedence: bulk
MIME-Version: 1.0
References: <CAEH1o2MoazX2RLOOg7+CK4BmvBbewPrCiQ+zPCDs_L8yq4F=aA@mail.gmail.com>
 <CAGBsUrd-egZrrJB3syD2-Z7TFoLtg09bcjHT5FVu-M4udPPe=A@mail.gmail.com>
 <68e5b45e-1ae1-452b-9b4d-788e11c8cfe1@scriptfusion.com> <f327f299-c826-4269-94f6-6cf28fcbc582@app.fastmail.com>
In-Reply-To: <f327f299-c826-4269-94f6-6cf28fcbc582@app.fastmail.com>
Date: Sat, 5 Apr 2025 22:02:20 +0300
Message-ID: <CAGwLL8V2WoGhuG-u1CgNaOAR6Bn+1ogVf4YvNFT6hWHGbo=nqg@mail.gmail.com>
Subject: Re: [PHP-DEV] RFC: blank() Function as a Complement to empty()
To: Rob Landers <rob@bottled.codes>
Cc: internals@lists.php.net
Content-Type: multipart/alternative; boundary="000000000000192a1406320ca509"
From: i.miroslavov@gmail.com (Iliya Miroslavov Iliev)

--000000000000192a1406320ca509
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

> echo blank(" "); // true

This case is not blank `empty` checks if a variable is set and contains
something so basically what your blank function does. In this example it
contains character 32. If you want to remove the "\0" (null terminator)
from the language, that's a different story.

On Sat, Apr 5, 2025 at 9:45=E2=80=AFPM Rob Landers <rob@bottled.codes> wrot=
e:

> On Sat, Apr 5, 2025, at 20:25, Bilge wrote:
>
> On 05/04/2025 15:32, Kamil Tekiela wrote:
> >
> > While it has its uses empty() should be avoided whenever possible.
> >
> Agree. A better RFC would be to just deprecate `empty()`.
>
> Cheers,
> Bilge
>
>
> empty() has very many uses. Once you understand what it is shorthand for,
> it makes a lot of sense to use it how it was meant to be used. For exampl=
e:
>
> empty($var) ?: foo($var);
>
> which is just shorter than:
>
> if (isset($var) && $varl !=3D false) {
>   foo($bool);
> }
>
> Generally, you don't use empty() on strings though, just arrays, in my
> style guides anyway. For strings, you use `$string =3D=3D ""` or to be mo=
re
> proper, maybe
>
> trim($value ?? '') =3D=3D ''
>
> ... but these days, trim doesn't accept `null`, so that makes it a bit
> more wordy than it really should be. However, it is just a deprecation
> notice, so it is easy to ignore. For now.
>
> =E2=80=94 Rob
>


--=20
Iliya Miroslavov Iliev
i.miroslavov@gmail.com

--000000000000192a1406320ca509
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr">&gt;=C2=A0echo blank(&quot; &quot;); // true<br><br>This c=
ase is not blank `empty` checks if a variable is set and contains something=
 so basically what your blank function does. In this example it contains ch=
aracter 32. If you want to remove the &quot;\0&quot; (null terminator) from=
 the language, that&#39;s a different story.</div><br><div class=3D"gmail_q=
uote gmail_quote_container"><div dir=3D"ltr" class=3D"gmail_attr">On Sat, A=
pr 5, 2025 at 9:45=E2=80=AFPM Rob Landers &lt;rob@bottled.codes&gt; wrote:<=
br></div><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8e=
x;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class=3D"ms=
g6041684219887197842"><u></u><div><div>On Sat, Apr 5, 2025, at 20:25, Bilge=
 wrote:<br></div><blockquote type=3D"cite" id=3D"m_6041684219887197842qt"><=
div>On 05/04/2025 15:32, Kamil Tekiela wrote:<br></div><div>&gt;<br></div><=
div>&gt; While it has its uses empty() should be avoided whenever possible.=
<br></div><div>&gt;<br></div><div>Agree. A better RFC would be to just depr=
ecate `empty()`.<br></div><div><br></div><div>Cheers,<br></div><div>Bilge<b=
r></div><div><br></div></blockquote><div><br></div><div>empty() has very ma=
ny uses. Once you understand what it is shorthand for, it makes a lot of se=
nse to use it how it was meant to be used. For example:<br></div><div><br><=
/div><div>empty($var) ?: foo($var);<br></div><div><br></div><div>which is j=
ust shorter than:<br></div><div><br></div><div>if (isset($var) &amp;&amp; $=
varl !=3D false) {<br></div><div>=C2=A0 foo($bool);<br></div><div>}<br></di=
v><div><br></div><div>Generally, you don&#39;t use empty() on strings thoug=
h, just arrays, in my style guides anyway. For strings, you use `$string =
=3D=3D &quot;&quot;` or to be more proper, maybe <br></div><div><br></div><=
div>trim($value ?? &#39;&#39;) =3D=3D &#39;&#39;<br></div><div><br></div><d=
iv> ... but these days, trim doesn&#39;t accept `null`, so that makes it a =
bit more wordy than it really should be. However, it is just a deprecation =
notice, so it is easy to ignore. For now.<br></div><div><br></div><div id=
=3D"m_6041684219887197842sig121229152">=E2=80=94 Rob<br></div></div></div><=
/blockquote></div><div><br clear=3D"all"></div><div><br></div><span class=
=3D"gmail_signature_prefix">-- </span><br><div dir=3D"ltr" class=3D"gmail_s=
ignature"><div dir=3D"ltr">Iliya Miroslavov Iliev<div><a href=3D"mailto:i.m=
iroslavov@gmail.com" target=3D"_blank"><font color=3D"#6aa84f">i.miroslavov=
@gmail.com</font></a><br></div></div></div>

--000000000000192a1406320ca509--