Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:46763
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain l-i-e.com designates 67.139.134.202 as permitted sender)
Message-ID: <1527.98.193.1263709030.squirrel@www.l-i-e.com>
In-Reply-To: <4B4DCC64.3030703@easyflirt.com>
References: <4B4DCC64.3030703@easyflirt.com>
Date: Sun, 17 Jan 2010 00:17:10 -0600 (CST)
To: "mathieu.suen" <mathieu.suen@easyflirt.com>
Cc: internals@lists.php.net
User-Agent: Hostbaby Webmail
MIME-Version: 1.0
Content-Type: text/plain;charset=iso-8859-1
Content-Transfer-Encoding: 8bit
Importance: Normal
Subject: Re: [PHP-DEV] Inconsistency redesign
From: ceo@l-i-e.com ("Richard Lynch")

Never use empty()

I've been burned by this too many times.

It's behaviour around '0' and friends changed over the years.

Some colleagues at a former job looked at me funny when they first
heard my rant about this.  Then they tracked down 3 separate bugs and
fixed them, all deriving from the use of empty().

ymmv

On Wed, January 13, 2010 7:36 am, mathieu.suen wrote:
> Hi,
>
> I came across this:
>
> echo sizeof(array());
> echo sizeof("");
> $a = "";
> var_dump( empty($a));
> $a = array();
> var_dump(empty($a));
>
> So funny! How something can have a size greater than 0 but still be
> empty?
> I think PHP is reinventing the inconsistency word.
>
> But then let assume that empty is just making a cast in array.
> $a = "";
> empty($a) //true
> empty((array)$a) //false
>
> Ok so empty is big ugly switch case on type.
>
> empty(0); //true
> empty(45); //false
>
> Wooow reinventing emptiness on number....
>
> -- Mathieu Suen
>
>
>
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>


-- 
Some people ask for gifts here.
I just want you to buy an Indie CD for yourself:
http://cdbaby.com/search/from/lynch