Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:99839
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain hoa-project.net from 217.70.183.198 cause and error)
To: internals@lists.php.net
References: <CAOe2Xe4j9dOa63wkG2gJxosH1fymWDB9C-zcXwDWGcB=Zw9w9g@mail.gmail.com>
Message-ID: <3e8e0194-c594-2c3f-4c55-9b21ddfee768@hoa-project.net>
Date: Tue, 11 Jul 2017 19:35:35 +0200
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:52.0)
 Gecko/20100101 Thunderbird/52.2.1
MIME-Version: 1.0
In-Reply-To: <CAOe2Xe4j9dOa63wkG2gJxosH1fymWDB9C-zcXwDWGcB=Zw9w9g@mail.gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Content-Language: en-US
Subject: Re: [PHP-DEV] array coalesce operator concept
From: ivan.enderlin@hoa-project.net (Ivan Enderlin)

Hello :-),

Thank you for the proposal. I have a question though: How is it 
different from:

* `foreach ($foo ?: [] as $bar) { … }` if `$foo` does not exist,

* `foreach ((array) $foo as $bar) { … }` if `$foo` is not an array.

I understand your issue, but you can probably type your data with 
`iterable`, `Generator`, `array`, `Iterator`…

Also, I am not sure that `??$x` really means `if (is_array($x)) { … }`.


Regards.


On 11.07.17 17:02, Mark Shust wrote:
> Hello,
>
> I wanted to garnish feedback on a RFC proposal. This is just a concept at
> this point, and is inspired by the null coalesce operator.
>
> Code will error if a non-array value is passed through a looping feature.
> For example, this code:
>
> <?php
>
> $foo = "abc";
>
> foreach ($foo as $bar) {
>
>    echo $bar;
>
> }
>
>
> will result in the error:
>
> PHP Warning:  Invalid argument supplied for foreach() in test.php on line 3
>
>
> To prevent this, the logical solution is to wrap this in an is_array check:
>
> <?php
>
> $foo = "abc";
>
> if (is_array($foo)) {
>
>    foreach ($foo as $bar) {
>
>      echo $bar;
>
>    }
>
> }
>
>
> This code runs successfully and does not error out. For a syntactic
> sugar/improvement, this can be shorthand for executing the loop instead of
> wrapping the block within an is_array check:
>
>
> <?php
>
> $foo = "abc";
>
> foreach (??$foo as $bar) {
>
>    echo $bar;
>
> }
>
>
> Let me know your thoughts.
>
>
> Cheers,
> Mark
>