Récupération de valeurs d'un formulaire dynamique

Invité · 17/04/2009, 11h18

Salut,
j'ai un formulaire dynamique sur ma JSP. En fait, mon formulaire doit pouvoir envoyer un nombre de pièces jointes au serveur qui est inconnu au chargement de la page. Pour faire ça, j'ai inséré un bout de Javascript qui ajoute un champ input de type file lors du clic sur un bouton.
Le code généré est :

Code :

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

1
2
3
<li><a href="#piecesjointes" onclick="$(this).parent().remove();"><img src="image/remove.png"></a><input name="piecejointe[1]" value="" class="piecejointe" id="1" type="file"></li>
<li><a href="#piecesjointes" onclick="$(this).parent().remove();"><img src="image/remove.png"></a><input name="piecejointe[2]" value="" class="piecejointe" id="2" type="file"></li>
<li><a href="#piecesjointes" onclick="$(this).parent().remove();"><img src="image/remove.png"></a><input name="piecejointe[3]" value="" class="piecejointe" id="3" type="file"></li>

Dans mon bean associé à ce formulaire, j'ai :

Code :

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

1
2
3
4
5
6
7
8
9
private FormFile[] piecejointe;
 
	public FormFile[] getPiecejointe() {
		return piecejointe;
	}
 
	public void setPiecejointe(FormFile[] piecejointe) {
		this.piecejointe = piecejointe;
	}

Et lors de la soumission du formulaire, j'obtiens cette exception :

javax.servlet.ServletException: BeanUtils.populate
org.apache.struts.util.RequestUtils.populate(RequestUtils.java:495)
org.apache.struts.action.RequestProcessor.processPopulate(RequestProcessor.java:816)
org.apache.struts.action.RequestProcessor.process(RequestProcessor.java:203)
org.apache.struts.action.ActionServlet.process(ActionServlet.java:1196)
org.apache.struts.action.ActionServlet.doPost(ActionServlet.java:432)
javax.servlet.http.HttpServlet.service(HttpServlet.java:647)
javax.servlet.http.HttpServlet.service(HttpServlet.java:729)

cause mère

java.lang.NullPointerException
org.apache.commons.beanutils.PropertyUtilsBean.setIndexedProperty(PropertyUtilsBean.java:1414)
org.apache.commons.beanutils.BeanUtilsBean.setProperty(BeanUtilsBean.java:1016)
org.apache.commons.beanutils.BeanUtilsBean.populate(BeanUtilsBean.java:811)
org.apache.commons.beanutils.BeanUtils.populate(BeanUtils.java:298)
org.apache.struts.util.RequestUtils.populate(RequestUtils.java:493)
org.apache.struts.action.RequestProcessor.processPopulate(RequestProcessor.java:816)
org.apache.struts.action.RequestProcessor.process(RequestProcessor.java:203)
org.apache.struts.action.ActionServlet.process(ActionServlet.java:1196)
org.apache.struts.action.ActionServlet.doPost(ActionServlet.java:432)
javax.servlet.http.HttpServlet.service(HttpServlet.java:647)
javax.servlet.http.HttpServlet.service(HttpServlet.java:729)

J'ai aussi essayé, comme déjà vu sur le net, de générer des champs input avec le même attribut name (et non pas avec un indice qui s'incrémente. J'obtiens alors :

javax.servlet.ServletException: BeanUtils.populate
org.apache.struts.util.RequestUtils.populate(RequestUtils.java:495)
org.apache.struts.action.RequestProcessor.processPopulate(RequestProcessor.java:816)
org.apache.struts.action.RequestProcessor.process(RequestProcessor.java:203)
org.apache.struts.action.ActionServlet.process(ActionServlet.java:1196)
org.apache.struts.action.ActionServlet.doPost(ActionServlet.java:432)
javax.servlet.http.HttpServlet.service(HttpServlet.java:647)
javax.servlet.http.HttpServlet.service(HttpServlet.java:729)

