mauvais resultats affichés sql

**kate59** · 06/07/2010, 16h06

bonjour,
voila j ai creer une class categories avec cette fonction qui permet de recuperer les elements de la table categories de ma base:

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
public function getById()
	{
 
		$req="SELECT * FROM achatsenfolie__categorie where id_categorie is NULL";
		//echo $req
		$reponse = mysql_query($req)or die ('Erreur : '.mysql_error());
 
		while($donnees=mysql_fetch_array($reponse))
		{
 
return $this->id=$donnees['id'];
						$this->id=$donnees['id'];
                        $this->titre=$donnees['titre'];
                        $this->description=$donnees['description'];
                        $this->estDansMenu=$donnees['estDansMenu'];
                        $this->estDansContact=$donnees['estDansContact'];
                        $this->estVisible=$donnees['estVisible'];
                        $this->Categorie=$donnees['id_categorie'];
               }
	}

voci l appel:

Code :

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

1
2
3
4
$objet = new categorie;
echo (' <li class="odd">');
echo $objet->getById();
echo ('</li>');

donc je fais un test et ca me retourne bien un numero mais le 1.
Mais normalement je devrait en avoir bc plus, j ai tester la requete sur la base directement et ai bien apparut plusieurs resultats, alors pourquoi lui n affiche que le 1 resultats je ne comprend pas.
Merci

j ai tester en mettant echo a la place de return dans la fonction et ca a fonctionner pourquoi?
est ce correcte?
merci.

**sabotage** · 06/07/2010, 22h52

return termine la fonction, il doit être appellé en dernier.

**RunCodePhp** · 07/07/2010, 07h04

Salut

Code :

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

1
2
3
4
while($donnees=mysql_fetch_array($reponse)) {	
    return $this->id=$donnees['id'];
    ... etc ...
}

Mise à part le problème du return, ici tu ré-affecte (ou écrase) chaque propriété ($this->id par exp.) à chaque tour de boucle, se qui fait que ce les valeurs seront obligatoirement celle de la dernière ligne.

Si tu veux que ça retourne toutes les lignes (plusieurs lignes et non 1 seul), et bien il faudrait les stocker dans un tableau, et à la fin le retourner.
Exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public function getById() {
    $donnees = false;
    ... etc ...
    while($donnee = mysql_fetch_array($reponse)) {	
        $donnees[] = array('id' => $donnees['id'],
                           'titre' => $donnee['titre'],
                           'description' => $donnees['description'],
                           ... etc ...
                           'Categorie' => $donnees['id_categorie']);
    }
    return $donnees;
}

Sinon, si tu veux récupérer les valeurs des champs d'1 seule ligne, alors faire une boucle ne sert à rien.
De plus, si tu compte réellement ré-affecter les valeurs des proprités (id, titre, etc ...), alors un return ne sert à rien aussi.

Faudrait voir ce que tu souhaite faire, car à l'origine il y a ceci : WHERE id_categorie is NULL
C'est quand même bizarre de vouloir récupérer des identifiants nulles. Théoriquement un champ de type ID ne peu pas (ou ne doit pas) être nulle, vu que ça devrait être une clé primaire ou clé étrangère.

**kate59** · 07/07/2010, 10h34

bonjour,
voila comme je disais je debute en php5, en faite je voulais afficher un menu dont les titre serait afficher grace a la table sql.Je ne savais pas trop par ou commencer donc j ai créer cette fonction, merci encore pour le tableau mais je n ai tj rien qui s'affiche:

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
	public function getById() {
    $donnees = false;
    $reponse = mysql_query("SELECT * FROM ".self::TB_PREFIX."categorie")or die ('Erreur : '.mysql_error());
    while($donnees = mysql_fetch_array($reponse)) {	
		$donnees[] = array(
							'id' => $donnees['id'],
                           'titre' => $donnees['titre'],
                           'description' => $donnees['description'],
                           'estDansMenu' => $donnees['estDansMenu'],
							'estDansContact'=> $donnees['estDansContact'],
						'estVisible'=> $donnees['estVisible'],
 
                           'Categorie' => $donnees['id_categorie'],
						   );
    }
    return $donnees;
}

si vous avez d'autre solution a part mes fonction a moi je suis preneuse, car je pense aussi que ma fonction n est pas le meilleur moyen au debut je voulais utiliser le getTitre en recuperant avant l id mais je n'ai pas reussit, de plus je ne sais pas si c est la meilleur solution.

Code :

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

1
2
3
public function getTitre() {
		return (string) $this->titre;
	}

merci de tout l aide que vous pourrez m apporter.

**RunCodePhp** · 07/07/2010, 12h47

