Supprimer une entrée de h:selectOneListbox

Version imprimable

27/04/2009, 16h59
moha1984

Supprimer une entrée de h:selectOneListbox
Bonjour à tous,

Je veut supprimer une entrée de ma h:selectOneListbox.

Voila mon code

JSF :
Code:

1 2 3 <h:selectManyListbox value="#{referenceMB.SelectedTechnologie}" > <f:selectItems value="#{referenceMB.listeTechnologies}" /> </h:selectManyListbox>
Managed Bean :

Code:

private int SelectedTechnologie;

a partir de l'identifiant je récupère la Technologie et puis je l'ajoute dans une autre liste en utilisant la méthode add ( SelectItem s )
Code:

1 2 3 4 5 tech=t.doFindTechnologie(selected); final SelectItem ld= new SelectItem(tech.getNotechnologie(),tech.getDescription(),note) ; this.listeReferenceTechnologies.add(ld);
Mon probleme réside dans la suppression du SelectItem de la liste ...
J'utilise la méthode remove(object o) qui prend en paramètre un objet pas un SelectItem comme pour l'ajout... et ca marche pas pour moi..
Est ce qu'il ya une méthode de convertir un SelectItem en un objet...j'ai meme essayer de récupérer l'index du SelectItem et j'ai tombé sur le meme probleme.
Code:

1 2 3 4 5 6 7 for (SelectItem i: listeTechnologies) { if(i.getValue()==ld.getValue()) { listeTechnologies.remove(i); // voila la ligne de l'erreur }
Merci de m'aider.
27/04/2009, 17h30
Sniper37

tu dois redéfinir la méthode equals et hashcode du selectItem,

tu peux le faire dans le constructeur.
27/04/2009, 17h41
moha1984

Citation:

Envoyé par Sniper37

tu dois redéfinir la méthode equals et hashcode du selectItem,

tu peux le faire dans le constructeur.

Dans quel constructeur? de mon managed Bean?
Est ce que tu peut m'expliquer plus avec un bout de code? que je doit mettre dans ces deux méthode?

tu peux faire un truc dans le genre:
Code:

1 2 3 4 5 6 7 8 9 MonBean value = ..; String libelle ..; SelectItem s = new SelectItem(value,libelle) {@Override public boolean equals(Object obj) { // return obj.getId() == value.getId(); } };
Sinon tu peux étendre la classe SelectItem et redéfinir la méthode equals.
ou faire le remove à la main..une fonction qui enleve l'objet de la liste..

Citation:
Envoyé par Sniper37
tu peux faire un truc dans le genre:
Code:

1 2 3 4 5 6 7 8 9 MonBean value = ..; String libelle ..; SelectItem s = new SelectItem(value,libelle) {@Override public boolean equals(Object obj) { // return obj.getId() == value.getId(); } };
Sinon tu peux étendre la classe SelectItem et redéfinir la méthode equals.
ou faire le remove à la main..une fonction qui enleve l'objet de la liste..
OK j'ai comprit le principe... je vait essayer avec ca...Merci .

Bonjour,

J'ai essayé de redéfinir la classe SelectItem et implémenter les méthode equals() et hashcode() mais j'avais la même erreur.

Par contre j'ai trouvé une autre piste... Je récupère maintenant l'index de l'élément sélectionné en faisant un Cast pour convertir de int à Object.
Ca marche bien pour moi , il récupère l'index..et supprime le SelectItem de la liste sauf que parfois il génère une exception :( :

voila mon code :

Code:

1
2
3
4
5
6
7
8
9
 for (SelectItem i: listeTechnologies) {
			if(i.getValue()==ld.getValue())
			{
			int index=	listeTechnologies.indexOf((Object)i);
			System.out.println("index est :"+index);
	     	listeTechnologies.remove(index);
 
			}
		}

et voila l'exception :

Code:

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
 java.util.ConcurrentModificationException
javax.faces.el.EvaluationException: java.util.ConcurrentModificationException
	at javax.faces.component.MethodBindingMethodExpressionAdapter.invoke(MethodBindingMethodExpressionAdapter.java:102)
	at com.sun.faces.application.ActionListenerImpl.processAction(ActionListenerImpl.java:102)
	at javax.faces.component.UICommand.broadcast(UICommand.java:387)
	at org.ajax4jsf.component.AjaxViewRoot.processEvents(AjaxViewRoot.java:321)
	at org.ajax4jsf.component.AjaxViewRoot.broadcastEvents(AjaxViewRoot.java:296)
	at org.ajax4jsf.component.AjaxViewRoot.processPhase(AjaxViewRoot.java:253)
	at org.ajax4jsf.component.AjaxViewRoot.processApplication(AjaxViewRoot.java:466)
	at com.sun.faces.lifecycle.InvokeApplicationPhase.execute(InvokeApplicationPhase.java:82)
	at com.sun.faces.lifecycle.Phase.doPhase(Phase.java:100)
	at com.sun.faces.lifecycle.LifecycleImpl.execute(LifecycleImpl.java:118)
	at javax.faces.webapp.FacesServlet.service(FacesServlet.java:265)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:290)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:206)
	at org.ajax4jsf.webapp.BaseXMLFilter.doXmlFilter(BaseXMLFilter.java:178)
	at org.ajax4jsf.webapp.BaseFilter.handleRequest(BaseFilter.java:290)
	at org.ajax4jsf.webapp.BaseFilter.processUploadsAndHandleRequest(BaseFilter.java:390)
	at org.ajax4jsf.webapp.BaseFilter.doFilter(BaseFilter.java:517)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:235)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:206)
	at org.apache.myfaces.webapp.filter.ExtensionsFilter.doFilter(ExtensionsFilter.java:301)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:235)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:206)
	at org.jboss.web.tomcat.filters.ReplyHeaderFilter.doFilter(ReplyHeaderFilter.java:96)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:235)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:206)
	at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:230)
	at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:175)
	at org.jboss.web.tomcat.security.SecurityAssociationValve.invoke(SecurityAssociationValve.java:182)
	at org.jboss.web.tomcat.security.JaccContextValve.invoke(JaccContextValve.java:84)

