Edit report at http://bugs.php.net/bug.php?id=52653&edit=1
ID: 52653
Comment by: t-bader at gmx dot net
Reported by: paulscheltema at gmail dot com
Summary: multiple instances made with a singleton
Status: Open
Type: Bug
Package: Reproducible crash
Operating System: debian
PHP Version: 5.3.3
Block user comment: N
New Comment:
It's not a bug.
At test::getInstance() the __construct method is invoked which retrieves
the instance from test2.
The __construct of test2 is invoked and clals test::getInstance(), since
the first call of test::getInstance doesn't finished the constructor
execution self::$instance will never be set and the __construct method
of test is invoked again.
Same with test2:getInstance, so you created an endless loop because the
__construct methods never can finish.
Previous Comments:
------------------------------------------------------------------------
[2010-08-20 11:28:36] paulscheltema at gmail dot com
Description:
------------
Hello, first off, ive been searching but i couldnt find any related
bugreport and im sorry if i wasted your time.
I have 2 classes with both an singleton function in them, an api (test)
and a user (test2) class. The api uses the user data and the user class
uses the api to get its userdata. Hence i load the user class instance
in the api and vice versa.
It works if i use an external class to serve as a singleton storage but
if i use the static $instance it doesnt.
The singleton function does work when i remove the test2::getInstance()
from class test.
Test script:
---------------
class test {
private static $instance;
private $test2;
private function __construct() {
$this->test2 = test2::getInstance();
}
public static function getInstance() {
if (!isset(self::$instance)) {
echo '<br>new test instance';
$c = __CLASS__;
self::$instance = new $c;
}
return self::$instance;
}
}
class test2 {
private static $instance;
private $test;
private function __construct() {
$this->test = test::getInstance();
}
public static function getInstance() {
if (!isset(self::$instance)) {
echo '<br>new test2 instance';
$c = __CLASS__;
self::$instance = new $c;
}
return self::$instance;
}
}
print 'start:<br>';
for ($i = 1; $i < 10; $i++) {
print '<br>loop: '.$i;
$t = test::getInstance();
}
Expected result:
----------------
start:
loop: 1
new test instance
new tes2 instance
loop: 2
loop: 3
loop: 4
loop: 5
loop: 6
loop: 7
loop: 8
loop: 9
Actual result:
--------------
start:
loop: 1
new test instance
new test2 instance
new test instance
new test2 instance
new test instance
new test2 instance
new test instance
new test2 instance
new test instance
new test2 instance
...
till php runs out of memory
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/bug.php?id=52653&edit=1
