Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:69185
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.82.182 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <004CB360-7D96-411D-B8A2-C0E963EDCF7A@wiedler.ch>
References: <CAPwsocGni4kjJq9c5wr77ohPU+YX2=0O7EgahdH=p0=hH1H_fA@mail.gmail.com>
 <004CB360-7D96-411D-B8A2-C0E963EDCF7A@wiedler.ch>
Date: Wed, 18 Sep 2013 14:14:50 +0200
Message-ID: <CALtuQzBxU6_haU1UGMWp4+GLnv421n-A2hcCPPp1AoU4J5_jwQ@mail.gmail.com>
To: Igor Wiedler <igor@wiedler.ch>
Cc: Leigh <leight@gmail.com>, PHP internals list <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=f46d043749f9b6c58e04e6a7642c
Subject: Re: [PHP-DEV] Allowing is_* functions to accept multiple parameters
From: krebs.seb@gmail.com (Sebastian Krebs)

--f46d043749f9b6c58e04e6a7642c
Content-Type: text/plain; charset=ISO-8859-1

2013/9/18 Igor Wiedler <igor@wiedler.ch>

> On Sep 18, 2013, at 11:53 AM, Leigh <leight@gmail.com> wrote:
>
> > Hi Internals.
> >
> > How do you feel about expanding the is_* functions to accept multiple
> > parameters similar to the way isset() already does?
> >
> > ...
> >
> > Thoughts?
>
> For isset() there is a good reason to do this, because the var might not
> exist at all, and as such you cannot use a function, as it would produce
> warnings.
>
> However, for is_* functions, this could easily be done with a higher-order
> "every" function. You pass a predicate and an array of values. It returns a
> boolean.
>
> Example:
>
>         if (!every('is_int', $numbers)) {
>                 throw new \InvalidArgumentException(...);
>         }
>

Actually it was about multiple parameters

    if (!every('is_int', $a, $b, $c)) { /* .. */ }

To me using `is_int()` directly feels more intuitive too.

    if (!is_int($a, $b, $c)) { /* .. */ }



>
> Not only is that much cleaner in my opinion, it also is composable without
> having to change *all* of the is_* predicates.
>
> As such, I don't see much value in changing this.
>

Well, asking me that sounds like the same value like it is for
array_column(). It is a minor improvement, but it is an useful one (imo).

Just my 2 cent


Regards,
Sebastian


> Igor
>
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>


-- 
github.com/KingCrunch

--f46d043749f9b6c58e04e6a7642c--