Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:115284
MIME-Version: 1.0
References: <CAF+90c--_H+=2yqkLKkFyGhKhwVJUJS8iRWw+hbZaSW1_q0xHw@mail.gmail.com>
 <802fdb7d-69e5-edfa-de94-70416f9093d0@ralphschindler.com> <CAAKU0un-SVJVZMJWyJX9DnWVWwTc3A8U5pNYhCge2yT73Og8cw@mail.gmail.com>
 <7f09e13c-7f35-b771-de4c-94c5a35df497@ralphschindler.com>
In-Reply-To: <7f09e13c-7f35-b771-de4c-94c5a35df497@ralphschindler.com>
Date: Sat, 3 Jul 2021 17:07:45 +0200
Message-ID: <CAF+90c_HOv4PKWqjYaCXAvvM8BMhcHDnxRESxvhnGopq6+pkDg@mail.gmail.com>
To: Ralph Schindler <ralph@ralphschindler.com>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="00000000000037309905c6397047"
Subject: Re: [PHP-DEV] [VOTE] First-class callable syntax
From: nikita.ppv@gmail.com (Nikita Popov)

--00000000000037309905c6397047
Content-Type: text/plain; charset="UTF-8"

On Sat, Jul 3, 2021 at 3:51 PM Ralph Schindler <ralph@ralphschindler.com>
wrote:

>
> > It would theoretically be:
> >
> >      $fn = MyController::myAction(...);
> >
> > It currently errors:
> >
> >> Fatal error: Uncaught Error: Non-static method MyController::myAction()
> cannot be called statically
> >
> > I would be okay with allowing this, as long as it's bound before it's
> called.
>
>
> One other question regarding the current patch. I have not seen this
> aspect discussed yet.
>
>  From the list in the first proposal section
> (https://wiki.php.net/rfc/first_class_callable_syntax#proposal), I am
> focusing this question on the few that are by name (strings).. so:
>
>    strlen(...);
>    Foo::method(...);
>    $classStr::$methodStr(...);
>    self::{$complex . $expression}(...);
>    'strlen'(...);
>    [Foo::class, 'method'](...);
>
> Do these have to scan the target function and its signature in order to
> wrap them? In the cases of classes above, this will trigger the
> autoloader.  I wonder if it could create a by-name reference that
> resolves the signature at just in time / at call time in these situations?
>
> In context, suppose I am suggesting that something like:
>
>      $routes = [
>        '/my-action' => MyController::myAction(...),
>        '/other-action => [OtherController::class, 'otherAction'](...),
>        ...
>      ];
>
> would not invoke the autoloader until one of them may be actually
> used/called.
>
> This would be similar in philosophy as to how Foo::class creates a fully
> qualified string without confirming a class exists at that name when the
> scanner and executor sees ::class.
>

No, something like this will definitely not be supported as part of this
syntax. It is fundamentally at odds with this proposal, which requires that
callability is validated at point of creation, rather than at point of call
(both of these may differ).

Regards,
Nikita

--00000000000037309905c6397047--