Discussion :

[kate59]

bonjour,
je suis complétement déséperer cela fait une semaine que j 'essaye de comprend un peu mieux le systeme php5. Voila je veux creer un site e-commerce donc j ai creer la base, ensuites les classes de cette base.

Maintenant j ai l'index.php de la base ou je voudrait afficher les nouveute donc j ai créer une fonction dans la class produit:

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
public function afficherLesNouveautes(){
$req ="SELECT * from ".self::TB_PREFIX."produit
JOIN ".self::TB_PREFIX."illustration_produit ON ".self::TB_PREFIX."illustration_produit.id = ".self::TB_PREFIX."produit.id_illustration
WHERE ".self::TB_PREFIX."produit.afficher_dans_nouveautes=1";
$res = mysql_query($req) or die ('Erreur : '.mysql_error());
$ret = array();
 
while( $data = mysql_fetch_object($res) ) {
$ret[] = new Produit (
$data->id,
$data->nom,
$data->description,
$data->prix,
$data->gamme,
$data->date_modification,
$data->date_creation,
$data->afficher_dans_nouveautes, //ICI il faut dire OUI.Par exemple. Mais je connais pas tout le contexte
$data->est_visible,
$data->est_disponible,
$data->id_categorie,
$data->id_illustration
);
}
 
if ( !empty($ret) )
return $ret;
 
return null;
}

et mit ca sur l index.php:

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
<?php
/*appelle de la class produit*/
$objet=new produit;
 
/*utilisation fonction pour afficher nouveautés*/
$Nouveautes=$objet->afficherLesNouveautes();
 
