[PHP5] Singleton-functie haalt niet goede instantie op

[Dutchmega]

Eerst maar eens wat source:

test.php

PHP:

<?php
ini_set ("zend.ze1_compatibility_mode", "0");
error_reporting (E_ALL|E_STRICT);

class test {
    public static $instance = NULL;
    public $variable = ":P";
    
    public static function getInstance() {
        if (self::$instance == NULL) {
            self::$instance = new test();
        }
        return self::$instance;
    }
    
    public function __construct () {
        $this->variable = ":D";
        $this->load();
    }
    
    private function load () {
        include ("test2.php");
        $class = test2::getInstance();
    }

}

test::getInstance();
?>

test2.php

PHP:

<?php
ini_set ("zend.ze1_compatibility_mode", "0");
error_reporting (E_ALL|E_STRICT);

class test2 {
    public static $instance = NULL;
    
    public static function getInstance() {
        if (self::$instance == NULL) {
            self::$instance = new test2();
        }
        return self::$instance;
    }
    
    private function __construct () {
                 // Probleem is deze regel :)
        print_r (test::getInstance());

        echo ":D";
    }

}
?>

Error: Fatal error: Cannot redeclare class test2 in f:\dutchmega\www\test2.php on line 7

Toelichting: Class test included test2 class en roept het aan d.m.v PHP5 singleton-pattern. (Lees: static functie )
In test2 class roep ik de instantie van de test class weer aan maar hij pakt een andere instantie waardoor test2.php nog een keer geincluded wordt..

Het punt is, getInstance() haalt niet de goede instantie op maar dat zou gewoon goed moeten zijn? Want ik roep de class gewoon statisch aan..Ik snap niet dat als je het vanuit een classfunctie roept, dat het gewoon niet werkt.. PHP-bug of wat anders

* Dutchmega draait trouwens PHP 5.02
EDIT: Ook @ laatste snapshots..

[ Voor 17% gewijzigd door Dutchmega op 26-11-2004 22:20 ]

[Delphi32]

Volgens mij is het wel logisch dat ie dit niet slikt. Er wordt inderdaad een tweede instantie van test aangemaakt en dat komt omdat test::$instance nog NULL is op het moment dat test2 test aanroept. test is nog bezig met zichzelf te construeren, test::$instance krijgt pas een waarde als de constructie klaar is. Zo bezien is het maar goed dat ie dit niet leuk vindt, anders zat je nu met een infinite loop
Hoe je het op moet lossen weet ik niet (ken geen PHP) maar is het geen optie op test::load() aan te roepen in test::getInstance in plaats van in test::__construct?

[Grijze Vos]

maak es van die include een include_once

[Dutchmega]

Ik heb het al.. Het is kleine fout van mij jah.. Ik roep namelijk eerder new test() aan dan $instance te defineren...

Het is op te lossen door self::$instance = &$this; in __construct() te zetten.. Iets viezer maargoed.