Discussion :

[Kontas]

Salut à tous, j'ai une petite question sur les bonnes pratiques à avoir en programmation orienté objet.

Quelle est la méthode la plus logique parmi ses trois ou une 4^^, sur l'utilisation des objets pour l'insertion en base de données. L'une je passe les valeurs à ajouter dans les paramètres de la méthode et l'autre je défini les valeurs des variables de l'objet


Méthode 1

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
 
 
class maClasse{
 
 var $id;
 var $valeur;
 
 function update($id,$valeur){
 
  $query = "UPDATE table SET valeur = '".$valeur."' WHERE id='".$id."' ";
  $result = mysqli_query ($link,$query);
 
  etc .....
 
 }
 
}
 
$maClasse = new maClasse;
$maClasse->update($id,$valeur);

Méthode 2

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
 
 
class maClasse{
 
 var $id;
 var $valeur;
 
 function update(){
 
  $query = "UPDATE table  SET valeur = '".$this->valeur."' WHERE id='".$this->id."' "
  $result = mysqli_query ($link,$query);
 
  etc .....
 
 }
 
}
 
$maClasse = new maClasse;
$maClasse->id = 10;
$maClasse->valeur = 'test';
$maClasse->update();

Un membre d'un autre forum m'a proposé cette 3ème méthode

Code php :

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
 
class maClasse{
protected $id;
protected $valeur;
 
function __construct($id, $val){
...
}
 
function save(){
 if(this->existeDeja(){
$this->update();
}
else{
$this->insert()
}
}
 
protected function update(){
  $query = "UPDATE table  SET valeur = '".$this->valeur."' WHERE id='".$this->id."' "
  $result = mysqli_query ($link,$query);
  etc .....
}
 
protected function insert(){
  $query = "insert into table  ...' "
  $result = mysqli_query ($link,$query);
  etc .....
}
 
}
$maClasse = new maClasse(10, 'test');
$maClasse->save();

Merci

[Tsilefy]

Ta question ramène à la question d'ordre générale de l'endroit où on doit injecter les paramètres.

- Solution no.1: injection dans le constructeur. À faire quand les paramètres sont obligatoires pour l'existence de l'objet. Dans ton cas, est-ce que ton objet peut exister sans $id et $valeur? Si non, il faut les injecter dans le constructeur.

- Solution no.2: si les paramètres ne sont pas obligatoires pour l'existence de ton objet, tu peux les injecter dans la méthode. On pourrait certes se demander si l'utilisation de paramètres non obligatoires n'entraîne pas une violation du principe de responsabilité unique - et c'est le cas dans un certain nombre de cas...

- Solution no.3: injection directe dans les propriétés - Oublie. Ça brise complètement l'encapsulation de ta classe, ce n'est plus la peine d'avoir des objets dans ce cas. En tout cas tant que PHP n'a pas les accesseurs à la C#

[Kontas]

Oky, donc vis a vis des exemple que j'ai donnée,

Soit l'exemple 1, avec le passage des infos dans les paramètre de la méthode donnée,
Soit l'exemple 3, avec le passage des info directement dans le constructeur.

Si je prend l'exemple d'une classe pour la gestion des utilisateurs, au moment de créer l'objet, je n'ai pas forcement toutes les informations, mais plutôt que sont id d'utilisateur. Dans ce cas si j'utilise le deuxieme exemple je vais créer un objet en passant que 1 paramètre sur les 10 (environ) au total de mon constructeur, c'est pas top top. A moins de passer par une méthode static , qui dans son code, renseigne les variable utilisateur et me retourne un objet avec tout les paramètres.

[grunk]

Ton objet peut tout à fait avoir un certains nombre d'attributs et n'en spécifier que certains lors de son instanciation. Les autres pouvant être ajouter via des setters :

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
class Personne {
 private $nom;
 private $prenom;
 private $age;
 
 public function __construct($nom) {
	$this->nom = $nom;
 }
 
 public function setNom($nom) {
	$this->nom = $nom;
 }
 
 public function setPrenom($prenom) {
	$this->prenom = $prenom;
 }
 
 public function setNom($age) {
	if($age > 0)
		$this->age = $nom;
	else
		throw new InvalidArgumentException("Age doit être positif");
 }
 
}

Tant qu'on est dans les bonnes pratiques la classe représentant ton objet ne devrait pas gérer les interactions avec la base de données. C'est une autre classe qui devrait être dédié à cette tache. Tu passerais simplement en paramètre à cette classe l'objet que tu veux enregistrer :

Code :

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

1
2
3
4
5
class PersonneDB {
	public function update(Personne $personne) {
		//Requeste sql pour enregistrer l'objet.
	}
}

Et pour finir oublie var devant les attributs de ta classe et donne leur une visibilité à la place (public, protected ou private)

[Kontas]

Oky, c'est simplement que créer un nouvel objet de cette manière je trouve ca assez moche

Code :

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

$utilisateur = new utilisateur('','','','','','',$nom);

Pour ce qui est des Var, c'est du au javascript que j'ai fais récemment, mais dans mon php il n'y sont jamais si cela peut te rassurer ^^

A force de lire des articles et autre tuto je retrouve assez fréquemment la séparation des classes qui font les action en base de donnée et celle qui manipule ces données, mais si je trouve cela affreusement redondant je le comprend assez, notamment dans le cas de changement d'un système de base, mais pour des petits projet, es ce bien nécessaire ?!

[Dendrite]

Pour des petits projets, je trouve extrêmement lisible de trouver dans la page appelante un code du genre :

Pour être plus claire, je dissocie dans mes classes les accès aux bases qui génèrent des array PHP, et les classes qui à partir d'un array PHP, génère des objets HTML.

Un espèce de MVC pour enfants. sans la partie modele... un VC quoi. Et ce sont les pages appelantes qui déroulent les procédures de façon assez lisible.

Ce que j'aime dans cette méthode, c'est que tous mes petits projets ont les mêmes noms de classes : connex, manager, view et tools... et que je ne me demande pas d'un projet sur l'autre comment ranger. C'est suffisamment abstrait pour ranger toujours pareil.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
$manager=new Manager();
$view=new View();
 
 
$ma_base=new Connex('ma_base');
$sql='select * from ma_table;';
 
$data=$manager->retourne_data($ma_base->pdo,$sql);
$table=$view->retourne_table($data,'Toutes les données de ma_table');
 
echo $table;

Mais bon, c'est ma bricole perso après 6 années d'expérience, et je lirai avec intérêt tous les conseils, cirtiques, admonestations, récriminations que vous jugerez bon de mettre sur ce fil.

[Tsilefy]

Oky, c'est simplement que créer un nouvel objet de cette manière je trouve ca assez moche

Code :

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

$utilisateur = new utilisateur('','','','','','',$nom);

Ouch, ouch, ouch. Si un constructeur a autant de paramètres, c'est le signe d'une mauvaise conception. Il est probable que ta classe fait beaucoup trop de choses (dans l'idéal, une classe ne doit faire qu'une seule et unique chose), ou que tous ces paramètres ne soient pas nécessaires pour instancier un object (et doivent donc aller dans une méthode).

Dans les rares cas où ta classe ne fait qu'une chose et que ces paramètres sont vraiment utiles, il peut être nécessaire de grouper ces paramètres en ce qu'on appelle un Data Transfer Object, ou même un simple tableau (l'avantage du Data Transfer Objet c'est qu'elle permet de valider les paramètres automatiquement, tandis que le tableau est plus flexible et plus rapide).