Discussion :

[koack]

Bonjour,

j'utilise une datatable pour afficher divers données mais j'ai remarqué que la datatable faisait plusieurs fois appel au getter pour obtenir la liste a afficher.

ma jsp : j'ai supprimé ce qui ne me paraisait pas utile

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
<%--BODY--%>
    <div id="body">
           
    <h:dataTable id="Account" width="100%" value="#{AccountManagedBean.accountEntityList}" var="uneLigne" rules="rows" >
    <h:column>
     	<f:facet name="header">
        	<h:outputText value="numero de compte"/>
        </f:facet>
        <h:form>
        <h:commandLink action="#{AccountManagedBean.showDetails}">
        <h:outputText value="#{uneLigne.account}"/>
        </h:commandLink>
        </h:form>
    </h:column>
    <h:column>
     	<f:facet name="header">
        	<h:outputText value="description"/>
        </f:facet>
        <h:outputText value="#{uneLigne.description}"/>
    </h:column>
  	</h:dataTable>
    </div>
</body>

Losque j'affiche ma jsp j'ai deja fais 3 appels au getter de la liste :

Code :

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

1
2
3
4
5
 
10:49:42,484 INFO  [STDOUT] UserAccountsBean constructor
10:49:42,484 INFO  [STDOUT] getAccountEntityList
10:49:42,505 INFO  [STDOUT] getAccountEntityList
10:49:42,515 INFO  [STDOUT] getAccountEntityList

mon 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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
@Stateful
public class UserAccountsBean implements UserAccountsBeanLocal {

       
	@PersistenceContext(unitName ="Mes_Tables")
	private EntityManager em;
	private UserAccountsEntity userAccountsEntity = new UserAccountsEntity(); 
	
	public UserAccountsBean() {
		System.out.println("UserAccountsBean constructor");
	}
	
	public void populate(){
						
	ici je rempli ma base pour les tests	
	}
	public UserAccountsEntity getUserAccountEntity(String login){
		
		Query query = em.createQuery("SELECT a FROM UserAccountsEntity a WHERE a.login=:login");
		query.setParameter("login", login);
		int a=0;		
		try {
			//TODO single result
			userAccountsEntity=(UserAccountsEntity)query.getResultList().get(0); //.getSingleResult();
			a=userAccountsEntity.getAccountsEntity().size();
		}
		
		catch (NoResultException nre){
			
		}
				
		
		return userAccountsEntity;
	}
	}

et mon managedbean :

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
52
53
54
55
56
 
 
public class AccountManagedBean {
 
	@EJB
	private UserAccountsBeanLocal userAccountsBean;
 
	private UserAccountsEntity userAccountsEntity = new UserAccountsEntity();
	private final Principal user;
 
 
	public AccountManagedBean() {
	/*Save the principal's user*/	
	FacesContext fc = FacesContext.getCurrentInstance();
	user = fc.getExternalContext().getUserPrincipal();	
 
	}
 
	public String showDetails() {
		System.out.println("showDetails");
		FacesContext fc = FacesContext.getCurrentInstance();
		HtmlDataTable selectedAccount = (HtmlDataTable) fc.getViewRoot().findComponent("Account");
		AccountEntity account = (AccountEntity)selectedAccount.getRowData();
 
		if(account!=null) {
			System.out.println("selected account ="+account.getId());
		}
 
		return "showDetails";
	}
 
	public List<AccountEntity> getAccountEntityList() {
		System.out.println("getAccountEntityList");
		userAccountsBean.populate();
		userAccountsEntity = userAccountsBean.getUserAccountEntity(user.getName());
		List<AccountEntity> maliste = new ArrayList<AccountEntity>();
		maliste= userAccountsEntity.getAccountsEntity();
		return maliste;
 
	}
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
}

Et lorsque je fais appel a showDetails, la c'est le ponpon je fais au moins 15 appels au getter de la liste.

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
 
11:05:15,663 INFO  [STDOUT] getAccountEntityList
 
11:05:15,673 INFO  [STDOUT] getAccountEntityList
 
11:05:15,685 INFO  [STDOUT] getAccountEntityList
 
11:05:15,693 INFO  [STDOUT] getAccountEntityList
 
11:05:15,702 INFO  [STDOUT] getAccountEntityList
 
11:05:15,709 INFO  [STDOUT] getAccountEntityList
 
11:05:15,715 INFO  [STDOUT] getAccountEntityList
 
11:05:15,715 INFO  [STDOUT] getAccountEntityList
 
11:05:15,725 INFO  [STDOUT] getAccountEntityList
 
11:05:15,735 INFO  [STDOUT] getAccountEntityList
 
