[AJAX] Listes déroulantes synchronisées sur bdd

**obito** · 12/03/2009, 10h23

ha de base les select * c'est pas bien du tout

ca prends de la ressource pour rien et ca augmente le temps de traitement le mieux serait déjà de sélectionner juste le champ à afficher, ca ira déjà plus vite lors de l'affichage

Après faut voir déjà en exécutant la requête sur phpMyAdmin pour voir les résultats

^^

**E.Bzz** · 12/03/2009, 10h27

Envoyé par obito

Après faut voir déjà en exécutant la requête sur phpMyAdmin pour voir les résultats

^^

Par exemple si la table n'est pas vide

**Hefgé** · 12/03/2009, 11h14

Vous voulez voir mes tables? J'vous fait un imprim écran... Mais pour moi la requète est juste, j'l'ai faite fonctionner sous phpmyadmin, oui je sais c'est un peu de la programmation pas sérieuse que je fais, même dans mes tables j'ai triché, pas le gout de faire un MCD et tout...

J'ai jamais fait d'ajax donc forcémment je peux pas non plus savoir ce qui est bien ou pas...

**obito** · 12/03/2009, 11h18

La requete marche donc, donc pour moi c'est bien le paramètre qui ne va pas quand tu le récupère il faudrait l'afficher. Comme je t'ai proposé à l'appel de la fonction AJAX au début de la fonction tu fait un alert pour afficher ce que tu récupère =)

**Hefgé** · 12/03/2009, 11h22

Par contre j'ai 2 num_typebr avec ma requète, est-ce que ça peut être ça le problème?

**obito** · 12/03/2009, 11h23

C'est mieux de préfixé avec la table en effet

Ce qui implique donc de ne pas utiliser de select * mais de sélectionner les champs que tu va vraiment utiliser

**Hefgé** · 12/03/2009, 11h31

C'est fait, mais ça ne change rien...
J'ai remplacé ma requète par...

Code :

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

SELECT type_brush.num_typebr, type_brush.lib_typebr FROM type_brush,type_int WHERE type_int.num_typebr=type_brush.num_typebr and type_int.num_type ='".$catid."'

J'vais voir cette histoire de paramètre...

**E.Bzz** · 12/03/2009, 11h40

Envoyé par obito

Ce qui implique donc de ne pas utiliser de select * mais de sélectionner les champs que tu va vraiment utiliser

+1 en donnant des alias différents aux champs ayant le même nom.

Sinon, tu es sûr que les quottes posent pas problème pour un champ apparemment numérique, là ?

Code :

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

and type_int.num_type = '".$catid."'";

A+

**obito** · 12/03/2009, 11h43

Envoyé par E.Bzz

+1 en donnant des alias différents aux champs ayant le même nom.

Sinon, tu es sûr que les quottes posent pas problème pour un champ apparemment numérique, là ?

Code :

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

and type_int.num_type = '".$catid."'";

A+

+1 (

)

Oui déjà ca, et puis faut vraiment voir ce que vaut $catid si c'est bien un numérique ou une chaine (si c'est une chaine c'est pas bon du tout ^^)

**Hefgé** · 12/03/2009, 11h47

Bah j'sais pas c'était comme ça dans le tuto et il me semble qu'ils avaient un champ numérique nan?
Ouais, normalement c'est bon, ils avaient bien une valeur numérique, d'ailleurs ils utilisaient le value et pas le selectedindex

**E.Bzz** · 12/03/2009, 11h49

Envoyé par Hefgé

Bah j'sais pas c'était comme ça dans le tuto et il me semble qu'ils avaient un champ numérique nan?

Un tuto est un tuto !

Les pages de tes applis n'affichent pas que des "Hello world !" ?

C'est donc qu'à partir d'un tuto, il faut faire des adaptations spécifiques à ton cas ...

A+

**obito** · 12/03/2009, 11h49

J'ai pas regarder le tuto mais un this.value pour une liste déroulante te retourne la chaine selectionnée pas l'id =) Essaie donc this.SelectedIndex comme je t'avais proposé

**E.Bzz** · 12/03/2009, 11h53

Envoyé par obito

J'ai pas regarder le tuto mais un this.value pour une liste déroulante te retourne la chaine selectionnée pas l'id =)

Retourne la value de l'option sélectionnée (les value sont toujours de type texte) ...

**obito** · 12/03/2009, 11h56

oui c'est ce que je dis (apparemment mal^^) par exemple dans ma liste j'ai "cuisine" le this.value me renverra cuisine et aucun id...

Pour ça que je propose le selectedIndex (qui n'est pas vraiment la meilleure solution mais la plus adaptée à son script par contre...)

**Hefgé** · 12/03/2009, 12h01

Bon, j'ai mis le selectedIndex...
Et dans la requète:

Code :

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

and type_int.num_type =".$catid."";

Mais toujours rien...

**obito** · 12/03/2009, 12h03

Mais comme je t'ai dit fait un alert au début de l'appel de la fonction pour voir ce qui est récuperer...

**E.Bzz** · 12/03/2009, 12h04

Envoyé par obito

oui c'est ce que je dis (apparemment mal^^) par exemple dans ma liste j'ai "cuisine" le this.value me renverra cuisine et aucun id...

Bon, on va faire autrement

Code :

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

<option value="id01">Cuisine</option>

La value est de type string et doit correspondre à ton "catid" dans ta table.
Le texte de l'option n'a généralement aucun intérêt technique.
Le selectIndex n'est que le n° d'ordre de l'option dans cette liste là. Il y a à peu près 0,0000001 % de chance qu'il corresponde au "catid"
Un select ne renvoie jamais aucun id (au sens JS/HTML).

Merci d'être plus précis dans les termes employés, sinon, on avance pas

A+

**Hefgé** · 12/03/2009, 12h07

Mais comme je t'ai dit fait un alert au début de l'appel de la fonction pour voir ce qui est récuperer...

J'veux bien mais je suis pas un pro comme vous en ajax et je sais pas trop comment m'y prendre...

**obito** · 12/03/2009, 12h07

Ouais autant pour moi je me suis mal exprimé (pour la cuisine je supposé aussi que le value était = à cuisine

parce que dans son code je crois que ca marche pareil, il a un value = au texte de sa liste déroulante...

Pour la probabilité 100% d'accord la on a une infime chance d'avoir le bon id ou alors faire dans la requete qui va remplir la liste un ORDER BY sur l'id =)

EDIT : Sauf que problème, on a une ligne vide dans la liste, donc a gérer ca...