Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:107886
MIME-Version: 1.0
References: <CAEsg9X0irm1F2CgvrQUXx+RwBhfnoZ49Sc1C8nJhhSUzUO12dQ@mail.gmail.com>
 <CA+kxMuSQSOE_BaMWJFTo8y5qWyQnsryz_wfY-o97277H7hxnsQ@mail.gmail.com>
In-Reply-To: <CA+kxMuSQSOE_BaMWJFTo8y5qWyQnsryz_wfY-o97277H7hxnsQ@mail.gmail.com>
Date: Thu, 28 Nov 2019 14:18:27 -0300
Message-ID: <CAEsg9X0Bvw7c_bGMRpxG=zDfFTRVKdzmFj5j_YXnExgYXwq=JQ@mail.gmail.com>
To: Dan Ackroyd <Danack@basereality.com>
Cc: PHP Internals <internals@lists.php.net>, Claude Pache <claude.pache@gmail.com>
Content-Type: multipart/alternative; boundary="000000000000ed606705986b4d03"
Subject: Re: [PHP-DEV] register_shutdown_function() not supports private methods
From: david.proweb@gmail.com (David Rodrigues)

--000000000000ed606705986b4d03
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Em qui., 28 de nov. de 2019 =C3=A0s 14:13, Dan Ackroyd <Danack@basereality.=
com>
escreveu:

> On Wed, 27 Nov 2019 at 23:33, David Rodrigues <david.proweb@gmail.com>
> wrote:
> >
> > Hi internals,
> >
> > I am using the register_shutdown_function() inside a class, and then I
> have
> > created a private method to be called during shutdown. I have noted tha=
t
> if
> > the method is public it works, but private does not.
> >
> > This shutdown function should be private,
> > to avoid execution at a public scope.
>
> That is one of the reasons Closure::fromCallable was added:
> https://www.php.net/manual/en/closure.fromcallable.php
>
> So like: register_shutdown_function(Closure::fromCallable([$this, 'leave'
> ]));
> Or in your example: https://3v4l.org/MSLA4


Thanks! It will solve the problem.



>
>
> > spl_autoload_register([ self::class, 'privateMethod' ]); // OK
>
> That is not OK. It might not give an error, but it's a bug if a
> private method can be called accidentally, without jumping through the
> deliberate hoops to work around private methods.
>

In that case, it should be deprecated?



>
> Claude Pache wrote:
> > Another workaround is to use a closure:
>
> Although that works, using Closure::fromCallable() has the benefits of
> preserving the parameters and their info.
>
> function foo(int $x) {
> }
>
> $fn =3D Closure::fromCallable('foo');
> $reflection =3D new ReflectionFunction($fn);
>
> foreach ($reflection->getParameters() as $param) {
>     echo "Param type is: " . $param->getType();
> }
> // Output is "Param type is: int"
>
> cheers
> Dan
> Ack
>


--=20
David Rodrigues

--000000000000ed606705986b4d03--