Discussion :

[laurentSc]

Bonjour,

étant débutant en POO, je me pose une question de conception à savoir si je dois créer une ou 2 classes.

J'ai déjà créé la bdd (MySQL). La partie qui nous intéresse ici est modélisée par le MCD suivant :

Il y aura donc une table SQL LI_license et une autre table SQL OWN_owner_license :

Code sql :

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
CREATE TABLE LI_license(
   LI_ident INT UNSIGNED AUTO_INCREMENT,
   LI_activate_date DATE NOT NULL,
   LI_deactivate_date DATE,
   OWN_ident INT UNSIGNED NOT NULL,
   AP_ident INT UNSIGNED NOT NULL,
   PRIMARY KEY(LI_ident),
   FOREIGN KEY(OWN_ident) REFERENCES OWN_owner_license(OWN_ident),
   FOREIGN KEY(AP_ident) REFERENCES AP_application(AP_ident)
);
 
 
CREATE TABLE OWN_owner_license(
   OWN_ident INT UNSIGNED AUTO_INCREMENT,
   OWN_email VARCHAR(50),
   US_ident INT UNSIGNED NOT NULL,
   PRIMARY KEY(OWN_ident),
   FOREIGN KEY(US_ident) REFERENCES US_user(US_ident)
);

Je veux maintenant créé le code PHP (POO/MVC) qui va gérer cette bdd. Donc ma question : est-ce que je crée une seule classe LI_license avec les propriétés LI_activate_date, LI_deactivate_date et OWN_email ou bien 2 classes :
classe LI_license avec les propriétés LI_activate_date, LI_deactivate_date et ref_OWN_owner_license et classe OWN_owner_license avec la propriété OWN_email ?
(Dans cette discussion je ne tiens pas compte de la classe US_user ni de l'héritage sous-jacent)

[grunk]

Une seule classe est suffisante et surtout bien plus pratique à l'utilisation. La notion de responsabilité limitée est surtout là pour éviter des classes qui font tout et n'importe quoi.

Dans cette discussion je ne tiens pas compte de la classe US_user ni de l'héritage sous-jacent

Attention à ne pas nous coller un héritage entre licence et user qui n'aurait absolument aucun sens.

Ce qui est éventuellement faisable c'est que ta classe Licence soit composé d'un objet User duquel tu pourras extraire l'email, mais ce n'est pas une obligation (plus complexe , moins performant).

[laurentSc]

Merci pour ta réponse.
OK pour une seule classe LI_license. Oui, l'héritage, j'y pensais surtout si je créais 2 classes. Ce que j'imagine, c'est une 2e classe US_user et dans la classe LI_license, une propriété ref_US_user pour que la classe LI_license puisse connaître les propriétés de la classe ref_US_user. C'est correct ?

[grunk]

Oui c'est ca.

Tu as n gros 2 solutions :

Soit tu n'auras besoin que d'un champs bien précis de ton utilisateur associé avec la license , auquel cas pas besoin de s'embeter à avoir un utilisateur complet.

Code :

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

1
2
3
4
5
6
class Li_License
{
    private int $id
    private string $date
    private string $email // J'ai mis email , mais ca pourrait être n'importe quel attribut utile représentant owner
}

Soit tu peux avoir besoin de plusieurs informations , auquel cas il est surement plus judicieux de lié l'objet user à chaque licence.

Code :

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

1
2
3
4
5
6
class Li_License
{
    private int $id
    private string $date
    private US_User $owner
}

Et dans ce cas précis il sera possible (mais pas obligatoire) de mettre en place des optimisation pour ne charger les données annexes que quand elle sont nécessaires.

[laurentSc]

Je suis encore sur cette question. En effet, dans mes classes du modèle (architecture MVC), j'ai une méthode seekId qui retourne un booléen ; true si l'objet est déjà présent en bdd et false sinon. Etant donné cela, si une classe PHP unique correspond à une seule table SQL, et si une seule propriété de la classe PHP permet d'identifier l'objet, la méthode seekId est simple à concevoir : il suffit de tester si cette propriété est déjà en bdd.
Par contre, si la classe PHP a certaines propriétés dans une table SQL et d'autres dans une autre table, alors je ne vois pas comment concevoir la méthode seekId...

[laurentSc]

Pour préciser ma question, voici un cas concret :

d'abord, j'ai une architecture SQL modélisée par ce MCD :

avec les 2 tables suivantes :

Code sql :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
CREATE TABLE CO_company(
   CO_ident INT UNSIGNED AUTO_INCREMENT,
   CO_company VARCHAR(30) NOT NULL,
   PRIMARY KEY(CO_ident),
   UNIQUE(CO_company)
);
 
CREATE TABLE BU_businessunit(
   BU_ident INT UNSIGNED AUTO_INCREMENT,
   BU_buname VARCHAR(30) NOT NULL,
   CO_ident INT UNSIGNED NOT NULL,
   PRIMARY KEY(BU_ident),
   UNIQUE(BU_buname),
   FOREIGN KEY(CO_ident) REFERENCES CO_company(CO_ident)
);

selon la remarque de grunk au post #2, je pense qu'il faut créer une seule classe PHP CO_company avec les 2 propriétés (CO_company et BU_buname). Mais du coup, comment concevoir la méthode seekId ? Si par exemple CO_company est en bdd mais pas BU_buname, doit-elle retourner true ou false ?