Singleton qui s'initialise deux fois ?

Version imprimable

Bonjour, j'ai un problème d'initialisation avec mon singleton :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 <?php if( ! class_exists( 'Lib_Session' ) ) { class Lib_Session { protected static $o_Instance; protected function __construct() { $handle = fopen( "C:\\Log.txt", 'a+' ); $string = "In __construct() \n"; fwrite( $handle, $string ); fclose( $handle ); } public static function GetInstance() { $handle = fopen( "C:\\Log.txt", 'a+' ); $string = "In GetInstance() \n"; fwrite( $handle, $string ); fclose( $handle ); if( ! self::$o_Instance instanceof self ) self::$o_Instance = new self; return self::$o_Instance; } } }
En effet si vous regardes le fichier de logs situé à la racine du disque dur et généré par ces lignes :
Code:

1 2 3 4 $handle = fopen( "C:\\Log.txt", 'a+' ); $string = "In ... \n"; fwrite( $handle, $string ); fclose( $handle );
vous obtenez les logs suivants :

Citation:

In GetInstance()
In __construct()
In GetInstance()
In __construct()

Dans un autre fichier j'initialise mon singleton comme ça :

Code:

$this->o_Session = Lib_Session::GetInstance();

Si je commente cette ligne, mon fichier de logs reste vierge, prouvant qu'aucun autre appel à GetInstance() n'est effectué ailleurs dans le programme.

Alors pourquoi ma fonction GetInstance() est-elle appelée deux fois ?
Et surtout pourquoi mon constructeur __construct() également ?

Merci pour votre aide.

Bonjour,

j'ai exécuté le code suivant :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
 
<?php
 
if( ! class_exists( 'Lib_Session' ) )
{
  class Lib_Session
  {
    protected static $o_Instance;
 
    protected function __construct()
    {
      echo 'in construct <br>';
    }
 
    public static function GetInstance()
    {
      echo 'in GetInstance <br>';
 
      if( ! self::$o_Instance instanceof self )
      self::$o_Instance = new self;
 
      return self::$o_Instance;
    }
  }
}
 
$toto = Lib_Session::GetInstance();
 
?>

et j'ai bien le résultat une seule fois :

Code:

1
2
3
 
in GetInstance 
in construct

Sinon, je crois que le pattern demande à mettre le constructeur & le membre en private.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
 
<?php
 
if( ! class_exists( 'Lib_Session' ) )
{
  class Lib_Session
  {
    private static $o_Instance;
 
    private function __construct()
    {
      echo 'in construct <br>';
    }
 
    public static function GetInstance()
    {
      echo 'in GetInstance <br>';
 
      if( !isset(self::$o_Instance) ) {
      self::$o_Instance = new self;
      echo "j'instancie <br>";
      }
 
      return self::$o_Instance;
    }
  }
}
 
$toto = Lib_Session::GetInstance();
 
?>

Enfin, attention à implémenter __clone() pour jeter une exception ou faire un trigger d'erreur, pour éviter que l'instance ne se trouve dupliqué.
Est-ce que si tu testes les bouts de code que je t'ai mis, ça fonctionne chez toi ?

Karl3i

26/08/2009, 15h11
bigltnt

Une petite erreur:

Code:

self::$o_Instance = new self;

je n'ai jamais appellé un contructeur avec un self, mais dans tous les cas il manque les parenthèses:

Code:

self::$o_Instance = new self();

En quoi est-ce une erreur pour un constructeur qui n'attend pas de paramètre ?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 <?php class A { protected $test; public function __construct() { $this->test = "init"; } } $a = new A; echo "<pre>"; print_r($a); echo "</pre>"; ?>
Citation:

A Object
(
[test:protected] => init
)