Discussion :

[pierrot10]

Bonjour à tous,

je retravaille un petit peu un code que j'avais fait en OO et je dois donc le corrigé..

J'ai une classe :

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
 
class Validate{ 						
	protected $toBeValidate = array();
 
	function __construct(){}
 
	public function __get($name){
		return $this->$name;
	}
 
	function Validate($labelName,$fieldName,$typeOfValidation,$condition){
 
		$this->toBeValidate[] = str_replace(" ","_",$labelName).'*'.str_replace(" ","_",$fieldName).'*'.$typeOfValidation.'*'.$condition;
	}
 
	function _Display(){															// Obsolete. It let's you to see the field name to be check for validation
		print_r($this->toBeValidate);
	}
}

Le tableau protected $toBeValidate est correctement "allimenté".

J'ai aussi une fonction
(je ne mets pas tout le code, car il est enorme et fonctionne bien)

Code :

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

1
2
3
 
function CheckNowTheFieldsContentAndRecordTheResult($method,$checker){
}

Dans cette fonction, j'ai besoin de récupérer la valeur de $toBeValidate de la class précédente, sans devoir faire ceci

Code :

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

CheckNowTheFieldsContentAndRecordTheResult($method,$checker,$toBeValidate){}

Comment puis-je donc le faire. Je pensais que ceci marcherait, mais non

Code :

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

1
2
3
4
 
CheckNowTheFieldsContentAndRecordTheResult($method,$checker){
 Validate->toBeValidate
}

Merci pour votre aide

Je me demande si dans ma class Validate, je ne devrais pas tranformer ma fonction Validate en static.
D'ailleur elle porte le même nom, es-ce bien?

Comme ceci:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
class Validate{ 						
	protected $toBeValidate = array();
 
	//function __construct(){}
 
 
	static function Validate($labelName,$fieldName,$typeOfValidation,$condition){
 
		$this->toBeValidate[] = str_replace(" ","_",$labelName).'*'.str_replace(" ","_",$fieldName).'*'.$typeOfValidation.'*'.$condition;
	}
 
}

Pourriez-vous me corriger?

Merci

[Herode]

Bonjour,

1 - utilisation de static : pourquoi mettre la fonction Validate() en static ? Qu'attends-tu exactement de ce comportement ? En tout état de cause, une méthode statique est une méthode de classe, pas une méthode d'instance. Une méthode ou un attribut statique sont partagés par tous les objets de la classe. Et tu n'as pas besoin d'instancier un objet pour appeler une méthode statique.

Exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
class Foo {
   public function nonStatique() {}
   public static function statique() {}
}
 
// Appel de la méthode non statique : instance obligatoire
$a = new Foo(); 
$a->nonStatique();
 
// appel de la méthode statique : instance facultative
Foo::nonStatique();
 
// mais on peut aussi l'appeler avec un objet instancié si on veut
$a->nonStatique()

Bien sûr, comme une méthode statique n'est pas rattachée à une instance, $this n'est pas défini dans le corps de cette méthode.

2 - utilisation de protected :

Code :

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

Validate->toBeValidate

Validate est un nom de classe, ça ne va pas marcher.

Code :

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

1
2
$obj = new Validate(); 
$obj->toBeValidate

serait mieux mais $toBeValidate est déclaré protected, tu ne peux donc pas y accéder en dehors de la classe.

La solution, vu le code de ta classe :

Code :

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

1
2
$obj = new Validate(); 
$obj->__get('toBeValidate');

Autres solutions :
- rendre $toBeValidate public (déconseillé)
- développer des accesseurs propres au lieu du __get(string), qui du reste va planter salement quand tu essayeras d'accéder à un attribut qui n'existe pas.

Classiquement :

Code :

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

1
2
3
4
class Validate {
   protected $monattribut;
   public function getMonAttribut() { return $this->monattribut; }
}

L'approche avec une méthode du genre de ton __get() est aussi régulièrement utilisée, mais plutôt sur des gros frameworks. Elle a un petit avantage (on n'est pas obligé de déclarer un accesseur pour chaque nouvel attribut, léger gain de temps) et quelques inconvénients notables - mais on sort ici du sujet.

3 - Noms des méthodes :

D'ailleurs elle porte le même nom, es-ce bien?

C'est probablement une erreur. En PHP5, si tu ne déclares pas de constructeur (méthode __construct() mais que tu déclares une méthode de même nom que la classe, celle-ci sera utilisée comme constructeur par défaut. Ceci garantit une compatibilité avec le code hérité de PHP4. Mais... est-ce vraiment ce que tu veux pour ta métode Validate() ? Je pense que non...

Cela dit, mon premier conseil serait que tu révises et que tu assimiles les concepts généraux de la POO car l'utilisation de static et les contrôles d'accès public/protected/private sont tout de même des concepts de base. Si tu ne connais pas ça, tu vas avoir beaucoup de problèmes

[pierrot10]

Ok, super! Je te remercie pour tes explications et pour le temps que tu as pris pour ça.
Je vais tout regardré ceci