En faisant du simple copier/coller j'ai commis quelques erreurs, donc je corrige en changeant les noms des variables/tableaux, comme ça plus de confusions :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public function getById() {
    $donnees = false;
    ... etc ...
    while($categorie = mysql_fetch_array($reponse)) {	
        $donnees[] = array('id' => $categorie['id'],
                           'titre' => $categorie['titre'],
                           'description' => $categorie['description'],
                           ... etc ...
                           'Categorie' => $categorie['id_categorie']);
    }
    return $donnees;
}

Ce n'était pas très visible entre $donnee (sans s) et $donnees.

Ceci dit, te ton coté faut aussi éviter de faire du copier/coller comme ça, mais adapter un code en respectant la logique.

Le mysql_fetch_array() renvoie un tableau, les valeurs des champs, mais pour chaque ligne (et non toutes) à chaque tour de boucle, donc ici $categorie (j'avais mis $donnee sans s).
Ensuite, comme le but ici c'est de stocker toutes les lignes, il faut donc créer un 2ème tableau, $donnes (cette fois avec un s), et c'est celui ci qui doit être retourné.
Faut pas que ça soit le même tableau quoi

**kate59** · 07/07/2010, 15h13

merci de ton aide, j ai bien appliquer ce que tu a ecrit mais je n obtient que Array en affichage

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
public function getById() {
    $donnees = false;
    $reponse = mysql_query("SELECT * FROM ".self::TB_PREFIX."categorie")or die ('Erreur : '.mysql_error());
    while($categorie = mysql_fetch_array($reponse)) {	
        $donnees[] = array('id' => $categorie['id'],
                           'titre' => $categorie['titre'],
                           'description' => $categorie['description'],
                          'estDansMenu' => $categorie['estDansMenu'],
							'estDansContact'=>$categorie['estDansContact'],
						'estVisible'=> $categorie['estVisible'],
                           'Categorie' => $categorie['id_categorie']);
    }
    return $donnees;
 
}

j ai fait une recherche et de cette erreur:

Comme php ne savait pas quoi afficher, il a choisi de te prévenir que c'était un tableau en affichant 'Array' ('tableau' en anglais).

mais il me donne pas de reponse.
merci.

**sabotage** · 07/07/2010, 19h10

getAllCategorie() retourne un tableau.
un echo d'un tableau donne la chaine "array"
Si tu veux voir ca que contient le tableau, tu peux utiliser print_r()

**kate59** · 09/07/2010, 15h42

merci sabotage, je sais que le print peut afficher mais ce n est pas cela que je veux en faite je veux afficher le titre de categorie pour le mettre en menu.
Je penser utiliser le getTitre() mais je n ai pas reussi.
peut tu m aider?
MERCI

**sabotage** · 09/07/2010, 15h47

Il faudrait alors constuire ton tableau en l'indexant par id.

Pour obtenir le nom d'une catégorie tu auras juste à utiliser $ton_tableau[$id_que_tu_veux]['titre']

**kate59** · 09/07/2010, 16h26

merci sabotage mais en faite ce n esp pas une catgeorie que je fait afficher c est tous les titres des categories.

j ai commerncer l indexation mais cela fait super lontemps que je l ai pas fait je sais plut exactement comment on fait l indexation.

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
public function getAllCategorie() {
    $donnees = false;
    $reponse = mysql_query("SELECT * FROM ".self::TB_PREFIX."categorie")or die ('Erreur : '.mysql_error());
    while($categorie = mysql_fetch_array($reponse)) 
	{	
			//ligne du tableau indexée par un index unique (champ id)
 
        $donnees[] = array('id' => $categorie['id'],
                           'titre' => $categorie['id']['titre'],
                           'description' => $categorie['id']['description'],
                          'estDansMenu' => $categorie['id']['estDansMenu'],
							'estDansContact'=>$categorie['id']['estDansContact'],
						'estVisible'=> $categorie['id']['estVisible'],
                 'Categorie' => $categorie['id']['id_categorie']);
 
 
    }
 
  return($donnees);
 
}

de plus pense tu que c est le meilleur moyen pour afficher le titre des categorie ce genre de function.
merci de tes conseils.

**sabotage** · 09/07/2010, 17h14

Dans tous les cas, indexer en utilisant l'id est une bonne idée :

Code :

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

     $donnees[$categorie['id']] = array('titre' => $categorie['id']['titre'], ....

Pour afficher ce que tu as recolté, tu as juste a parcourir le retour de ton getAllCategorie() avec un foreach()

mauvais resultats affichés sql

PHP & Base de données

Discussions similaires

Partager

Partager