Objet PHP et Nombre attribut variable

Version imprimable

Bonjour,

Je butte sur un problème en php orienté objet, je vous explique:

Mon objet => fiche, a comme attribue :

Code:

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
function fiche($Obj_FI='', $Des_FI='', $Duree_FI='', $Local_FI='', $Cont_FI='', $Situa_FI='', $Act_FI='', $Res_Q1_FI='', $Res_Q2_FI='', $Valid_FI='', $Lieu_FI='', $Saisie_FI='', $ID_COMP='', $ID_PRO='', $ID_PROC='',$Lib_PROC='',$Co_PROC='',$ID_DO ='',$Lib_DO='',$Co_DO ='',$ID_ACT='',$Lib_ACT='',$Co_ACT='',$ID_COMP='',$Lib_COMP='',$Co_COMP='',$ID_PRO='',$Lib_PRO='',$Des_PRO='')
    {
        $this->Obj_FI = $Obj_FI;
        $this->Des_FI = $Des_FI;
        $this->Duree_FI = $Duree_FI;
        $this->Local_FI = $Local_FI;
        $this->Cont_FI = $Cont_FI;
        $this->Situa_FI = $Situa_FI;
        $this->Act_FI = $Act_FI;
        $this->Res_Q1_FI = $Res_Q1_FI;
        $this->Res_Q2_FI = $Res_Q2_FI;
        $this->Valid_FI = $Valid_FI;
        $this->Lieu_FI = $Lieu_FI;
        $this->Saisie_FI = $Saisie_FI;
        $this->ID_PROC = $ID_PROC;
        $this->Lib_PROC = $Lib_PROC;
        $this->Co_PROC = $Co_PROC;
        $this->ID_DO = $ID_DO;
        $this->Lib_DO = $Lib_DO;
        $this->Co_DO = $Co_DO;
        $this->ID_ACT = $ID_ACT;
        $this->Lib_ACT = $Lib_ACT;
        $this->Co_ACT = $Co_ACT;
        $this->ID_COMP = $ID_COMP;
        $this->Lib_COMP = $Lib_COMP;
        $this->Co_COMP = $Co_COMP;
        $this->ID_PRO = $ID_PRO;
        $this->Lib_PRO = $Lib_PRO;
        $this->Des_PRO = $Des_PRO;
}

Mais une fiche peut avoir un ou plusieurs supports (5 max) (composé d'un ID, d'un libéllé, d'une nature et d'un origine).

de ce fait, Dois-je créer les 5 supports directement dans l'objet fiche ? (Donc 20 attributs en plus)

Comme ceci :

