Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:57985
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.212.170 as permitted sender)
Received-SPF: pass (google.com: domain of keisial@gmail.com designates 10.216.138.7 as permitted sender) client-ip=10.216.138.7;
Message-ID: <4F469FB7.7040802@gmail.com>
Date: Thu, 23 Feb 2012 21:21:11 +0100
User-Agent: Thunderbird
MIME-Version: 1.0
To: Sebastian Krebs <krebs.seb@googlemail.com>
CC: PHP internals list <internals@lists.php.net>
References: <CAG=uTXFQUY=EDb=ut97nNSm+m8sMQNRK-9YPcfb2quaoS246-g@mail.gmail.com> <CABBJUpd7XkDjckY-5=D6B=kouUYMrWvzMVnHR_Hch74oGWM-TQ@mail.gmail.com> <CAO8n+WJrqUguS4FSq==z9rF6iLoz3uXqSLBrrkY6yMBY9DrTWA@mail.gmail.com> <CALtuQzD6f_vcZiGi2HWdP8SSqf9744FucUC+4sX7PnGhOnh+rg@mail.gmail.com> <4F455C96.50706@gmail.com> <4F455E91.2060408@googlemail.com> <4F4570D6.8040003@gmail.com> <4F457597.3060405@sugarcrm.com> <4F468D09.7070004@gmail.com> <4F468E1B.5030800@googlemail.com>
In-Reply-To: <4F468E1B.5030800@googlemail.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] [RFC] Enum proposal (yet another)
From: keisial@gmail.com (=?ISO-8859-1?Q?=C1ngel_Gonz=E1lez?=)

I don't see your point, Sebastian.

> And
>
> | $studipNamedVariable = Databases::Mysql;
> | // ... much code
> | database_select($stupidNamedVariable, $sql);
>
> is better? The problem here seems to be more the developer, that
> avoids the use of constants, then less the missing enums. 
You can obviously do weird things, as with in any feature. In this case
you bring up, I see little sense in passing enums in variables, unless
that comes itself from a parameter for that function, or comes from an
external source, like a configuration data.


> If you use a constant like
>
> | Databases::Mysql = 1024;
>
> the call
>
> | database_select(Database::IDontExists, $sql);
>
> will fail too. Other examples, that supports type hints, can be found
> in other mails around here.
Yes. With a parser error as IDontExists doesn't belong to Database enum.
How does it relate with assigning 1024 to Databases::Mysql ?