cause mère

java.lang.IllegalArgumentException: Cannot invoke logica.forms.DemandeForm.setPiecejointe - argument type mismatch
org.apache.commons.beanutils.PropertyUtilsBean.invokeMethod(PropertyUtilsBean.java:1778)
org.apache.commons.beanutils.PropertyUtilsBean.setSimpleProperty(PropertyUtilsBean.java:1759)
org.apache.commons.beanutils.PropertyUtilsBean.setNestedProperty(PropertyUtilsBean.java:1648)
org.apache.commons.beanutils.PropertyUtilsBean.setProperty(PropertyUtilsBean.java:1677)
org.apache.commons.beanutils.BeanUtilsBean.setProperty(BeanUtilsBean.java:1022)
org.apache.commons.beanutils.BeanUtilsBean.populate(BeanUtilsBean.java:811)
org.apache.commons.beanutils.BeanUtils.populate(BeanUtils.java:298)
org.apache.struts.util.RequestUtils.populate(RequestUtils.java:493)
org.apache.struts.action.RequestProcessor.processPopulate(RequestProcessor.java:816)
org.apache.struts.action.RequestProcessor.process(RequestProcessor.java:203)
org.apache.struts.action.ActionServlet.process(ActionServlet.java:1196)
org.apache.struts.action.ActionServlet.doPost(ActionServlet.java:432)
javax.servlet.http.HttpServlet.service(HttpServlet.java:647)
javax.servlet.http.HttpServlet.service(HttpServlet.java:729)

Est ce que ça vient du fait que ce ne soit pas un type String simple, mais un champ file ?

**c_nvy** · 17/04/2009, 11h33

Cela fonctionne-t'il mieux en ajoutant getter et setter indexé dans le form-bean :

Code :

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

1
2
3
4
5
6
7
8
9
public void setPiecejointe(int index, FormFile formFile)
{   
  this.piecejointe[index] = formFile;   
}   
 
public FormFile getPiecejointe(int index)
{   
  return this.piecejointe[index];   
}

EDIT : Il faut aussi initialiser le tableau de FormFile au nombre voulu :

Code :

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

private FormFile[] piecejointe= new FormFile[3];

Invité · 17/04/2009, 11h46

A priori avec ta solution j'arrive sur une autre exception :

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
javax.servlet.ServletException: BeanUtils.populate
	org.apache.struts.util.RequestUtils.populate(RequestUtils.java:495)
	org.apache.struts.action.RequestProcessor.processPopulate(RequestProcessor.java:816)
	org.apache.struts.action.RequestProcessor.process(RequestProcessor.java:203)
	org.apache.struts.action.ActionServlet.process(ActionServlet.java:1196)
	org.apache.struts.action.ActionServlet.doPost(ActionServlet.java:432)
	javax.servlet.http.HttpServlet.service(HttpServlet.java:647)
	javax.servlet.http.HttpServlet.service(HttpServlet.java:729)
 
cause mère
 
java.lang.reflect.InvocationTargetException
	sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
	sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
	java.lang.reflect.Method.invoke(Unknown Source)
	org.apache.commons.beanutils.PropertyUtilsBean.invokeMethod(PropertyUtilsBean.java:1773)
	org.apache.commons.beanutils.PropertyUtilsBean.setIndexedProperty(PropertyUtilsBean.java:1391)
	org.apache.commons.beanutils.BeanUtilsBean.setProperty(BeanUtilsBean.java:1016)
	org.apache.commons.beanutils.BeanUtilsBean.populate(BeanUtilsBean.java:811)
	org.apache.commons.beanutils.BeanUtils.populate(BeanUtils.java:298)
	org.apache.struts.util.RequestUtils.populate(RequestUtils.java:493)
	org.apache.struts.action.RequestProcessor.processPopulate(RequestProcessor.java:816)
	org.apache.struts.action.RequestProcessor.process(RequestProcessor.java:203)
	org.apache.struts.action.ActionServlet.process(ActionServlet.java:1196)
	org.apache.struts.action.ActionServlet.doPost(ActionServlet.java:432)
	javax.servlet.http.HttpServlet.service(HttpServlet.java:647)
	javax.servlet.http.HttpServlet.service(HttpServlet.java:729)
 
