Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:61395
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.216.42 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAOhuFvYD8FQw=xbK3hFqi4GYkOvCNzWjuhk2fqwBjKG9n2kuuQ@mail.gmail.com>
References: <CAOhuFvYD8FQw=xbK3hFqi4GYkOvCNzWjuhk2fqwBjKG9n2kuuQ@mail.gmail.com>
Date: Wed, 18 Jul 2012 10:20:43 -0400
Message-ID: <CAAyV7nEpp+Srxemgi913K2nZ7rztz6RV-9bNdXdPJx49q-AenQ@mail.gmail.com>
To: Rafael Dohms <listas@rafaeldohms.com.br>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=00248c768f9e4c4f9304c51b5e76
Subject: Re: [PHP-DEV] Implicit isset in ternary operator
From: ircmaxell@gmail.com (Anthony Ferrara)

--00248c768f9e4c4f9304c51b5e76
Content-Type: text/plain; charset=ISO-8859-1

Rafael,

One point of clarity:

On Wed, Jul 18, 2012 at 10:15 AM, Rafael Dohms <listas@rafaeldohms.com.br>wrote:

> Hey Everyone,
>
> With the syntax improvements for 5.4 and some of the de-referencing work,
> code looks so much sleeker then before, but there is still one use case
> that really bugs me.
>
> Given this code:
>
> $width = (isset($config['width']))? $config['width'] : 300;
>
> The code is pretty straight forward, if width is set in the array, use it
> otherwise use 300.
>
> This code would look much better if it could make use of the ternary
> operator
>
> $width = $config['width'] ?: 300;
>
> The only reason for this to not work is: it throws a notice if the array
> key is not there (which is the case we are covering anyway)
>
> This is basically because the ternary operator does not do a internal
> implicit isset, only an empty.
>

It does not do an empty. Empty would avoid notices as well. All it does is
an implicit bool cast...


> Does this seem like a possible improvement we can work on? Anyone
> interested in championing the change?
>

I like it in principle. My only concern is *why* wasn't it done this way in
the first place... Is there a reason?

Anthony

--00248c768f9e4c4f9304c51b5e76--