Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:57572
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain mindplay.dk from 209.85.214.170 cause and error)
MIME-Version: 1.0
In-Reply-To: <1327883576.6780.722.camel@guybrush>
References: <CADqTB_hEP4yQHa8jfm4s7YCvbBwUdH4eHiXZ68PDc_S-n4Zm4Q@mail.gmail.com>
	<1327883576.6780.722.camel@guybrush>
Date: Mon, 30 Jan 2012 09:33:20 -0500
Message-ID: <CADqTB_ijUe5ObVA3KWrQS=LbBLuNgABRsR=fwZhSSZVrvBCd_g@mail.gmail.com>
To: internals@lists.php.net
Content-Type: multipart/alternative; boundary=f46d0444ee556771d904b7bfba11
Subject: Re: [PHP-DEV] ReflectionFile missing
From: rasmus@mindplay.dk (Rasmus Schultz)

--f46d0444ee556771d904b7bfba11
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

From my point of view, the concept of a "file" has become semantically more
important, and increasingly relevant to Reflection, with the latest PHP
features added in the past couple of years.

I can see what you mean though - it's probably not a small effort, and
there are probably more important aspects of PHP that need attention...

2012/1/29 Johannes Schl=FCter <johannes@schlueters.de>

> Hi,
>
>
> On Sun, 2012-01-29 at 18:51 -0500, Rasmus Schultz wrote:
> > I realized the other day that ReflectionFile is missing from the
> Reflection
> > API.
>
> As is ReflectionNamespace and some others one might think about. In the
> end it boils down to the fact that we don't have structures internally
> representing them as for PHP there is no need to keep the data and then
> we're restrictive in adding such meta-data just for the purpose of
> reflection. Mind that we'd have to keep all that in memory and opcode
> caches have to copy it around etc. in the end the consensus was: the
> effort needed doesn't seem worthwhile.
>
> The alternative would be to "emulate" it, something like
>
>        class ReflectionFile {
>            private $filename;
>            private $classes =3D array();
>            private $functions =3D array();
>
>            private initFunctions($filename) {
>                foreach(get_defined_functions()['user'] as $f) {
>                    $rf =3D new ReflectionFunction($f);
>                    if ($rf->getFilename() =3D=3D $filename) {
>                        $this->functions[$rf->getName()] =3D $rf;
>                    }
>                }
>            }
>
>            private initClasses($filename) { /* ... */ }
>
>            public __construct($filename) {
>                $this->filename =3D realpath($filename);
>                $this->initFunctions($this->filename);
>                /* ... */
>            }
>
>            public getFunctions() {
>                return $this->functions;
>            }
>
>            public getFunctionNames() {
>                return array_keys($this->functions);
>            }
>
>            /* ... */
>        }
>
> But that feels more like a hack than a solution, too.
>
> johannes
>
> PS. Mind that the example you've given even works on files not included
> by parsing files, whereas internal reflection provides information what
> actually is available from engine point of view ...
>
>

--f46d0444ee556771d904b7bfba11--