Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:96665
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain ohgaki.net designates 180.42.98.130 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <35e0c893-435e-d5da-58be-4f63796775b4@fleshgrinder.com>
References: <44244cfd-c8ad-e0a9-4e70-380a8a8c940c@syberisle.net>
 <CAFMT4Nr+d1k20aY0P9o1G9KU_C9wz2cg2UOPrbE3qbXDBhqe3w@mail.gmail.com>
 <CAF+90c_7g_YdxTfUX8t_OifVJTL=OcvjsC_smWc95M-CcRYaNw@mail.gmail.com>
 <CADqTB_jN6RQxEP5tAJY4T+kuv_rkTzRNvL9T1j-iVX5LG5H3Gg@mail.gmail.com>
 <CAF+90c_rE7DiJ8KEYzFO9GHqx56rin+CiByPe7hRF0fUm-zKBQ@mail.gmail.com>
 <01f2f3da-592b-fd23-709d-ad620422f56d@fleshgrinder.com> <CADqTB_jReEH9pGpfKaYnuYiucPcJ7Y+74VL-mNuJJ+THkUFRVA@mail.gmail.com>
 <a6c950d9-9484-a970-3fe4-e03762d108ff@fleshgrinder.com> <CADqTB_hd+vL1woB7r2OiNNpCT+2MRthiNw2LzLgjsX1RbbgiHw@mail.gmail.com>
 <402447f2-6007-95c4-6b1d-2bbb30dfdd1d@fleshgrinder.com> <CADqTB_hH_kxZNyE1K3ZuKPsy9UkqpfuHCEsDDkOXo5iK9889FQ@mail.gmail.com>
 <35e0c893-435e-d5da-58be-4f63796775b4@fleshgrinder.com>
Date: Mon, 31 Oct 2016 07:59:12 +0900
Message-ID: <CAGa2bXasv2kx1Zs3=sVDPUyipz-wq8_=tHaG5DmBgb=p=VPzaA@mail.gmail.com>
To: "internals@lists.php.net" <internals@lists.php.net>
Cc: Rasmus Schultz <rasmus@mindplay.dk>, Levi Morrison <levim@php.net>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [PHP-DEV] Allow Iterator to be used with current, next, reset,
 key functions
From: yohgaki@ohgaki.net (Yasuo Ohgaki)

Hi all,

On Mon, Oct 31, 2016 at 4:08 AM, Fleshgrinder <php@fleshgrinder.com> wrote:
> On 10/30/2016 4:08 PM, Rasmus Schultz wrote:
>> But either way, I think I'm changing my mind about this one.
>>
>> And if the these functions support only arrays, then also the names
>> should probably be array_first() and array_last() for consistency with
>> other array-only functions.
>>
>> Perhaps better to make people be explicit, rather than trying to hide
>> side-effects to achieve shorter syntax...
>>
>
> +1

+1

There should be delete option. It's more useful if there is this option.

mixed array_first(array $arr [,bool $get_key = FALSE [, bool
$remove_entry=FALSE]]);
mixed array_last(array $arr [,bool $get_key = FALSE [, bool
$remove_entry=FALSE]]);

Problem with this is error condition. When array is empty, what should
be returned from these with default option?  Returning NULL for empty
array would be enough. IMO.

On Mon, Oct 31, 2016 at 1:52 AM, Levi Morrison <levim@php.net> wrote:
> To clarify, do you use the return value of these functions or rely on
> the IAP moving to also get the key?

Depends on code, but I usually use them just to get value of head/tail
element. It could be array_slice/splice, so I don't have much problem
with removing internal array position pointer. However, I still need
current() or each() to get value from returned array. This applies to
array_first/last($arr, TRUE);

Regards,

--
Yasuo Ohgaki
yohgaki@ohgaki.net