Discussion :

[renaud26]

Bonjour à tous,

Je veux utiliser un opérateur ternaire dans une boucle pour que mon select soit selected selon les données de la base. Il y a 4 valeurs possibles: header, footer, gauche et droite...Voici ce que j'ai fait:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
$a = 0;
$sell = "";
$posit = array("header", "footer", "droite", "gauche");//je déclare les 4 valeurs dans un tableau
 
while($row = mysql_fetch_array($sql))
{
 
$sell[$a] = ($posit[$a] == $row["posi"] ) ? ' selected="selected" ' : '';
 
echo "<tr><td><select name='posi[]'><option value='header' ".$sell[$a].">Header</option><option value='footer' ".$sell[$a].">Footer</option><option value='gauche' ".$sell[$a].">Gauche</option><option value='droite' ".$sell[$a].">Droite</option></select></td></tr>"
$a++;
}

Je n'ai ni notice, ni warning, mais l'affichage des selects est complètement fantaisiste...
Si quelqu'un voit l'erreur...je veux bien un tuyau ! D'avance merci.

[Amara]

Tu peux expliquer un peu parce que tu incrémentes $a lors du parcours des résultats ; du coup $a prend une valeur supérieure à 4 au bout de quatre tours de boucle, et ton tableau n'a que 4 éléments.

[renaud26]

d'abord merci de te pencher sur mon cas !
et oui il doit y avoir un loup à ce niveau...
Ma table n'a que 4 lignes pour le moment, mais quand elle en aura 10 ça va coincer parce que effectivement mon tableau n'aura toujours que 4 valeurs...
Je suis un peu perdu, là...

[Amara]

Essaie de nous expliquer un peu ce que tu souhaite faire.

Sinon essaie d'abord avec un if classique avant d'attaquer l'opérateur ternaire.

[renaud26]

dans une table nomée page, j'enregistre la mise en page. Comme ceci

Code :

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

1
2
3
4
5
6
7
 
id_mod  posi  ordre  
 
1     droite     1 
2     header   1 
3     gauche   1 
4     gauche   2

en mode modification, je lis cette table et j'affiche 4 select, un par module. le user doit pouvoir modifier la position et l'ordre de chaque module. Je voulais que les select soient selected en fonction de son choix à la création.

[Amara]

Je pense qu'il y a un problème dans ta condition elle-même. Affiche les valeurs que tu utilises pour ton test, tu verras si ça correspond à ce que tu veux.

[renaud26]

je viens d'afficher les valeurs de la table et celles du tableau, elles correspondent bien ! à s'arracher les cheveux. Je vais essayer avec un if classique...

[renaud26]

Je galère vraiment grave ! personne n'a une idée ou une piste ?

[ePoX]

moi il y'à un truc qui me choque direct dans ton code. Tu passe d'un string ($sell = "") à un tableau ($sell[$a]).

Tu peux deja regarder par la.

Pour le reste j'ai pas trop compris la manoeuvre.

[Yogui]

Salut

Je pense que tu compliques les choses pour rien. Par exemple le résultat de ton opérateur ternaire : tu ne l'utilisera pas en-dehors de la boucle, c'est donc une valeur temporaire au même titre que $row (tu peux les écraser sans complexe lors de chaque itération). À ce titre, je te suggère de prendre un nom de variable simple plutôt qu'un tableau : $selected me paraît relativement judicieux.

Ensuite, je te recommande de sortir de php (à l'aide de ?>) lorsque tu souhaites afficher des blocs de HTML, surtout si tu imbriques des balises dedans. Tu t'y retrouveras bien mieux, surtout si ton éditeur de code fait de la colorisation de HTML (si ce n'est pas le cas, tu peux éventuellement considérer de changer d'éditeur...).

Pour terminer, l'idée me semble globalement mauvaise. Un <select> ne devrait contenir qu'un seul element <option> qui soit "selected", or ton code en autorise visiblement plusieurs.
Typiquement, dans ce genre de cas, nous ne comparons pas 2 tableaux mais un tableau et ue valeur fixe. Pourquoi ? Parce que, en comparant 2 tableaux comme tu le fais, tu t'exposes à avoir éventuellement 2 "selected", ce qui est incorrect.

Code :

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

1
2
if ($posit[$a] == $row["posi"] ) // pas bon
if ($posit == $row["posi"] ) // ok si `posi` est un champ UNIQUE dans la BDD

[renaud26]

Bonjour, oui, je pense que vous avez raison, et que je vais tout recommencer à 0. Pourtant au départ l'idée me paraissait simple: le tableau HTML affiche 5 select parce qu'il y a 5 modules écrits dans la table, donc 5 positions. Je ne dois bien sûr avoir qu'un seul option selected par select, celui qui figure dans le champ "posi". Mais Yogui je comprends pas un truc: tu me dis que ça c'est bon:

Code :

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

1
2
 
if ($posit == $row["posi"] ) // ok si `posi` est un champ UNIQUE dans la BDD

Comment je peux comparer les valeurs de $row["posi"] avec $posit dont la valeur est array ???
Mais de toute façon mon idée n'est pas bonne puisque mon array() ne prend que 4 élèments, alors que la boucle while peut lire 10 lignes dans la table, donc afficher 10 selects contenant chacun les 4 élèments (header, droite,gauche,footer)...ou la la...

[Yogui]

Salut

En effet, tu as mis le doigt sur le problème : tu compares avec un array, ce n'est pas la bonne méthode. Tu dois comparer un élément qui ne changera pas du tout pendant ta boucle avec un élément qui change pendant ta boucle (et non comparer deux éléments qui changent pendant ta boucle, comme tu le fais actuellement).
C'est tout ce qu'il y a à changer.

[renaud26]

oui, je crois que j'imagine le truc...si tu veux, dans mon idée 1ere, le array contenant les 4 élèments de chaque select était l'élèment qui ne changeait pas dans la boucle, en comparaison avec la lecture table elle-même...me revoila au point de départ... sauf que je vois pas avec quoi comparer ! et le plus rageant, c'est que je l'affiche en clair à coté de chaque select dans une colonne "ancienne position" avec

Code :

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

1
2
 
$row["posi"];

qui lit bêtement le champ posi...
mais pas moyen de faire bouger ce maudit select !