Discussion :

[alejandro]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

vous obtenez les logs suivants :

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

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$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.

[karl3i]

Bonjour,

j'ai exécuté le code suivant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
in GetInstance 
in construct

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[bigltnt]

Une petite erreur:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 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 :

Sélectionner tout - Visualiser dans une fenêtre à part

 self::$o_Instance = new self();

[karl3i]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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>";
?>

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