if (is_array($Nouveautes)) {
 
//$body.='<div class="Nouveaute_center">';
foreach( $Nouveautes as $value ) {
 
/*construction url dynamique*/
$body.='<ul><li><a>';
 
$IdIllustrations=$value->getIDillustrations().'';
 
$body.=''.$value->getNom().'';
 
$body.='</a></li></ul>';
......
?>

mais on me dit que ce n est pas dut tout la solution qu'il faut.

Comment je doit m 'y prendre?

J'aimerais tellement qqun m'eclaire avant ce soir ou je perd ma tête.

[kate59]

j ai chercher sur le web mais pas beaucoup de réponse je déséspere.

[patouche]

Maintenant j ai l'index.php de la base ou je voudrait afficher les nouveute donc j ai créer une fonction dans la class produit:

Pardon d'être un peu "titilleur" mais dans une classe, on ne crée pas des fonctions mais des méthodes. Ensuite, je suis pas sûr que ta méthode afficherLesNouveautes soit placer au bon endroit, ni qu'elle affiche les nouveautés.

Si tu veux garder ta méthodes dans la classes produit (même si amha elle serait mieux placer dans une classe Catalogue), il faut passer par une méthode statique.

En effet, ce que tu fais ici n'a pas trop de sens. Tu crées une nouvelle instance de produit pour récupérer plusieurs produit Réfléchis un peu et pense catalogue / produit au niveau de tes objets


Cordialement,
Patouche

[kate59]

excuse moi je suis débutante php5 donc j'ai du mal a comprendre certaine chose. Pour le site e-commerce j'ai créer une table sql ensuite on m'a dit de créer une class pour chaque table ce que j ai fait donc ma class produit:

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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
<?php
 
/*
 * Produit
 * --------------------------------
 *
 */
class Produit  {
 
	const TB_PREFIX = 'achatsenfolie__';
    /**
     * Nom de la table utilisée
     * @var String
     */
    const TB_NAME = 'PRODUIT';
 
 
    /**
     * Numéro d'identifiant
     * @access private
     * @var Integer
     */
    private $id = 0;
 
    /**
     * Nom
     * @access private
     * @var String
     */
    private $nom = null;
 
    /**
     * Description
     * @access private
     * @var String
     */
    private $description = null;
 
    /**
     * Prix
     * @access private
     * @var Float
     */
    private $prix = 0.0;
 
    /**
     * Gamme (basse, moyenne, haute)
     * @access private
     * @var String
     */
    private $gamme = 'moyenne';
 
    /**
     * Date de dernière modification
     * @access private
     * @var Integer
     */
    private $date_modification = 0;
 
    /**
     * Date de création
     * @access private
     * @var Integer
     */
    private $date_creation = 0;
 
    /**
     * S'affiche dans la liste des nouveautés
     * @access private
     * @var Boolean
     */
    private $afficher_Dans_Nouveautes = false;
 
 
 
 
 
    /**
     * Est encore disponible
     * @access private
     * @var Boolean
     */
    private $est_visible = true;
    /**
     * Est encore disponible
     * @access private
     * @var Boolean
     */
    private $est_disponible = true;
 
 
    /**
     * Categorie dans laquelle est affiché le produit
     * @var Categorie
     */
    private $id_categorie = null;
	    /**
     * illsutration est affiché du produit
     * @var Categorie
     */
    private $id_illustration = null;
 
	//private $_nouveautes = array();
 
 /**
     * Constructor
     * @access protected
     */
     public function __construct($id=0,$nom = "sans titre",$description = null,$prix=0, $gamme= null, $date_modification=0,$date_creation = 0,
	$afficher_Dans_Nouveautes=false,$est_visible=1,$est_disponible=1,$id_categorie=NULL,$id_illustration=NULL ) {
		 $this->id = $id;
         $this->nom = $nom;
         $this->description = $description;
		 $this->prix = $prix;
         $this->gamme = $gamme;
		 $this->date_modification = $date_modification;
         $this->date_creation = $date_creation;
 
         $this->afficher_Dans_Nouveautes = $afficher_Dans_Nouveautes;
		 $this->est_visible = $est_disponible;
         $this->est_disponible = $est_disponible;
		 $this->id_categorie = $id_categorie;
		$this->id_illustration = $id_illustration;
     }
 
 
 
 
 /*Recupere informations tous les sous categories par id*/
	public  function findProduitById($id){
		$req ="SELECT * from ".self::TB_PREFIX."produit where id_categorie=".$id."";
		$res = mysql_query($req) or die ('Erreur : '.mysql_error());
		$ret = array();
 
		while( $data = mysql_fetch_object($res) ) {
			$ret[] = new Produit( 
			$data->id,
			$data->nom,
			$data->description,
			$data->prix,
			$data->gamme,
			$data->date_modification,
			$data->date_creation,
			$data->afficher_dans_nouveautes,
			$data->est_visible,
			$data->est_disponible,
			$data->id_categorie,
			$data->id_illustration
 
			);
		}
 
		if ( !empty($ret) )
		return $ret;
 
		return null;
	}
 
 
 
 
 
    /**
     * Met le numéro d'identifiant
     *
     * @access protected
     * @param integer $id
     * @return void
     */
    protected function setId($id) {
        if (is_int($id) && $id>0) {
            $this->id = $id;
        } else {
            throw new InvalidArgumentException('Numéro d\'identifiant incorrect');
        }
        return $this;
    }
 
 
.....
 
 
?>

Ensuite je voulais afficher les nouveaute donc j ai fait la methode afficherLesNouveaute() pour ma page accueil.php. Mais si tu a une autre methode j' ecoute avec attention.

[patouche]

Il serait mieux de faire une classe Catalogue (ou catégorie) comme suit :

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
class Catalogue {
 
    public function __construct() {
 
    }
 
    /**
     * @return array de Produit
     */
    public function getNewProduct () {
        //Ta requete sql ici et tu retourne un tableau
    }
 
}

Enfin, ça me parait plus logique....

Cordialement,
Patouche

[RunCodePhp]

Salut

Peut être que ce qui t'ont dit que ce n'était pas correcte n'ont peut être pas tord, car j'y vois un truc pas zen.

Au départ tu as un objet Produit, et ce produit contient une méthode afficherLesNouveautes().
Petite parenthèse déjà, c'est que son nom ne reflète pas la réalité, car il n'y a pas d'affichage, cette méthode prévois de retourner un tableau.
Par convention d'écriture (ou de règle de nommage), on ferait : getNouveautes().
(c'était une petite parenthèse)

Le soucis, c'est que tu prévois de retourner un tableau, mais que chaque élément contient un objet Produit.
Et bien là, c'est quand même pas banal, car un objet se contenant lui même, ça risque de faire désordre.

En finalité, ça veut dire qu'on pourrait à nouveau appeler la méthode afficherLesNouveautes() pour chaque élément du tableau.
Donc faire ceci :

Code :

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

1
2
3
4
5
6
7
8
 
$objet = new produit;
 
/*utilisation fonction pour afficher nouveautés*/
$Nouveautes = $objet->afficherLesNouveautes();
 
//
$Nouveautes[0]->afficherLesNouveautes();

La solution évoquée par Patouche mes semble tout à fait correcte, du faite qu'il faudrait un autre objet qui serait un ensemble de produits (objet Produit).

Personnellement, je verrais une classe Produits (avec un S), qui par son nom évoque explicitement un objet contenant plusieurs produits, donc plusieurs objet Produit.
M'enfin, catalogue, produits, finalement, c'est pareil.