Discussion :

[johannsan]

Bonjour.

J'ai un probleme simple avec hibernate et struts 2.

J'ai un objet Tarif qui contient une liste de tarifValeur :

Code :

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

1
2
3
4
5
6
7
8
9
<class name="cpte.hibernate.CpteTarif" table="CPTE_TARIF">
        ....
        <set name="cpteTarifvaleurs" table="CPTE_TARIFVALEUR" inverse="true" lazy="true" cascade="all" fetch="select">
            <key>
                <column name="TAR_ID" precision="22" scale="0" not-null="true" />
            </key>
            <one-to-many class="cpte.hibernate.CpteTarifvaleur" />
        </set>
    </class>

Sur la page jsp qui affiche la liste des tarifs, je peux cliquer sur un lien qui affiche le détail de celui-ci (dont ces valeurs).

Dans struts, je fais donc :

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
 
	public String loadTarif() throws Exception
    {
		HttpServletRequest request = (HttpServletRequest) ActionContext.getContext().get(ServletActionContext.HTTP_REQUEST);
		long tarifId = Long.parseLong(request.getParameter("tarif.tarId"));
		tarif = CpteTarifDao.getInstance().get(tarifId); 
		if(tarif==null)
		{
			addActionError(getText("error.baseDeDonnees"));
			return INPUT;
		}
 
		HibernateUtil.closeSession();
		return SUCCESS;
    }

puis dans la jsp qui suis, j'ai le code suivant :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
<c:forEach var="tarifValeur"
					items="${tarif.cpteTarifvaleurs}"
					varStatus="status">
					<tr>
						<td>${tarifValeur.valId}</td>
						<td><fmt:formatDate  pattern="dd/MM/yyyy"  value="${tarifValeur.valDateEffet}"/></td>
						<td><fmt:formatDate  pattern="dd/MM/yyyy"  value="${tarifValeur.valDateFin}"/></td>
						<td><fmt:formatNumber value="${tarifValeur.valMontant}" type="currency" currencySymbol="&euro;"/></td>
					</tr>
 
				</c:forEach>

Ce code affiche l'erreur : org.hibernate.LazyInitializationException: failed to lazily initialize a collection of role: cpte.hibernate.CpteTarif.cpteTarifvaleurs, could not initialize proxy - no Session
car la session est close (je la ferme dans loadTarif()).
Bien sur, si je ne ferme pas la session préalablement, ca fonctionne correctement, mais j'avais vu que ne pas fermer une session pouvait poser des problèmes lors d'update dans un précédent message sur le forum.

Doit-on mettre dans chaque jsp une ouverture de session hibernate en début et une fermeture en fin? je ne pense pas que c'est la solution non plus...
Je ne souhaite pas utiliser le mode "eager". Le chargement à la demande m'a l'air mieux car ca ne va pas tout charger dès le début.

OButterlin : dans un précédent message vous disiez :

Dans mon exemple, effectivement, il n'y a pas l'ouverture/fermeture de la session parce que c'est géré au niveau d'un filtre.
La session est ouverte en début de request et fermée en fin de response.

.
Comment faire cela sachant qu'il y a deja un filtre sur mon application :

Code :

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