Tu ne peux pas supprimer un élément d'une liste sur laquelle tu es en train d'itérer :
Code:

1 2 3 4 for (SelectItem i : listeTechnologies) { ... listeTechnologies.remove(xxx); }
Tu peux passer par une liste intermédiaire par exemple (c'est pas top optimisé, je le reconnais, mais ça marche quand même) :
Code:

1 2 3 4 5 6 7 8 List<SelectItem> toRemove = new ArrayList<SelectItem>(); for (SelectItem i : listeTechnologies) { ... toRemove.add(i); } for (SelectItem i : toRemove) { listeTechnologies.remove(i); }

Citation:
Envoyé par romaintaz
Tu ne peux pas supprimer un élément d'une liste sur laquelle tu es en train d'itérer :
Code:

1 2 3 4 for (SelectItem i : listeTechnologies) { ... listeTechnologies.remove(xxx); }
Tu peux passer par une liste intermédiaire par exemple (c'est pas top optimisé, je le reconnais, mais ça marche quand même) :
Code:

1 2 3 4 5 6 7 8 List<SelectItem> toRemove = new ArrayList<SelectItem>(); for (SelectItem i : listeTechnologies) { ... toRemove.add(i); } for (SelectItem i : toRemove) { listeTechnologies.remove(i); }
Ah bon...
J'ai trouvé un moyen plus pratique , vu que je supprime qu'un seul élément à la fois... j'ai déplacé la méthode remove() hors de la boucle :
Code:

1 2 3 4 5 6 7 8 for (SelectItem i: listeTechnologies) { if(i.getValue()==ld.getValue()) { index= listeTechnologies.indexOf((Object)i); System.out.println("index est :"+index); } } listeTechnologies.remove(index);
Maintenant ca marche à merveille..... Merci pour ton aide :king:

Effectivement, si tu ne supprimes qu'un seul élément.
Mais tu peux optimiser un peu ta boucle :
Code:

1 2 3 4 5 6 7 8 9 10 11 int i = 0; int index = -1; while (i < listeTechnologies && index == -1) { SelectItem si = listeTechnologies.get(i); // Ton code... ... i++; } if (index != -1) { listeTechnologies.remove(index); }
ça permet de ne parcourir la liste que jusqu'à ce que tu trouves ton élément à supprimer...

Citation:
Envoyé par romaintaz
Effectivement, si tu ne supprimes qu'un seul élément.
Mais tu peux optimiser un peu ta boucle :
Code:

1 2 3 4 5 6 7 8 9 10 11 int i = 0; int index = -1; while (i < listeTechnologies && index == -1) { SelectItem si = listeTechnologies.get(i); // Ton code... ... i++; } if (index != -1) { listeTechnologies.remove(index); }
ça permet de ne parcourir la liste que jusqu'à ce que tu trouves ton élément à supprimer...
Ok , Merci