Discussion :

[Lolosan]

Bonjour,

je viens vers vous car j'ai un problème avec le selectOneMenu de PrimeFaces. Je n'arrive pas à le faire fonctionner correctement et avec ce que je lis sur le net je suis pomé. J'ai tellement testé de trucs que je ne sais pas si ce que j'ai fini par mettre dans mes fichiers m'aide ou non.

Mon problème : Je ne parviens pas à mettre une valeur par défaut dans ma liste (c'est toujours la 1ere valeur qui est présélectionnée). Et je ne parviens pas à récupérer la valeur sélectionné par l'utilisateur, elle reste toujours inchangée : en fait le setter n'est jamais appelé.

La fameuse liste :

Code :

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

1
2
3
4
5
6
7
8
 
<h:outputLabel for="envselect" value="Environnement :" />  
<h:selectOneMenu id="envselect" value="#{pageInstanceBean.envSelected}" converter="#{environnementConverter}">  
	<f:selectItems value="#{pageInstanceBean.envAvailable}" />
	<f:ajax execute="envselect" render="inputEnvNom" />		         
</h:selectOneMenu>  
<p:message for="envselect" />
<h:outputText id="inputEnvNom" value="#{pageInstanceBean.envSelected.nomEnv}" />

Ma pageInstanceBean :

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
 
@ManagedBean
@ViewScoped
public class PageInstanceBean implements Serializable
{
    private Environnement envSelected;
 
    private List<SelectItem> envAvailable;
 
    @EJB
    private EnvironnementDao _environnementDao;
 
    //getter setter
 
    public Environnement getEnvSelected() 
    {
    	System.out.println("GET env = "+envSelected+" nom = "+envSelected.getNomEnv());
    	return envSelected;
    }
 
    public void setEnvSelected(Environnement envSelected) 
    {
	this.envSelected = envSelected;
	System.out.println("SET env = "+this.envSelected.getNomEnv());
     }
}

Sur le net un coup il faut pour l'item sélectionné le mettre en String, un autre il faut l'id, et un autre moment c'est sous forme d'Objet... Finalement pour la suite je me suis dis que le mieu est de l'avoir sous forme d'objet. Et de même pour la liste des item disponible j'ai vu une liste<Objet> ou une liste<SelectItem>.
Franchement avec tous ces choix, j'ai un peu tout testé et finalement rien compris. Donc si quelqu'un pourrais m'expliquer tout sa pour que je puisse faire le bon choix...

Avec tout sa aparement j'avais besoin d'un converter :

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
 
@FacesConverter(forClass = Environnement.class)
public class EnvironnementConverter implements Converter 
{
	@EJB
	private EnvironnementDao _environnementDao;
 
	@Override
	public Object getAsObject(FacesContext context, UIComponent component, String value) 
	{
		if (value.trim().equals("") || value.trim().length() == 0) 
		{
            return null;
        }
 
		if (_environnementDao != null)
		{
	        Environnement env = new Environnement();
	        env =_environnementDao.trouver(value);
	        return env;
		}
		else
		{
			return null;
		}
    }
 
	@Override
	public String getAsString(FacesContext context, UIComponent component, Object value) 
	{
		if (value == null)
		{
			return null;
		}
 
		if (value instanceof Environnement)
		{
			Environnement env = (Environnement) value;
			return env.getNomEnv();
		}
		else
		{
			return null;
		}
	}   
}

Et pour pouvoir utiliser des ejb dans un converter j'avais besoin dans mon fichier faces-config de sa :

Code :

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

1
2
3
4
5
6
 
<managed-bean>
	<managed-bean-name>environnementConverter</managed-bean-name>
	<managed-bean-class>com.converter.EnvironnementConverter</managed-bean-class>
	<managed-bean-scope>request</managed-bean-scope>
</managed-bean>

Et j'avais besoin d'ajouter dans ma classe Environnement ceci :

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
 
@Entity
@Table(name="Environnement")
public class Environnement 
{
    @Id
    @GeneratedValue( strategy = GenerationType.IDENTITY )
    @Column( name = "idEnv" )
    private int idEnv;
 
    @Column( name = "nomEnv" )
	private String nomEnv;
 
    @OneToMany(mappedBy="idEnv")
    private List<Instance> instanceList; 
 
	//getter setter
 
	public boolean equals(Environnement e)
	{
		boolean result = true;
 
		if (this.nomEnv != e.nomEnv)
		{
			result = false;
		}
 
		return result;
	}
 
	@Override
    public int hashCode() 
	{
        int hash = 1;
        hash = hash * 17 + idEnv;
        return hash;
    }
 
	@Override
    public String toString() 
	{
        return this.nomEnv;
    }
}

Avec tout sa maintenant quand je tente d'afficher directement envSelected j'obtient son nomEnv. Ce que je voulais. Comme j'ai cru comprendre que pour préselectionner une valeur PrimeFaces comparait le itemSelect avec la listItems et s'il y avait une correspondance c'était bon. Mais finalement il compare quoi ?
Néanmoins ma liste se remplis correctement, la conversion fonctionne (dans mes log je le vois), mais le reste...

Le seul endroit où je vois que la valeur sélectionnée est bien la bonne c'est dans la méthode getAsObject de mon converter car le String value prend bien la valeur que je viens de sélectionner... Le converter envois l'object généré à qui apres ?

Si quelqu'un à une idée, une explication ? Merci

[jeffray03]

salut,
ya t-il eu une erreur de copy ou pas:

Code :

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

1
2
3
@OneToMany(mappedBy="idEnv")
    private List<Instance> instanceList;

car

Code :

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

"idEnv"

est le id de classe

Code :

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

Environnement

et si non peux-tu nou montrer le contenu de la classe Instance

[Lolosan]

Alors non il n'y a pas d'erreur de copie mais je sens qu'on va me dire que finalement j'ai tout compris de travers... ><

J'ai cru comprendre que quand on a une bd de ce type (ce qui suis est du merise):

Table A ---------------------------------------- Table B
idA -----1,1 -------------------------- 1,* ----- idB


On devait traduire sa par :

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
 
@Entity
public class A
{
    @Column
    private int idA;
 
    @ManyToOne
    @JoinColumn...
    private B idB;
}
 
@Entity
public class B
{
    @Column
    private int idB;
 
    @OneToMany(mappedBy="idB")
    private List<A> listB;
}

Après j'ai peu etre mal compris et dans ce cas dites le moi !

Donc ma classe instance :

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
 
@Entity
@Table(name="Instance")
public class Instance 
{
    @Id
    @GeneratedValue( strategy = GenerationType.IDENTITY )
    @Column( name = "idInstance" )
    private int idInstance;
 
    //pleins d'autres proprietes
 
    @ManyToOne(cascade={CascadeType.PERSIST, CascadeType.MERGE, CascadeType.REFRESH })
    @JoinColumn( name = "idEnv" )
    private Environnement idEnv;
 
    //getter setter
}

Si j'ai mis le nom idEnv c'est que finalement dans ma base de donnée c'est une clé étrangère vers ma table. Et j'ai gardé ce nom pour ne pas m'embrouiller.

[jeffray03]

voici un exemple:

[Lolosan]

Si je ne me trompe pas c'est ce que j'ai fais...

[Mishulyna]

Bonjour Lolosan,

Tu parles d'un selectOneMenu de PrimeFaces, ce devrait être p:selectOneMenu et pas h:selectOneMenu.

[Lolosan]

Effectivement... j'ai modifié sa mais j'ai toujours le même problème. Et j'ai vérifié sur le site de PrimeFaces et ils ont deux exemples d'utilisation du SelectOneMenu l'un avec p et l'autre avec h... je ne sais pas trop quelle est la différence.

[Mishulyna]

Bonjour,

Je ne comprends pas pourquoi "envAvailable" c'est une liste d'objets SelectItem et pas d'objets Environnement. Essaie de changer en "private List<Environnement> envAvailable;".

[OButterlin]

Pour l'utilisation d'une liste d'objets dans une combo, il ne faut pas oublier de coder les méthodes hashCode et equals.
Pour le reste, la page ressemble à ceci (dans mon cas, un objet DTOClient)

Code :

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

1
2
3
4
5
6
7
8
 
<p:selectOneMenu value="#{rechercheFactureOptions.client}" converter="clientConverter">
    <f:selectItem itemLabel="" itemValue="#{null}" />
    <f:selectItems value="#{rechercheFactureOptions.clients}" var="cli" itemLabel="#{cli.nom}" itemValue="#{cli}"/>
</p:selectOneMenu>
/*
 * rechercheFactureOptions.clients est une List<DTOClient>, rechercheFactureOptions.client est un DTOClient
 */

Le converter

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
50
51
 
package com...ger.recherche.facture;
 
import javax.faces.component.UIComponent;
import javax.faces.context.FacesContext;
import javax.faces.convert.Converter;
import com...authentification.dto.DTOContexte;
import com....authentification.jaas.ClemessyPrincipal;
import com....recherches.ejb.client.RecherchesEJBLocator;
import com....recherches.ejb.client.dto.DTOClient;
 
public class ClientConverter implements Converter
{
 
    @Override
    public Object getAsObject(FacesContext arg0, UIComponent arg1, String arg2)
    {
        if (arg2 == null)
        {
            return null;
        }
        DTOClient dtoClient = null;
        try
        {
            DTOContexte dtoContexte = ((ClemessyPrincipal)FacesContext.getCurrentInstance().getExternalContext().getUserPrincipal()).getDtoContexte();
            dtoClient = RecherchesEJBLocator.getInstance().getRecherchesFacadeLocal().getClient(null, 0, arg2, dtoContexte);
        }
        catch (Exception e)
        {
            e.printStackTrace();
        }
        return dtoClient;
    }
 
    @Override
    public String getAsString(FacesContext arg0, UIComponent arg1, Object arg2)
    {
        if (arg2 == null)
        {
            return null;
        }
        try
        {
            return ((DTOClient)arg2).getCode();
        }
        catch (Exception e)
        {
            return null;
        }
    }
}

[Lolosan]

Bonjour,

Je ne comprends pas pourquoi "envAvailable" c'est une liste d'objets SelectItem et pas d'objets Environnement. Essaie de changer en "private List<Environnement> envAvailable;".

A l'origine mon code était comme sa. Mais la plupart des gens qui utilisent le selectOneMenu le font avec des list de SelectItem alors j'ai tenté. J'ai re-modifier mon code avec une liste d'environnement. Mais aucune évolution, le setter n'est jamais appelé.

A noté :
- mon selectOneMenu est bien entre des balise <h:form>
- j'ai modifié mon converter qui ne fonctionnais pas vraiment, maintenant je suis sure qu'il fonctionne
- j'ai bien un hashCode, un toString et un equals (modifié aussi) dans ma classe Environnement
- mon bean est en ViewScoped donc normalement ma liste ne varie pas au cours du temps
- j'ai tenté d'ajouter dans le selectitems les propriétées va, itemLabel et itemValue comme le montre OButterlin
- j'ai tenté d'ajouter <p:ajax event="change" update="@this"/>

J'ai l'impression d'avoir lu tous les tutos où les gens ont le même problème, testé toutes les solutions, vérifié mon code, sa ne me saute toujours pas aux yeux...

Si quelqu'un à une autre idée, ou s'il pourrait m'expliquer comment être sure que telle partie de code (comme mon converter, ou mon equals par exemple) est vraiment fonctionnel, pour pouvoir éliminer certaines possibilitées...

Merci à tous ceux qui tentent de m'aider !

[Lolosan]

Je viens de résoudre mon problème et j'ai bien honte... je vais donner la solution pour tous ceux qui sont, comme moi, un peu tête en l'air.

Quand on utilise un selectOneMenu il faut redéfinir la fonction equals et je dis bien redéfinir...

J'avais écris :

Code :

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

1
2
3
4
5
 
public boolean equals(Environnement e)
{
	return this.nomEnv.equals(env.nomEnv);
}

Et il fallait écrire :

Code :

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

1
2
3
4
5
6
7
 
@Override
public boolean equals(Object e)
{
	Environnement env = (Environnement) e;
	return this.nomEnv.equals(env.nomEnv);
}

J'ai découvert mon erreur grâce à ce petit lien bien utile pour faire apparaitre les erreurs cachées :
http://www.softwareengineeringsoluti...eListeners.htm

Encore merci à tous !

[OButterlin]

Comme tu as fais pas mal de modifications, pourrais-tu montrer le code actuel ?

[Lolosan]

Oui bien sur. Donc actuellement ce qui fonctionne.

La liste :

Code :

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

1
2
3
4
5
6
7
8
 
<h:outputLabel for="envselect" value="Environnement :" />  
<h:selectOneMenu id="envselect" value="#{pageInstanceBean.envSelected}" converter="#{environnementConverter}">  
	<f:selectItems value="#{pageInstanceBean.envAvailable}" var="environnement" itemValue="#{environnement}" itemLabel="#{environnement.nomEnv}"/>
	<f:ajax event="change" update="@this" render="inputEnvNom" /> 	         
</h:selectOneMenu>  
<p:message for="envselect" />
<h:outputText id="inputEnvNom" value="#{pageInstanceBean.envSelected.nomEnv}" />

Le bean :

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
 
@ManagedBean
@ViewScoped
public class PageInstanceBean implements Serializable
{	
    private Environnement envSelected;
 
    private List<SelectItem> envAvailable;
 
    @EJB
    private EnvironnementDao _environnementDao;
 
    // getter setter ...
	// finalement sa fonctionne avec une List<SelectItem> mais sa doit etre la
	// même chose avec une List<Environnement>
}

Le converter :

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
 
@FacesConverter(forClass = Environnement.class)
public class EnvironnementConverter implements Converter 
{
	@EJB
	private EnvironnementDao _environnementDao;
 
	@Override
	public Object getAsObject(FacesContext context, UIComponent component, String value) 
	{
		List<Environnement> env = _environnementDao.lister();
		for(Environnement e : env)
		{
			if (e.getNomEnv().equals(value))
			{
				return e;
			}
		}
		return null;
    }
 
	@Override
	public String getAsString(FacesContext context, UIComponent component, Object value) 
	{
		Environnement env = (Environnement) value;
		return env.getNomEnv();
	}   
}

Et ma classe Environnement :

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
 
@Entity
@Table(name="Environnement")
public class Environnement 
{
    @Id
    @GeneratedValue( strategy = GenerationType.IDENTITY )
    @Column( name = "idEnv" )
    private int idEnv;
 
    @Column( name = "nomEnv" )
	private String nomEnv;
 
    @OneToMany(mappedBy="idEnv")
    private List<Instance> instanceList; 
 
    //setters getters
 
	@Override
	public boolean equals(Object e)
	{
		Environnement env = (Environnement) e;
		return this.nomEnv.equals(env.nomEnv);
	}
 
	@Override
    public int hashCode() 
	{
        int hash = 1;
        hash = hash * 17 + idEnv;
        return hash;
    }
 
	@Override
    public String toString() 
	{
        return this.nomEnv;
    }
}

Mais bon malgrès toutes ces modifs je suis persuadé que le seul problème dès le départ été le equals. Le JSF c'est pas mal mais alors à débugger... c'est tellement prise de tête ><

[Alkhan]

Bonjour,
c'est effectivement une solution, mais tu n'aurais pas eu le problème si tu avais utilisé :

Code :

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

itemValue="#{environnement.idEnv}"

Tu as besoin de redéfinir equals uniquement parce que tu utilise l'objet en value.