11:05:15,745 INFO  [STDOUT] getAccountEntityList
 
11:05:15,755 INFO  [STDOUT] getAccountEntityList
 
11:05:15,765 INFO  [STDOUT] getAccountEntityList
 
11:05:15,775 INFO  [STDOUT] getAccountEntityList
 
11:05:15,785 INFO  [STDOUT] getAccountEntityList
 
11:05:15,785 INFO  [STDOUT] showDetails
 
11:05:15,785 INFO  [STDOUT] selected account =3
 
11:05:15,805 INFO  [STDOUT] getAccountEntityList
 
11:05:15,805 INFO  [STDOUT] getAccountEntityList
 
11:05:15,825 INFO  [STDOUT] getAccountEntityList

est ce normal ???

[romaintaz]

En fait il ne s'agit pas d'un comportement étrange, les getters sont effectivement appelés plusieurs fois (savoir si 3 ou 15 fois est normal ou pas, ça reste à voir).
Cela vient du fonctionnement même de JSF, qui a parfois besoin de récupérer une donnée (via le getter) à différents moments du cycle de vie de la requête JSF.

Cela ne devrait normalement pas poser trop de problème à condition que tes getters soient de simples méthodes retournant une valeur / un objet. Initialiser des éléments(*), ou pire, faire du code métier dans un getter est forcément une mauvaise chose !

Initialiser des propriétés peut se faire dans d'autres endroits que le getter, par exemple :

• le constructeur du bean ;
• des blocs static dans ta classe ;
• à l'initialisation du bean (si tu utilises par exemple l'annotation @PostConstruct) ;
• pendant un événement (une action Java lancée par un action / actionListener)

(*) cela peut être acceptable dans certains cas, par exemple pour initialiser une liste d'objets de référence, de préférence avec un scope du bean différent de request. On aura alors un test qui vérifiera si la liste est déjà initialisée ou pas. Cela donne :

Code :

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

1
2
3
4
5
6
7
8
9
10
private List<SelectItem> countries = null;
 
public List<SelectItem> getCountries() {
    if (countries == null) {
        countries = new ArrayList<SelectItem>();
        countries.add(new SelectItem("FR", "France"));
        ...
    }
    return countries;
}

Dans cet exemple, la liste est initialisée au premier appel du getter, les autres appels ne faisant alors que retourner la liste déjà remplie.

[koack]

merci pour la reponse tres rapide, c'est vrai que mon getter est un peu fait a la va vite, voir meme horrible :p

Mais j'etais pressé de tester ma datatable. Je vais corriger tout ca.

Mais j'aurai encore une derniere question (je laisse de côté le nombre d'appel)pourquoi JSF fait appel au getter de la liste sur la methode showDetails de mon managedbean alors que j'ai testé avec ce code:
Il n'y a aucun rapport entre une liste de la datatable et un h:commandLink

Code :

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

1
2
3
4
5
6
 
....
public String showDetails {
   return null;
....
}

[tchize_]

D'abord, tout le formulaire va être exploré, ainsi que la datatable et le action, lors de la phase "applyrequestvalue". C'est dans cette phase que les inputs de la datatable seront remplie et que l'action visée sera enregistrée. Pour info, votre navigateur va envoyer, entre autres, à JSF: form1:Account:2:_idjsf31

avec "2" le numéro de la ligne où vous avez cliqué, et _idjsf31 l'id autogénéré pour votre action puisque vous n'en avez pas spécifié.

Ensuite, on va de nouveau tout parcourir dans la phase de validation et de correction
Enfin, si la validation est ok, l'action visée sera appelée
Un nouveau rendu sera effectué.

Donc, oui, il est normal que la datatable soit parcourue plusieurs fois. Et chaque parcours implique 1 getter sur la value + 1 ou plusieurs getter / ligne

[romaintaz]

Qu'entends tu par "cela n'a aucun rapport" ?

De toutes façons, ton commandLink va générer une requête HTTP, et comme il n'y a pas de navigation (i.e. l'utilisateur va rester sur la même page), JSF va devoir recréer toute la page. JSF devra donc regénérer l'intégralité de ton tableau, y compris les valeurs qui y sont contenues. Donc il faudra que JSF accède à ta liste de données, via le getter.

De plus, ton commandLink se situe dans le tableau. Or le datatable dispose de fonctionnalité avancée pour savoir sur quelle ligne du tableau (donc quel élément lié dans ta liste de données) cette action a été faite (voir le getRowData() des tableaux). Cela devrait expliquer quelques appels supplémentaires...

[koack]

Merci pour vos réponses je comprend un peu mieux pourquoi tout ces appels sur le getter.