Discussion :

[vince-nantes]

Bonjour,

J'ai une question basique pour bien organiser mes méthodes de classe, mais j'ai un doute et j'aurai besoin de vos avis.

Je prends un exemple simplifié : j'ai un catalogue de produits et chacun à une famille, quand j'affiche le catalogue des produits avec une famille donnée, j'ai besoin d'une requète du style "SELECT * FROM produit WHERE id_famille = 5".

Est-ce que je dois placer une méthode "getProducts(id_famille)" dans le manager de ma classe "Famille" ou une méthode "getListByIdFamily(id_famille)" dans le manager de ma classe "Produit" ?

Je pencherai pour la 2ème solution, parce que ça concerne plus les produits que la famille en elle-même.

Merci d'avance pour vos réponses

[papajoker]

Bonjour,
Je répondrais aussi la seconde solution.

Je trouve que ce tuto ne vas pas assez loin.
Il n'y a aucune notion d'héritage

il devrait avoir a minima
une classe abstraite Model, et donc l'étendre avec Famille et Produit.
une classe abstraite DbManager, et donc l'étendre avec FamilleManager et ProduitManager

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
52
53
54
55
56
abstract class Model { }
class Personnage extends Model { }
 
 
abstract class DbManager
{
  private $_db; // Instance de PDO
  protected $_tableName;
 
  public function __construct($db)
  {
    $this->setDb($db);
  }
 
  public function save(Model $model)
  {
	if ($model->getId()<1)
		return $this->insert($model);
	else
	    return $this->update($model);
  }
 
  protected function setDb(PDO $db)
  {
    $this->_db = $db;
  }
 
  abstract protected function insert(Model $model);
  abstract protected function update(Model $model);
  abstract public function delete(Model $model);
 
}
 
class PersonnagesManager extends DbManager
{
	protected $_tableName='personnages';
 
    protected function insert(Model $perso)
	{
		if (!($perso instanceof Personnage)) throw new Exception('Type incompatible'); 
		$q = $this->_db->prepare('INSERT INTO '.$this->_tableName.' SET nom = :nom');
		$q->bindValue(':nom', $perso->nom());
		$q->execute();
    }
 
	protected function update(Model $perso)
	{
		if (!($perso instanceof Personnage)) throw new Exception('Type incompatible'); 
		$q = $this->_db->prepare('UPDATE '.$this->_tableName.' SET forcePerso = :forcePerso WHERE id = :id');
		$q->bindValue(':forcePerso', $perso->forcePerso(), PDO::PARAM_INT);
		$q->bindValue(':id', $perso->getId(), PDO::PARAM_INT);
		$q->execute();
	}
 
	public function delete(Model $model) {}
}

[vince-nantes]

Merci pour ta réponse, le lien vers le tutoriel a été supprimé par Bovino, sans doute parce qu'il faisait référence à un site de développement concurrent.

D'autres avis ?

[papajoker]

Bon je vais répondre un peux plus a ta question :

ProduitManager ne devrait connaitre(retourner) comme class que Produit
FamilleManager que Famille

Se sont 2 entités SQL différentes.
Sinon tu te retrouves avec un couplage (très) fort :
le jour ou tu changes quelque chose dans Produit, ça ne devrais pas casser FamilleManager


sinon il y a cette approche : Activerecord
http://julien-pauli.developpez.com/tutoriels/php/pdo/

[vince-nantes]

ProduitManager ne devrait connaitre(retourner) comme class que Produit
FamilleManager que Famille

C'est le cas dans la 2ème solution, par exemple : la méthode "get(id_produit)" du manager de ma classe "Produit" me retourne bien un objet "Produit", tandis que que la méthode "getListByIdFamily(id_famille)" me retourne un tableau d'objets "Produit" (en prenant exemple sur la méthode "getList()" du tutoriel).

Ca me semble effectivement plus cohérent que la 1ère solution, où le manager de ma classe "Famille" devrait me retourner un tableau d'objets "Produit"...

le jour ou tu changes quelque chose dans Produit, ça ne devrais pas casser FamilleManager

Ca confirme encore la 2ème solution qui ne cassera pas le manager de ma classe "Famille".

Merci pour tes réponses et tes propositions d'évolution avec des héritages.
Je mets le sujet en résolu, si quelqu'un a un avis différent, merci de continuer le sujet.