Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:63369
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.215.42 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <a3a0505dc42b4ae4a2b54192266f983a@mohiva.com>
References: <a3a0505dc42b4ae4a2b54192266f983a@mohiva.com>
Date: Fri, 12 Oct 2012 05:18:42 -0400
Message-ID: <CAAyV7nGVKujbOhEdYW3FUmLx6sg5pkKKAOpYjPfak3jtrtFCKg@mail.gmail.com>
To: Christian Kaps <christian.kaps@mohiva.com>
Cc: internals@lists.php.net
Content-Type: multipart/alternative; boundary=f46d0401f77f8b50eb04cbd92c68
Subject: Re: [PHP-DEV] Closures and type hinting
From: ircmaxell@gmail.com (Anthony Ferrara)

--f46d0401f77f8b50eb04cbd92c68
Content-Type: text/plain; charset=ISO-8859-1

On Oct 12, 2012 10:59 AM, "Christian Kaps" <christian.kaps@mohiva.com>
wrote:
>
> At the moment it isn't possible to restrict/define the arguments for a
closure which is defined as parameter for a method. Please look at this
small example.
>
> interface Broker {
>
>     public function scan(Request $request, Closure $onFound);
> }
>
> An implementation of the interface could be as follows:
>
> class BrokerImpl implements Broker {
>
>     /**
>      * @var Service
>      */
>     private $service = null;
>
>     public function scan(Request $request, Closure $onFound) {
>
>         if ($request->contains('some value')) {
>             $onFound($this->service);
>         }
>     }
> }
>
> The problem is that I can pass every closure to the scan method.
>
> $broker = new BrokerImpl();
> $broker->scan($request, function() { /* do something */ });
>
> Sometimes I would like to restrict the closure passed to this method. So
that only closures of a certain type which has an argument "Service
$service" could be passed to this method.
>
> $broker = new BrokerImpl();
> $broker->scan($request, function(Service $service) { /* do something */
});
>
> Would it not be possible to extend the Closure class and then define an
abstract method signature for the __invoke method.
>
> class OnFoundClosure extends Closure {
>
>     public abstract function __invoke(Service $service);
> }
>
> And then you can define the interface as follows:
> interface Broker {
>
>     public function scan(Request $request, OnFoundClosure $onFound);
> }
>
> Now if you pass a closure to the scan method which doesn't follow the
signature of the __invoke method, the engine should throw an error.
>
> What do you think?
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>

Honestly, I would suggest using a class directly if you want that kind of
type control. Closures are meant to be a conscience, not the end all be all
implementation.

I understand what you are looking for. But to me, unless we implemented
function pointers (typeable callbacks), I can't see the generalized use
case. At least that isn't already solvable using classes, interfaces and
polymorphism...

Anthony

--f46d0401f77f8b50eb04cbd92c68--