Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:87297
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.220.46 as permitted sender)
References: <CAOsHV+uApxF=pQngonMppam2AEnW0Sm1ESAaB9exLH0K0hMBWA@mail.gmail.com>
 <CAGa2bXYx-MGjhHVkDRYsHSRR4b+x4ba5PVxdhw9W2B1z0fWCEQ@mail.gmail.com>
 <CAOsHV+td_i5Z3JPmPF5WT8gAvhPFReO28L3TGPaYWgybSVnbSA@mail.gmail.com>
 <CAGa2bXbgDcjXWgoDw_X61KDbeiV_tcHpGk5RFSFsnQ1-LSNVPw@mail.gmail.com>
 <CAOsHV+tY4CmLD6tYt7R9eiNkBb9ibh4sjAVD0oysJTCuTY25Tw@mail.gmail.com>
 <CAGa2bXYtbhOg4XqniTtZj-rz-E7cc=C1cmqGSmphLmbN+cqASA@mail.gmail.com>
 <CAOsHV+vhvYD5JoyuKQOOmDzbN0rurGX58eMuRjn4cmH3H9gvsw@mail.gmail.com>
 <CAGa2bXa0=gZ=+5oSWp-kRBpWXoozHEdJb1-nuKN9r6RmGZj8Sw@mail.gmail.com>
 <CAOsHV+vjPxFZhN-OH8+5S57tK_P_G80B+HujXy07zeExOEMNVg@mail.gmail.com>
 <CACwt+JdxCF=F-PzPtGsZJb0WapEzZk--Qb2CvMaowxr4m8-htw@mail.gmail.com>
 <CAOsHV+vbMteQhRHw7RKkp27BXEBaR+JzEns2CdngLwpEzMmpKA@mail.gmail.com>
 <CACwt+JfUVYaFrvsezFpU2=5tNBehLq9bGFQkSsoBgX5Vno663w@mail.gmail.com>
 <CALW=WEDkJ0eUk3viUtgk-RG4D8w5THtcWa8DTgowT+WEmmPxKA@mail.gmail.com>
 <CAOsHV+sCicPRob3jpJztg46Q9M5fAZTRmsbp0fHq-t7PSAmdVQ@mail.gmail.com> <CALW=WEChn9gzykdMfbF60BYRcud96t=GnwmDBM2dbczG6k33gQ@mail.gmail.com>
Mime-Version: 1.0 (1.0)
In-Reply-To: <CALW=WEChn9gzykdMfbF60BYRcud96t=GnwmDBM2dbczG6k33gQ@mail.gmail.com>
Content-Type: text/plain;
	charset=us-ascii
Content-Transfer-Encoding: quoted-printable
Message-ID: <1AE1CE66-869C-44E8-A961-86DE8D05909C@gmail.com>
Cc: Marcio Almada <marcio.web2@gmail.com>,
 Patrick ALLAERT <patrickallaert@php.net>,
 Marcio Almada <marcio3w@gmail.com>, Yasuo Ohgaki <yohgaki@ohgaki.net>,
 PHP internals <internals@lists.php.net>
Date: Sun, 26 Jul 2015 15:57:56 +1000
To: "S.A.N" <ua.san.alex@gmail.com>
Subject: Re: [PHP-DEV] Re: Voting choice for language changes (Was: "Re: [PHP-DEV][RFC][DISCUSSION] Strict Argument Count")
From: davidkmuir@gmail.com (David Muir)



> On 24 Jul 2015, at 2:33 am, "S.A.N" <ua.san.alex@gmail.com> wrote:
>=20
> 2015-07-23 18:10 GMT+03:00 Marcio Almada <marcio.web2@gmail.com>:
>> Hi, you replied to the wrong thread ;)
>>=20
>> 2015-07-22 19:38 GMT-03:00 S.A.N <ua.san.alex@gmail.com>:
>>> I am satisfied, the possibility of group declarations, but the that lack=
:
>>>=20
>>> <?php
>>>=20
>>> use App\RestException\   // name "RestException", not imported to
>>> current namespace :(
>>> {
>>>    Gone,
>>>    NotFound,
>>>    BadRequest
>>> };
>>>=20
>>> ?>
>>>=20
>>> Unfortunately have to write so:
>>>=20
>>> <?php
>>>=20
>>> use App\RestException;
>>> use App\RestException\{Gone, NotFound, BadRequest};
>>>=20
>>> ?>
>>>=20
>>> It looks ugly and very strange.
>>=20
>> There is nothing strange on it (except, possibly, the trailing `\`
>> which was discussed to death and voted).
>>=20
>> Even if we had no trailing '\' it wouldn't make any sense to import
>> "App\RestException" when "use App\RestException{Gone, NotFound,
>> BadRequest};" is used.
>>=20
>>> My proposition, the imported end name, if end of without slash.
>>>=20
>>> Like this:
>>>=20
>>> <?php
>>>=20
>>> use App\RestException
>>> {
>>>    Gone,
>>>    NotFound,
>>>    BadRequest
>>> };
>>>=20
>>> echo RestException::class; // App\RestException
>>>=20
>>> ?>
>>=20
>> Importing from within a namespace is not the same thing as importing a
>> class. I'd be against it.
>>=20
>> My 2 cents: why do you have "RestException" with "Exception" suffix
>> and then don't have the same suffix on the other exception names? This
>> unpredictable exception hierarchy is the "ugly" part and subtle
>> alternative syntax won't make it better, IMMO.
>>=20
>> Marcio
>=20
> RestException is the base class for classes Gone, NotFound, BadRequest...
> He needed to catch all RestExceptions class.
>=20
> We do not use suffixes for names of exception classes, simple clear
> named classes of exceptions, this is our agreement.
>=20
> --=20
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>=20

Then why not put the base class in the namespace of which it is a base class=
 for?

Cheers,
David