Code:

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
function fiche($Obj_FI='', $Des_FI='', $Duree_FI='', $Local_FI='', $Cont_FI='', $Situa_FI='', $Act_FI='', $Res_Q1_FI='', $Res_Q2_FI='', $Valid_FI='', $Lieu_FI='', $Saisie_FI='', $ID_COMP='', $ID_PRO='', $ID_PROC='',$Lib_PROC='',$Co_PROC='',$ID_DO ='',$Lib_DO='',$Co_DO ='',$ID_ACT='',$Lib_ACT='',$Co_ACT='',$ID_COMP='',$Lib_COMP='',$Co_COMP='',$ID_PRO='',$Lib_PRO='',$Des_PRO='')
{
    $this->Obj_FI = $Obj_FI;
    $this->Des_FI = $Des_FI;
    $this->Duree_FI = $Duree_FI;
    $this->Local_FI = $Local_FI;
    $this->Cont_FI = $Cont_FI;
    $this->Situa_FI = $Situa_FI;
    $this->Act_FI = $Act_FI;
    $this->Res_Q1_FI = $Res_Q1_FI;
    $this->Res_Q2_FI = $Res_Q2_FI;
    $this->Valid_FI = $Valid_FI;
    $this->Lieu_FI = $Lieu_FI;
    $this->Saisie_FI = $Saisie_FI;
    $this->ID_PROC = $ID_PROC;
    $this->Lib_PROC = $Lib_PROC;
    $this->Co_PROC = $Co_PROC;
    $this->ID_DO = $ID_DO;
    $this->Lib_DO = $Lib_DO;
    $this->Co_DO = $Co_DO;
    $this->ID_ACT = $ID_ACT;
    $this->Lib_ACT = $Lib_ACT;
    $this->Co_ACT = $Co_ACT;
    $this->ID_COMP = $ID_COMP;
    $this->Lib_COMP = $Lib_COMP;
    $this->Co_COMP = $Co_COMP;
    $this->ID_PRO = $ID_PRO;
    $this->Lib_PRO = $Lib_PRO;
    $this->Des_PRO = $Des_PRO;
    $this->ID_SUPP_0 = $ID_SUPP_0;
    $this->Nom_SUPP_0= $Nom_SUPP_0;
    $this->Org_SUPP_0= $Org_SUPP_0;
    $this->Nat_SUPP_0= $Nat_SUPP_0;
    $this->ID_SUPP_1 = $ID_SUPP_1;
    $this->Nom_SUPP_1= $Nom_SUPP_1;
    $this->Org_SUPP_1= $Org_SUPP_1;
    $this->Nat_SUPP_1= $Nat_SUPP_1;
    $this->ID_SUPP_2 = $ID_SUPP_2;
    $this->Nom_SUPP_2= $Nom_SUPP_2;
    $this->Org_SUPP_2= $Org_SUPP_2;
    $this->Nat_SUPP_2= $Nat_SUPP_2;
    $this->ID_SUPP_3 = $ID_SUPP_3;
    $this->Nom_SUPP_3= $Nom_SUPP_3;
    $this->Org_SUPP_3= $Org_SUPP_3;
    $this->Nat_SUPP_3= $Nat_SUPP_3;
    $this->ID_SUPP_4 = $ID_SUPP_4;
    $this->Nom_SUPP_4= $Nom_SUPP_4;
    $this->Org_SUPP_4= $Org_SUPP_4;
    $this->Nat_SUPP_4= $Nat_SUPP_4;
}

Ou faire une requête sql et compter le nombre de support pour la fiche en question ?

Il faut savoir aussi que cette objet me permet de modifier ma fiche donc de pouvoir enregistrer 1 à 5 support.

Je sais pas trop comment ou quoi faire.

Merci d'avance de votre aide.

(N'hésiter pas a me demandé des partie de code.)

27/02/2014, 15h55
rawsrc

Salut,

A la vue de ton code, cela serait bien si tu pouvais poster le code complet de ta classe Fiche
27/02/2014, 15h56
val373

1 pièce(s) jointe(s)

Voila ;)
27/02/2014, 16h29
rawsrc
Houlà, c'est space...

Allez rapidement :
- Toutes les propriétés de ta classe sont public : quelle que soit l"instance on peut tout modifier sans aucun contrôle ^^
- Qu'un constructeur porte le nom de la classe dans laquelle il est défini peut s'avérer très problématique : regarde ici
- Avoir toutes les propriétés public et se taper un prototype de constructeur avec la liste des 30 variables, c'est inutile
- Quand tu as autant de variables, tu y gagne à utiliser un tableau en guise de paramètre
- Tout étant public, on peut même s'amuser à changer les types de données en base : $pog_attribute_type
- Généralement, on n'a pas à "dé-échapper" les données de la base à la lecture...
- T'es sûr d'avoir besoin d'utiliser utf8_encode() ? Si t'es homogène sur toute la ligne, tu n'as pas à te préoccuper des encodages
- Tu devrais nommer plus explicitement tes variables, parce que quand tu y reviendras dans 10 mois, tu te maudiras
J'arrête là,

Euh, c'est étrange :
Code:

1 2 $thisObjectName = get_class($this); $fiche = new $thisObjectName();
Tu peux remplacer par :

Code:

$fiche = new self();

