Discussion :

[Vincinho]

Bonjour tout le monde,

Dans un site web j'aimerais mettre une liste deroulante dans un formulaire sur une JSP qui donnerais le choix de régions. Jusque la ca va mais j'aimerais conserver seulement le numéro de cette région qui est dans ma base de données mais afficher le nom des régions. J'ai fais ca :

Code :

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

1
2
3
4
5
6
7
8
9
10
<td><FORM>
<SELECT name="regioncreation" size="1">
<%for (int j=0;j<z.size();j++){
        Region r=(Region)z.get(j);
%>
<OPTION><%out.println(r.getNomRegion()); %>
<%} %>
</SELECT>
</FORM>
</td>

Ca m'affiche bien toutes mes regions mais j'aimerais que lorsque j'en ai sélectionné une, ca envoie a ma servlet qui s'occupe de créer mon camping uniquement le numéro de la région. Et je vois pas comment faire.

[Jidefix]

Code :

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

<OPTION value="<%=r.getNumeroRegion()%>"><%=r.getNomRegion()%></OPTION>

penses aussi à la syntaxe

Code :

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

<%=toto%>

qui est beaucoup plus simple que

Code :

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

<% out.println(toto)%>

[regis1512]

Bonjour,

Conseil :

Code :

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

1
2
3
4
5
6
 
