Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:58592
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain googlemail.com designates 209.85.214.170 as permitted sender)
Received-SPF: pass (google.com: domain of simonsimcity@googlemail.com designates 10.60.11.40 as permitted sender) client-ip=10.60.11.40;
MIME-Version: 1.0
In-Reply-To: <CAMyk_Ab8sMFKNA6Tpgo9+aeEzu-tiBtDxUBTQKQ5LAQe7_sYyg@mail.gmail.com>
References: <CAAyV7nF+yKwrUvyJ7mytQpyLr_HG3M3Pan5cSpLmYjMHFe9X3Q@mail.gmail.com>
	<CAKxcST-ByD1M0jwktJyJLNeCPpKEqf0v4CP90b2Um+1+KKypag@mail.gmail.com>
	<CAMyk_Ab8sMFKNA6Tpgo9+aeEzu-tiBtDxUBTQKQ5LAQe7_sYyg@mail.gmail.com>
Date: Mon, 5 Mar 2012 09:15:53 +0100
Message-ID: <CAJWj5+Gt3DMUtxbpVOswgghXWr4PcGC+-Z1MfY+rb3P8YeZgZQ@mail.gmail.com>
To: Lazare Inepologlou <linepogl@gmail.com>
Cc: Paul Dragoonis <dragoonis@gmail.com>, Anthony Ferrara <ircmaxell@gmail.com>, internals@lists.php.net
Content-Type: multipart/alternative; boundary=e89a8fb204d4f5bb4804ba7a8899
Subject: Re: [PHP-DEV] [RFC - Discussion] Parameter Type Casting Hints
From: simonsimcity@googlemail.com (Simon Schick)

--e89a8fb204d4f5bb4804ba7a8899
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Hi, Lazare

This is something not obvious, at least for me, and should be handled in
the function itself.

Here it would be better to update the function *add()* that you could also
pass mixed and it will try to generate a *DateInterval *out of that or
whatever.
But that's another RFC.

Type-juggling applies just to weak-types like string, int, float and
boolean. I don't like to have this for resource, array or class excepted
they are compatible to each other (like array and SplFixedArray,
SplObjectStorage, ArrayIterator, ...)
If you have to do something special to get the expected type (like creating
a new object) type-juggling would not be the good-way to go.

Bye
Simon

2012/3/4 Lazare Inepologlou <linepogl@gmail.com>

> >
> > I wouldn't want people to put class typehints in there such as
> >
> function foo( (SomeClass) $foo)
> >
>
> Why not? I would definitely like something like that, in the future. Here
> is an example:
>
> class DateTime {
>  ...
>  public function add( (DateInterval) $interval ) { ... }
>  ...
> }
> $date =3D new DateTime;
> $date->add( 'P3D' ); // string to DateInterval casting
> $date->add( 3600 );  // int (seconds) to DateInterval casting
>
> Personally, I find this to be much closer to free type-juggling PHP style
> than the current implementation with type hints (
> http://www.php.net/manual/en/datetime.add.php).
>
> [ Actually, I would prefer even type hints to work like this. Check if th=
e
> passed argument if of the correct type and, if not, try to cast it. Failu=
re
> to do so will lead to the E_RECOVERABLE_ERROR as today. ]
>
>
> Lazare INEPOLOGLOU
> Ing=C3=A9nieur Logiciel
>
>
> 2012/3/4 Paul Dragoonis <dragoonis@gmail.com>
>
> > Can you make sure that only scalar or array casts can be done?
> >
> > I wouldn't want people to put class typehints in there such as
> > function foo( (SomeClass) $foo)
> >
> > - Paul.
> >
> > On Sun, Mar 4, 2012 at 1:28 AM, Anthony Ferrara <ircmaxell@gmail.com>
> > wrote:
> > > Hey all,
> > >
> > > I've drafted an RFC for the Parameter type casting hint proposal that
> > > I posted to before.  Attached to the RFC is a patch that's proposed
> > > for inclusion in core for functionality (it doesn't include news
> > > entries, or documentation, or any of the other steps that would be
> > > needed prior to commit).
> > >
> > > https://wiki.php.net/rfc/parameter_type_casting_hints
> > >
> > > Please provide feedback here on the implementation and RFC topics.
> > >
> > > Thanks,
> > >
> > > Anthony
> > >
> > > --
> > > PHP Internals - PHP Runtime Development Mailing List
> > > To unsubscribe, visit: http://www.php.net/unsub.php
> > >
> >
> > --
> > PHP Internals - PHP Runtime Development Mailing List
> > To unsubscribe, visit: http://www.php.net/unsub.php
> >
> >
>

--e89a8fb204d4f5bb4804ba7a8899--