Donc pour répondre à ta question, je te dirais qu'il faut passer par un tableau de gestion des supports.
Mais bon, faudrait d'abord reprendre entièrement ce code.
28/02/2014, 09h10
val373

La base de se code vient de POG (Php object generator) où j'ai modifier quelque petite chose.

je relis un cours de php objet et je modifie tous ça,

pour le (utf8_encode), s'il y en a pas, les 4 libellés sont Null.
28/02/2014, 10h27
val373

1 pièce(s) jointe(s)

Je suis arrivé à ça :

Pièce jointe 139223

Cela me retourne le même résultat qu'avant mais avec 2 fois moins de ligne ;) , Mais si je les met en 'private' cela ne marche plus !
et je ne vois pas pourquoi
28/02/2014, 10h49
rawsrc

Je suis allé voir à tout hasard ce projet open source POG.

Je ne te cache pas que j'ai chopé une hémorragie oculaire !

Échapper des valeurs en base avec base64_encode(), c'est de l'art moderne ou je ne m'y connais pas !!

Utiliser encore mysql_real_escape_string() qui est obsolète, c'est ballot.
Et faire des stripcslashes() alors qu'il n'y a nulle part son pendant : addcslashes(), c'est comment dire : complètement inutile.

Un conseil, laisse tomber cette librairie POG, c'est du grand n'importe quoi.
Comme 99% des générateurs de code, le résultat est pourri.

Prends toi par la main, siphonne les tutos sur DVP, mène tes recherches, pose des questions mais par pitié, jette moi de ce torchon.
28/02/2014, 13h51
val373

Ok bah je vais faire ça alors

Merci du conseil ;)

Après avoir lu plusieurs tuto, et fais pas mal de recherche j'ai réussis (du moins je pense ^^)

j'ai un doute avec :
Code:

1 2 3 4 5 6 $req ="SELECT ID_SUPP FROM supp_fi WHERE ficheid= ".$ficheId; $data = Database::Requete($req); while ($row = Database::Lecture($data)) { $this->Supp[]= new Support($row->ID_SUPP); }
Entre les crochet de Supp, je n'ai pas à incrémenter ?

Il y a aussi le faite que si dans les info que je reçois il y a des accents, je dois mettre utf8_encode, il y t'il un moyen d'éviter cela ?

PS :
Code:

1 2 3 4 5 6 7 8 9 public function Lecture($cursor) { return mysql_fetch_object($cursor); } public function Requete($query) { $query = mysql_query($query); return $query; }

28/02/2014, 16h34
Spartacusply

Citation:

Entre les crochet de Supp, je n'ai pas à incrémenter ?

Non pas besoin d'incrémenter, les crochets signifient "ajout à la fin du tableau"

Citation:

Il y a aussi le faite que si dans les info que je reçois il y a des accents, je dois mettre utf8_encode, il y t'il un moyen d'éviter cela ?

Normalement, si tu es carré sur ton encodage du début à la fin, tu n'as pas besoin d'utiliser des fonctions comme celle-ci. Je t'invite à regarder ce tuto, tu ne devrais plus avoir de problème après l'avoir lu (et compris ;))

Merci de ta réponse,

Je vais lire ce tuto ;)

Par contre j'ai un problème pour initialiser mes attributs je voulais mettre un paramètre optionnel :
Code:

1 2 3 4 5 6 7 8 9 10 public function __construct($ficheId = '0' ){ if ($ficheId == '0') { $this->Obj_FI=''; $this->Des_FI=''; } else{ $req = "select * from `fiche` where `ficheid`=".$ficheId."LIMIT 1"; $data = Database::Requete($req); .... }
Mais cela ne marche pas

28/02/2014, 17h11
Spartacusply

Qu'est ce qui ne marche pas ? Message d'erreur ou tu rentres toujours dans le "else" ?

Ton code devrait marcher si tu fait juste un :

Code:

$fiche= new Fiche(); //Pas de paramètre

Au passage, privilégie la valeur NULL à '0' (qui peut-être tendancieuse) et test avec isset() pour voir si ta variable est définie ou non.