<%
final Iterator lIterator = z.iterator();
while(lIterator.hasNext()){
        Region r=(Region)lIterator.next();
%>

au lieu de :

Code :

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

1
2
3
4
 
<%for (int j=0;j<z.size();j++){
        Region r=(Region)z.get(j);
%>

L'iterator est meilleur que de faire un get() sur chaque élément.

Bon courage.

[Vincinho]

C'est bon j'ai bien la liste des régions qui s'affiche. Maintenant ce que je voudrais c'est envoyé a ma servlet uniquement le numéro de la région sélectionné et non son appelation. Et je vois pas trop comment faire.

[regis1512]

Bonjour,

Si tu fais comme t'a indique Jidefix en précisant la value de l'option (en rouge), ca doit le faire.

Bon courage.

[Jidefix]

Ben si tu as appliqué mon exemple, le contenu de la variable

Code :

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

1
2
 
request.getParameter("regioncreation");

est le numéro de ta région puisque tu affiche la veleur qui est entre les balise <OPTION> et </OPTION> mais le contenu réel de la variable qui sera envoyé au serveur est le contenu de l'attribut "value"

[Vincinho]

Mais comment je fais mon input dans ce cas et où ?

J'ai ca désormais :

Code :

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

1
2
3
4
5
6
7
8
9
10
<td><FORM>
<SELECT name="regioncreation" size="1">
<%for (int j=0;j<z.size();j++){
        Region r=(Region)z.get(j);
%>
<OPTION value="<%=r.getIdRegion()%>"><%=r.getNomRegion()%></OPTION>
<%} %>
</SELECT>
</FORM>
</td>

J'ai pas encore mis l'iterator je verrais ca pour une autre fois ou alors si j'ai le temps
Mais quand je cliques sur mon bouton il fais rien et plante le prog si je touche a un autre bouton

Pour récupéré mon numéro de région dans ma servlet je faisais ca :

Code :

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

c.setNumRegion(Integer.valueOf(request.getParameter("regioncreation")));

[Jidefix]

Normal tu dois mettre un attribut "action" dans la balise "form" pour lui dire vers quelle servlet le bouton doit envoyer le formulaire.

Sinon ça me semble correct

[Vincinho]

J'ai mis ca maintenant :

Code :

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

1
2
3
4
5
6
7
8
9
10
<td><FORM action="/Camping/listeCamping" method="get" target="gestion">
<SELECT name="regioncreation" size="1">
<%for (int j=0;j<z.size();j++){
        Region r=(Region)z.get(j);
%>
<OPTION value="<%=r.getIdRegion()%>"><%=r.getNomRegion()%></OPTION>
<%} %>
</SELECT>
</FORM>
</td>

Du coup il envoie bien vers la servlet mais ca marche pas mieux je vois pas trop comment ca doit marcher

[Jidefix]

J'ai une question con mais il est ou ton

Code :

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

<INPUT type=submit>

?
Tu l'as défini comment ton bouton pour envoyer le formulaire?

[Vincinho]

Je vous met le bout de code pour entrer un nouveau camping

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
<h2>Création d'un nouveau camping</h2>
<table border=1>
<tr>
	<th>Région</th>
	<th>Nom</th>
	<th>Description</th>
	<th>Image</th>
</tr>
<tr>
<td><FORM action="/Camping2009/listeCamping" method="get" target="gestion">
<SELECT name="regioncreation" size="1">
<%for (int j=0;j<z.size();j++){
        Region r=(Region)z.get(j);
%>
<OPTION value="<%=r.getIdRegion()%>"><%=r.getNomRegion()%></OPTION>
<%} %>
</SELECT>
</FORM>
</td>
 
 
	<td><input type="text" name="nomcreation"</td>
	<td><input type="text" name="descreation"</td>
	<td><input type="file" name="imgcreation"</td>
</tr>
</table>
<input type="submit" name="bouton" value="Enregistrer">

Et ma servlet :

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
if(request.getParameter("bouton").equalsIgnoreCase("Enregistrer")){
				Camping c;
				c = new Camping();
								c.setNomCamping(request.getParameter("nomcreation"));
				c.setDescCamping(request.getParameter("descreation"));
				c.setImgCamping(request.getParameter("imgcreation"));
				c.setNumRegion(Integer.valueOf(request.getParameter("regioncreation")));
				try {
					Campings.enregistrer(c);
					v=Campings.listeCampings();
					i=v.size()-1;
					request.setAttribute("liste", v);
					request.setAttribute("indice", i);
					request.setAttribute("regions",z);
				} catch (Exception e) {
					e.printStackTrace();
				}
			}

Au debut de ma page j'ai deja une <form action="... donc je sais pas si c'est necessaire de le remettre. En tout cas dans l'etat actuel mon bouton ne fais rien, il ne plante pas tout de suite le programme mais je peux rien faire

[Jidefix]

Ben oui il plante pas le programme parce qu'il ne fait strictement rien

Pour info la structure d'un formulaire ça ressemble à ça:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
<form action="nomDeTaServlet" type = get>
    <input type = text name = "text1">
   <select name = "select1">
      <option value="1">option1</option>
      <option value="2">option2</option>
   </select>
    ... (autres input)
 
   <input type=submit name="bouton" value="Valider le formulaire">
 
</form>

Donc en gros ce qui ne va pas dans ton formulaire c'est que le submit doit etre entre les balises <form> et </form> du formulaire de ta page.

Autres choses en vrac:
- tu ne peux pas imbriquer deux formulaires, donc ton autre balise <form>, en plus de ne servir à rien, risque de faire mal marcher ta page
- un input en dehors des deux balises form ne sera pas envoyé au serveur (donc tous tes autres input ne seront pas pris en compte


Essaye de respecter un peu plus l'arborescence logique de l'HTML, l'imbrication des balises est fondamentale

[Vincinho]

En fait c'est bon j'ai réglé le problème. Comme j'avais deja un formulaire plus haut, le fait de mettre une balise </form> desactivait logiquement le bouton. J'ai donc enlevé le formulaire autour de ma liste déroulante et ca tourne nickel.

Code :

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

1
2
3
4
5
6
7
8
<SELECT name="regioncreation" size="1">
<%for (int j=0;j<z.size();j++){
        Region r=(Region)z.get(j);
%>
<OPTION value="<%=r.getIdRegion()%>"><%=r.getNomRegion()%></OPTION>
<%} %>
</SELECT>
</td>

Je met le code au cas où ca interesse certains. Du coup dans cet exemple ca affiche sur ma JSP le nom de mes regions ("r.getNomRegion()") mais ca n'envoit a ma servlet que le numéro correspondant ("r.getIdRegion()").