Discussion :

[rib]

j'utlise une solution xampp (PHP Version 5.1.6),apparement il y a des gens sur le net qui disent que c un bug de php mais y aurait'il qqun ici pour m'aider a trouver la source du probleme svp?

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
<?php
class Items {
   static private $instance=false;
   private $Collection=false;
 
 
    // Ajouter un item dans la colection
	public function AddItem($item, $key){
		$Collection[$key] = $item; 
	}
 
	// Recuperer un item grace a son nom
	public function GetItem($key){
		if(Items::set($this->Collection, $key)){
			return current($this->Collection);
		}
		else{ return "Error0";}
	}
 
	//fonction set qui sert a se positioner dans la collection
	private function set($Collection, $key){
    	reset($Collection);
	    while($current=key($Collection)){
		   if($current==$key)
		   {
			   return true;
		   }
		   next($Collection);
	   }
	   return false;
	 }	
 
   // le constructeur est privé, pour empêcher qu'il soit directement instancié 
   private function __construct() {
      $this->Collection = array();       
   }
 
   // méthode statique pour créer une instance de la classe Multilingue
   static function instance() {
       if(!Items::$instance) {
           Items::$instance = new Items();
       }
       return Items::$instance;
   }
}
 
?>

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
<?php
include 'Items.php';
 
$I = Items::instance();
Items::AddItem("test", "key1");
Items::AddItem("test2", "key2");
Items::AddItem("test3", "key3");
 
$A = Items::GetItem("key1");
$B = Items::GetItem("key2");
$C = Items::GetItem("key3");
 
echo $A + ";" + $B + ";" + $C + ".";
 
?>

[Eusebius]

Pour moi AddItem et GetItem ne sont pas des méthodes statiques, elles doivent être appelées sur une instance de la classe.

[rib]

AddItem ne retourne pas d'erreur, l'ereur se produit a l'appel dela fonction set()

[rib]

ici il disent que c un bug de php mais je sait pas comment metre a jour php de xampp
http://bugs.php.net/bug.php?id=24403

[Eusebius]

AddItem ne retourne pas d'erreur, l'ereur se produit a l'appel dela fonction set()

Le problème que ça me pose est surtout d'ordre logique : comment ajoute-t-on un item à un objet qui n'est pas défini ? C'est une autre manière de dire qu'on ne peut pas utiliser $this en-dehors d'une instance.

Pour moi PHP aurait dû t'empêcher d'appeler AddItem.

ici il disent que c un bug de php mais je sait pas comment metre a jour php de xampp
http://bugs.php.net/bug.php?id=24403

Je n'ai pas l'impression que ça soit le même problème (mais je peux me tromper)

[rib]

comment sa pas definit?

[Dia_FR]

t'as testé avec $I->AddItem au lieu de Items::AddItem ?
pareil pour get

[Eusebius]

comment sa pas definit?

quand tu appelles Items::AddItem(...), tu n'appelles pas la méthode d'une instance, tu appelles une méthode statique d'une classe (ce qui n'est pas le cas). La méthode n'est pas mise en relation avec une instance (donc, pas de $this).
Je pense qu'il faudrait que tu fasses :

Code :

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

$I->AddItem(...)

[rib]

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
<?php
include 'Items.php';
 
$I = Items::instance();
//Items::AddItem("test", "key1");
//Items::AddItem("test2", "key2");
//Items::AddItem("test3", "key3");
$I->AddItem("test", "key1");
$J->AddItem("test", "key2");
$K->AddItem("test", "key3");
 
 
$A->GetItem("key1");
$B->GetItem("key2");
$C->GetItem("key3");
 
 
 
//$A = Items::GetItem("key1");
//$B = Items::GetItem("key2");
//$C = Items::GetItem("key3");
 
echo $A + ";" + $B + ";" + $C + ".";
 
?>

Fatal error: Call to a member function AddItem() on a non-object in C:\Program Files\xampp\htdocs\KORPORATE\test_item.php on line 9