cause mère
 
java.lang.NullPointerException
	logica.forms.DemandeForm.setPiecejointe(DemandeForm.java:53)
	sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
	sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
	java.lang.reflect.Method.invoke(Unknown Source)
	org.apache.commons.beanutils.PropertyUtilsBean.invokeMethod(PropertyUtilsBean.java:1773)
	org.apache.commons.beanutils.PropertyUtilsBean.setIndexedProperty(PropertyUtilsBean.java:1391)
	org.apache.commons.beanutils.BeanUtilsBean.setProperty(BeanUtilsBean.java:1016)
	org.apache.commons.beanutils.BeanUtilsBean.populate(BeanUtilsBean.java:811)
	org.apache.commons.beanutils.BeanUtils.populate(BeanUtils.java:298)
	org.apache.struts.util.RequestUtils.populate(RequestUtils.java:493)
	org.apache.struts.action.RequestProcessor.processPopulate(RequestProcessor.java:816)
	org.apache.struts.action.RequestProcessor.process(RequestProcessor.java:203)
	org.apache.struts.action.ActionServlet.process(ActionServlet.java:1196)
	org.apache.struts.action.ActionServlet.doPost(ActionServlet.java:432)
	javax.servlet.http.HttpServlet.service(HttpServlet.java:647)
	javax.servlet.http.HttpServlet.service(HttpServlet.java:729)

Mais d'après ce que je comprends ça viendrait du fait que le tableau soit null lors du set. Je vais élaborer un peu ton set pour vérifier ça

Invité · 17/04/2009, 11h57

Comme je le pensais il suffisait d'élaborer le set pour étendre le tableau lorsque l'index à setter est en dehors du tableau. Voici les fonctions réécrites :

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 void setPiecejointe(int index, FormFile formFile)
	{
		if(this.piecejointe == null)
			this.piecejointe = new FormFile[1];
 
		//tableau trop petit, on réalloue
		if(this.piecejointe.length < index)
		{
			FormFile[] temp = new FormFile[index+1];
			for(int i = 0 ; i < piecejointe.length ; i++)
				temp[i] = piecejointe[i];
			piecejointe = temp;
		}
 
		this.piecejointe[index] = formFile;
	}
 
	public FormFile getPiecejointe(int index)
	{
	  return this.piecejointe[index];
	}

Il y a peut être une manière plus classe de le faire, mais ça marche bien comme ça. Encore merci c_nvy

**c_nvy** · 17/04/2009, 12h23

Effectivement, il fallait prévoir le cas où l'index est supérieur à la taille du tableau.

Ne connaissant pas la taille initiale du tableau, tu aurais pu utiliser une List de FormFile plutôt qu'un tableau, ce qui aurait été plus simple à gérer :

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
private List piecejointe = new ArrayList() ;
 
public void setPiecejointe(int index, FormFile formFile)
{   
  this.piecejointe.add(formFile);   
}
public FormFile getPiecejointe(int index)
{
   if ( piecejointe == null ) this.piecejointe = new ArrayList() ;
   while (index >= this.piecejointe.size())
   {
       this.piecejointe.add(new FormFile());
   }
   return (FormFile) this.piecejointe.get(index);
}

Invité · 17/04/2009, 13h14

Exact, c'était la solution plus classe !
je ne connaissais pas le principe des get / set indexés. J'aurai appris quelque chose !

Récupération de valeurs d'un formulaire dynamique

Struts 1 Java

Vue hybride

Discussions similaires

Partager

Partager