Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:110906
MIME-Version: 1.0
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>
In-Reply-To: <CADyq6sJoaBznTWy=1YrzS=_+Rfo+0qdoRZuEBWzzb4SDwkng8g@mail.gmail.com>
Date: Thu, 9 Jul 2020 15:33:55 -0300
Message-ID: <CAOsHV+syvRLCz9L=hMOfBF2xbOjt0p+6eBfpe8Z_MhTN6kTZ-g@mail.gmail.com>
To: Marco Pivetta <ocramius@gmail.com>
Cc: Josh Bruce <josh@joshbruce.dev>, PHP Internals List <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] [CONCEPT][DISCUSSION] Instance as boolean
From: marcio.web2@gmail.com (Marcio Almada)

Hi,

>
> Hey Marcio,
>
>>
>> Hi
>>
>> >
>> > Re casting - In the previous thread the following concern was presente=
d and I don=E2=80=99t know enough about that area to respond effectively: "=
I'd endorse avoiding object-to-<scalar> casts via cast operations: they are=
 a good source of bugs. My rationale for the discouragement of magic cast m=
ethods is explained with some code examples at https://github.com/ShittySof=
t/symfony-live-berlin-2018-doctrine-tutorial/pull/3#issuecomment-460441229=
=E2=80=9D
>> >
>>
>> Frankly this is something subject to personal opinions and will change
>> over time according to community shifts.
>>
>
> Not sure what's "subject to personal opinions" in the objective bug expos=
ed in the snippet above?
>

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` ;)

> Greets,
>
> Marco Pivetta
>
> http://twitter.com/Ocramius
>
> http://ocramius.github.com/