[Dia_FR]

dans Instance tu crées un objet de classe Items mais tu récupères pas de référence dessus, Instance renvoie un booléen

donc $I est un booléen à priori

donc ça risque pas de marcher comme ça, j'avais regardé d'assez prêt
fais un constructeur classique où tu changes la valeur de $instance (ce que tu ne fais pas dans la méthode Instance d'ailleurs) et tu l'appelles avec un new

de plus $J, $K, $A, $B et $C ne sont pas définis

[rib]

la methode instance sert juste a verifier si il ny a pas deja un objet de cet classe instancie et il sert a totaliser le nombre d'instances(static) et si y en a pas elle instance ma classe Item()

Code :

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

1
2
3
4
5
 if(!Items::$instance) {Items::$instance = new Items();}
ce qui appelle automatiquement ce constructeur :
private function __construct() {
      $this->Collection = array();       
   }

et celui ci instancie mon tableau ...
c du php5 enfin reprenez moi si je me trompe mais cet methode fonctionne tres bien avec d'autres classes que j'utilise.

[rib]

sinon y aurai pas une collection toute faite dans l'api php dans le style ds hashtable en java?

[Dia_FR]

oulala j'ai lu en diagonale désolé

donc ouais, c'est bon, la seule remarque à retenir de ce que j'ai dit avant c'est que y a que $I qu'est un objet de classe Items, y a que lui qui peut appeler les méthodes

[rib]

bon :

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
<?php
include 'Items.php';
 
//$I = Items::instance();
//Items::AddItem("test", "key1");
//Items::AddItem("test2", "key2");
//Items::AddItem("test3", "key3");
$I = new Items();
 
$I->AddItem("test", "key1");
$I->AddItem("test1", "key2");
$I->AddItem("test2", "key3");
 
$A = $I->GetItem("key1");
$B = $I->GetItem("key2");
$C = $I->GetItem("key3");
 
//$A = Items::GetItem("key1");
//$B = Items::GetItem("key2");
//$C = Items::GetItem("key3");
 
echo $A + ";" + $B + ";" + $C + ".";
 
?>

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
<?php
class Items {
   static private $instance=false;
   private $Collection=false;
 
 
    // Ajouter un item dans la colection
	public function AddItem($item, $key){
		$Collection[$key] = $item; 
	}
 
	// Recuperer un item grace a son nom
	public function GetItem($key){
		if(Items::set($this->Collection, $key)){
			return current($this->Collection);
		}
		else{ return "Error0";}
	}
 
	//fonction set qui sert a se positioner dans la collection
	private function set($Collection, $key){
    	reset($Collection);
	    while($current=key($Collection)){
		   if($current==$key)
		   {
			   return true;
		   }
		   next($Collection);
	   }
	   return false;
	 }	
 
 
	public function Items(){
	$this->Collection = array();
	}
   // le constructeur est privé, pour empêcher qu'il soit directement instancié 
   //private function __construct() {
     //        
   //}
 
   // méthode statique pour créer une instance de la classe Multilingue
   static function instance() {
       if(!Items::$instance) {
           Items::$instance = new Items();
       }
       return Items::$instance;
   }
}
 
?>

bon la y a un constructeur public normal mais sa me retourne 0 sur l'echo au lieu de la chaine de caractere attendue...?
mais cet methode me plait pas car le constructeur est public....
je prefererais pouvoir utiliser:

Code :

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

1
2
3
$I = Items::instance();
Items::AddItem("test", "key1");
$A = Items::GetItem("key1");

[jeff_!]

ou

Code :

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

1
2
3
 $tab = ArrayObject();
$tab['key1'] = $val1;
echo $tab['key1'];

et tu peut etendre ArrayObjet et utiliser un singleton pour l'unicité

[Mr N.]

Code :

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

if(Items::set($this->Collection, $key)){

Tu fais un appel statique sur une methode qui ne l'est pas

Sinon y a moyen de mettre ton constructeur en protected ?