1
2
3
4
5
6
7
8
9
    <filter>
        <filter-name>struts2</filter-name>
        <filter-class>org.apache.struts2.dispatcher.ng.filter.StrutsPrepareAndExecuteFilter</filter-class>
    </filter>
 
    <filter-mapping>
        <filter-name>struts2</filter-name>
        <url-pattern>/*</url-pattern>
    </filter-mapping>

Merci d'avance

Quelqu'un sait

[OButterlin]

Je ne connais pas struts2 mais j'étendrais la classe en redéfinissant la méthode doFilter(...)
Quelque chose comme 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
...
    public void doFilter(ServletRequest request, ServletResponse response, FilterChain chain) throws IOException, ServletException
    {
        try
        {
            // on ouvre éventuellement la session ici

            chain.doFilter(request, response);
        }
        catch (Throwable t)
        {
            System.out.println("doFilter error : " + t.toString());
        }
        finally
        {
            // on ferme la session ici
        }
    }
...

Mais bon, comme dit, je ne connais pas struts2 et je ne sais pas ce que fait ce filtre, donc, c'est sans garantie

[johannsan]

J'ai lu qu'on pouvais avoir deux filtres déclarés dans le sens des <filter-mapping>.

Donc quand vous dites :

j'étendrais la classe

, il suffit que je crée un deuxième filtre (placé en premier) avec le code donné qui ne fait qu'ouvrir et fermer la connexion hiberate.

A suivre...

[tchize_]

Le plus propre est d'avoir effectivement un deuxième filtre qui ouvre et ferme la session hibernate. Et tu mappe ce filtre sur les requête qui utilisent hibernate. Donc a priori sur struts


Par contre, etendre le filtre Struts => beurk, ça n'a aucun intérêt selon moi.

[OButterlin]

Le plus propre est d'avoir effectivement un deuxième filtre qui ouvre et ferme la session hibernate. Et tu mappe ce filtre sur les requête qui utilisent hibernate. Donc a priori sur struts

Par contre, etendre le filtre Struts => beurk, ça n'a aucun intérêt selon moi.

Je n'utilise pas struts2 donc je ne sais pas trop ce qu'est sensé faire le filtre mais en struts1, j'étendais le préprocesseur de struts, c'était bien plus intéressant qu'un simple filtre.

Ceci dit, oui, on tu peux créer un autre filtre et le placer devant, pas de soucis...

[johannsan]

Cela fonctionne en gérant un second filtre.

Par contre, j'ai une question sur (pas sur la même erreur), mais qui concerne le meme objet (Tarif et TarifValeur).

Je n'avais pas de problème avant l'utilisation d'Hibernate. Il doit donc y avoir une subtilité.
Avec Hibernate, le tarifvaleurs est un set, alors qu'avant, avec mes dao manuelles, je faisais un arrayList.

La page de modification est la suivante : formulaire sur le tarif modifiable et formulaire sur la liste des valeurs (non modifiable) mais qui s'affiche en dessous pour que la liste des valeurs du tarifs soit transmise lors du sumit :

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
 
			<s:form name="tarifForm" action="">
				<s:textfield name="tarif.tarId" readonly="true" 
					label="%{getText('label.tarif.id')}" />
				<s:textfield name="tarif.tarLibelle" required="true"
					label="%{getText('label.tarif.libelle')}" />
				<s:textfield name="tarif.tarNomArticle" required="true"
					label="%{getText('label.tarif.nomArticle')}" />
				<s:textfield name="tarif.tarGroupeMarchandise" required="true"
					label="%{getText('label.tarif.groupeMarchandise')}" />
 
				<table id="TarifValeur" class="tablesorter">
			<thead>...</thead>
			<tbody>
 
				<s:iterator  value="tarif.cpteTarifvaleurs" var="tarifValeur" status="status"  >
				<tr>
					<td><s:textfield theme="simple" cssClass="inputReadOnlyTransparent" readonly="true" name="tarif.cpteTarifvaleurs[%{#status.index}].valId" value="%{valId}"/></td>
					<td><s:textfield theme="simple" cssClass="inputReadOnlyTransparent" readonly="true" name="tarif.cpteTarifvaleurs[%{#status.index}].valDateEffet" value="%{valDateEffet}"/></td>
					<td><s:textfield theme="simple" cssClass="inputReadOnlyTransparent" readonly="true" name="tarif.cpteTarifvaleurs[%{#status.index}].valDateFin" value="%{valDateFin}"/></td>
					<td><s:textfield theme="simple" cssClass="inputReadOnlyTransparent" readonly="true" name="tarif.cpteTarifvaleurs[%{#status.index}].valMontant" value="%{valMontant}"/></td>
					<td>
				</tr>
				</s:iterator>

Les valeurs du tarifs s'affichent correctement, mais à la validation du formulaire, j'ai des erreurs sur tous les champs du tableau :
2014-03-07 09:45:56,577 WARN com.opensymphony.xwork2.ognl.OgnlValueStack - Error setting expression 'tarif.cpteTarifvaleurs[0].valId' with value '[Ljava.lang.String;@65919c1f'
ognl.NoSuchPropertyException: java.util.HashSet.0

Cela se comprends car l'id est un long, les dates sont des dates et le montant est un bigDecimal donc pas des String.

Pour votre information, je suis obligé de mettre la liste dans un textField car si je fais un tableau basique, les champs sont perdu si la validation du formulaire ci-dessus (via les fichier_validation.xml) retourne un fieldError.

Cdt.