Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:65088
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.220.174 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <236801cdf8b6$9cfdfdc0$d6f9f940$@starin.biz>
References: <CAKZZz6dUPzGKL2mCph5QogPn-qjTVMR12yHJdUvEBGC44pk64A@mail.gmail.com>
 <1358859827.1894.3158.camel@guybrush> <236801cdf8b6$9cfdfdc0$d6f9f940$@starin.biz>
Date: Tue, 22 Jan 2013 16:52:33 +0100
Message-ID: <CADyq6sL2DAQyu6ArNav0YnKssNkphHPbm_vjxhVKOAM7KM0AdA@mail.gmail.com>
To: nathan@starin.biz
Cc: =?UTF-8?Q?Johannes_Schl=C3=BCter?= <johannes@php.net>, 
	Pete Boere <pete@the-echoplex.net>, PHP Internals List <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=f46d043c7c080f213c04d3e2921b
Subject: Re: [PHP-DEV] __init magic method
From: ocramius@gmail.com (Marco Pivetta)

--f46d043c7c080f213c04d3e2921b
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

@Pete just FYI, PSR-1 excludes that such a logic should exist in a loaded
file:

https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-1-basic-c=
oding-standard.md#23-side-effects

Not sure if you are interested in following the FIG on these ideas, but the
guideline makes actually sense.


Marco Pivetta

http://twitter.com/Ocramius

http://ocramius.github.com/


On 22 January 2013 16:39, <nathan@starin.biz> wrote:

> I am actually going to +1 this idea I thought about discussing it, but
> opted out of it because classes are most often in their own file and if y=
ou
> need to do some static binding you can do it at the end of the file after
> the class definition. However, I do believe this idea would make it easie=
r
> to extend classes and make the code look much cleaner.
>
> @Johannes In your example I'd say it would be ADBCE because I would say i=
t
> should work like as if you had each one in it's own file autoloading them=
.
> (also I believe the function would be static)
>
> One of the potential problems I can see is the visibility
> (public/private/protected) of the function as it'd likely need to be publ=
ic
> because it could never be auto-executed if anything else.
>
> Software Developer
> Nathan Bruer
>
> -----Original Message-----
> From: Johannes Schl=C3=BCter [mailto:johannes@php.net]
> Sent: Tuesday, January 22, 2013 7:04 AM
> To: Pete Boere
> Cc: internals@lists.php.net
> Subject: Re: [PHP-DEV] __init magic method
>
> Hi,
>
> On Tue, 2013-01-22 at 12:29 +0000, Pete Boere wrote:
> > Has there ever been any discussion on an __init magic method for
> > bootstraping static properties on class load?
>
> Adding an __init method as you wish will just add complexity and gain ver=
y
> little. What complexity am I talking about? Well look at this single file=
:
>
>    <?php
>    class A {
>        public function __init() { echo __CLASS__; }
>    }
>    class B extends D {
>        public function __init() { echo __CLASS__; }
>    }
>    class C {
>        public function __init() { echo __CLASS__; }
>    }
>    class D {
>        public function __init() { echo __CLASS__; }
>    }
>    class E {
>        public function __init() { echo __CLASS__; }
>    }
>    ?>
>
> What should this print? ABCDE? Then the derived class B is initialized
> before the base class D, which might have strange effects and make
> debugging hard. So maybe it is ACDEB? This makes the resolution quite
> complicated.
>
> On the other hand: You are showing a goos solution, which is clear and
> works well. PHP has the global scope, we don't force everything into
> classes and then create Java-style static{}-madness.
>
> johannes
>
>
>
> --
> 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
>
>

--f46d043c7c080f213c04d3e2921b--