Discussion :

[maysa]

bonjour, je débute en POO et j'ai créé une classe qui doit me lister tous les noms, prenoms et adresses de ma base de données :

j'ai mon fichier data.class.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
23
24
25
26
27
28
29
30
31
32
33
 
class client
 
{
 
 var nom;
 var prenom;
 var adreses;
 
 function afficheClient()
 {
  $valide = " SELECT nom, prenom, adresse 
              FROM client "
 
 
  $res = $mysql_query($valide);
 
  while ($data = mysql_fetch_assoc($res))
  {
   $nomIntervenant = $data['nom'];
   $nomClient = $data['prenom];
   $numNoteFrais = $data['adersse'];
 
   $nom [] = $nom;
   $prenom[] = $prenom;
   $adresse[] = $adresse;
 
  }			   
return $nom[], $prenom[], $adresse []
 
 }
 
}

mais le problème c'est que je ne peux retourner qu'un seul élément ?
donc dans ma page data.php :
je fais un include data.class.php
et là je veux mettre mes données dans un tableau :

mais comment faire ?

Code :

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

1
2
3
 
$client = new client();
$client->afficheClient();

je suis perdu avec les objets !

Merci de votre aide...

[goodpz]

Une possibilité: retourner un array car on ne peut pas retourner plusieurs valeurs distinctes en php.

Code :

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

return array($nom, $prenom, $adresse);

[ratapapa]

A mon avis ta classe client est destinée à instancier des clients, et non des groupes de clients.

Or ta fonction présente dans ta classe liste un ensemble de clients.
Au niveau conceptuel c'est comme si tu demandais à un de tes clients de te citer tous les autres clients que tu possèdes : il n'est pas censé les connaitre.

Le deuxième point qui m'a interpellé c'est une fonction afficheClient qui retourne un résultat.
Il te faut faire un choix ici : ou bien la fonction affiche, ou bien elle retourne un résultat, dans ce cas ce n'est pas de l'affichage (enfin c'est un point de vue plus personnel ).

Personnellement je traiterais l'affichage client par client : la fonction afficheClient afficherait seulement ses informations (nom, prénom, adresse), et le tableau serait géré dans une meta classe (Clientele par exemple).
Tu peux aussi gérer l'affichage du tableau hors des classes.

[maysa]

merci,

et comment on gère l'affichage hors des classes ?

[ratapapa]

Simplement dans ton code php, qui donnerait quelque chose du style :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
$valide = " SELECT nom, prenom, adresse 
              FROM client "
$res = $mysql_query($valide);
 
echo "<table>";
 
  while ($data = mysql_fetch_assoc($res))
  {
      //Le constructeur du client est à construire
      $client = new Client($data);
      echo "<tr><td>".$client->afficheClient()."</td></tr>";
  }
echo "</table>";

[s.n.a.f.u]

Oui, ça le fait.
Mais tant qu'à utiliser la POO, autant pousser un peu : je ferais une collection 'LesClients' composée d'objets 'UnClient'.

J'avais galéré un peu à mettre en place des collections, mais maintenant je m'en sers à tout bout de champs : http://www.developpez.net/forums/sho...1&postcount=13

[maysa]

Simplement dans ton code php, qui donnerait quelque chose du style :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
$valide = " SELECT nom, prenom, adresse 
              FROM client "
$res = $mysql_query($valide);
 
echo "<table>";
 
  while ($data = mysql_fetch_assoc($res))
  {
      //Le constructeur du client est à construire
      $client = new Client($data);
      echo "<tr><td>".$client->afficheClient()."</td></tr>";
  }
echo "</table>";

et si on veut le nom dans un <td> le prénon aussi ect... c'est là ou je bloque avec le afficheClient();.....

[ratapapa]

Dans ce cas ou tu préformates ton affichage dans ta fonction (sale ) ou bien tu utilises de simples getter pour renvoyer les infos au goutte à goutte, que tu formattes ensuite.

Etant plutot puriste de l'objet je te conseille vivement de suivre le conseil de jml94 et d'utiliser une collection si tu es amenée à continuer à développer en objet.

Edit : les getter ne sont que des méthodes envoyant la valeur de tes attributs.
Exemple : function getNom() { return this->name; }