Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:110920
Message-ID: <7E7C7E10-FFC1-4332-AEFC-A656E1784219@joshbruce.dev>
Content-Type: multipart/alternative;
	boundary="Apple-Mail=_F2087BAF-8DD6-4318-BDB6-8AD3C00D9C9B"
Mime-Version: 1.0 (Mac OS X Mail 13.4 \(3608.80.23.2.2\))
Date: Fri, 10 Jul 2020 09:21:32 -0500
In-Reply-To: <CAOsHV+syvRLCz9L=hMOfBF2xbOjt0p+6eBfpe8Z_MhTN6kTZ-g@mail.gmail.com>
Cc: Marco Pivetta <ocramius@gmail.com>,
 PHP Internals List <internals@lists.php.net>
To: Marcio Almada <marcio.web2@gmail.com>
References: <65EE4215-3F80-435C-B271-DBE30D26E2CB@joshbruce.dev>
 <CAOsHV+s3=cyv=Q38uAJ35DyP_LUjjmzTNEx3ivH_W0jR_pOzpw@mail.gmail.com>
 <CC1C92D8-B9B3-4E73-B298-C414F66BE438@joshbruce.dev>
 <CAOsHV+tFg10YVYUo6xPehgS59chtKVgPtLd-CVe9C=SC-LNp_Q@mail.gmail.com>
 <CADyq6sJoaBznTWy=1YrzS=_+Rfo+0qdoRZuEBWzzb4SDwkng8g@mail.gmail.com>
 <CAOsHV+syvRLCz9L=hMOfBF2xbOjt0p+6eBfpe8Z_MhTN6kTZ-g@mail.gmail.com>
Subject: Re: [PHP-DEV] [CONCEPT][DISCUSSION] Instance as boolean
From: josh@joshbruce.dev (Josh Bruce)

--Apple-Mail=_F2087BAF-8DD6-4318-BDB6-8AD3C00D9C9B
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain;
	charset=utf-8


>>>=20
>>> Frankly this is something subject to personal opinions and will =
change
>>> over time according to community shifts.
>>>=20
>>=20
>> Not sure what's "subject to personal opinions" in the objective bug =
exposed in the snippet above?
>>=20
>=20
> So there is a bug in a contrived code snippet therefore casting is a
> universal bad practice?
> It's at least a huge stretch. Some might say your bad practice was
> actually using `null` ;)


Good stuff - very helpful (and I=E2=80=99m grateful), thank you both.

This might actually be an interesting (possibly the) use case for what =
we=E2=80=99re talking about.

Updated: https://bit.ly/php-0001 <https://bit.ly/php-0001>

Added pre-roll section on perceived roadblocks as I=E2=80=99m =
experiencing one.

Added =E2=80=9CUsage=E2=80=9D section.

Renamed code sample header to =E2=80=9CRationale"

***

I didn=E2=80=99t have a chance to look at the code linked before and =
tend to agree:

1. Avoid null (I=E2=80=99m a bit extreme with this one)
2. I tend to avoid casting as well and, if the first is followed, a =
casting problem could become moot. PHP doesn=E2=80=99t like to halt - =
most primitives when cast to bool either effectively return the result =
of an empty check or a not-zero check - an object, if instantiated, is =
always true. __toBool would give us a way of saying it=E2=80=99s not =
true (it=E2=80=99s not empty, it's not zero, but it's not okay for some =
reason - maybe we need to do some follow-on configuration method =
calls=E2=80=A6just making stuff up). :)

Cheers,
Josh


--Apple-Mail=_F2087BAF-8DD6-4318-BDB6-8AD3C00D9C9B--