Discussion :

[Bobble]

Bonjour,

je travail avec Tomcat 5.5, Java et JSP.
J'ai une page JSP contenant une table:

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
<h:dataTable id="logDetails" var="item"
    value="#{logDetails.items}" styleClass="standardTable"
    headerClass="standardTable_Header"
    footerClass="standardTable_Header"
    rowClasses="standardTable_Row1,standardTable_Row2"
    columnClasses="standardTable_Column,standardTable_Column,standardTable_ColumnCentered,standardTable_ColumnCentered,standardTable_ColumnCentered,standardTable_Column">
    <h:column>
        <f:facet name="header">
            <h:outputText value="Time" />
        </f:facet>
        <h:outputText value="#{item.CreationTime}" style="font-size: 10px" >
            <f:convertDateTime pattern="HH:mm:ss" />
        </h:outputText>
    </h:column>
.......

Cette table est "remplie" avec son bean Java:

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 Vector getItems() throws SQLException
{
    Connection connection = DBHelper.getDBConnection();
    try
    {
        PreparedStatement stmt = connection.prepareStatement(SQL_SELECT_LOG_DETAILS);
        stmt.setString(1, sessionID);
        ResultSet rs = stmt.executeQuery();
        return ResultSupport.toVector(rs);
    }
    finally
    {
        connection.close();
    }
}

et ma fonction Java, toVector:

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
public static Vector toVector(ResultSet rs) throws SQLException
{
    ResultSetMetaData rsmd = rs.getMetaData();
    int noOfColumns = rsmd.getColumnCount();
    String[] columnNames = new String[noOfColumns];
    for (int i = 1; i <= noOfColumns; i++)
    {
        columnNames[i-1] = rsmd.getColumnName(i);
    }
    Vector result = new Vector();
    while (rs.next())
    {
        SortedMap<String, Object> columnMap = new TreeMap<String, Object>(String.CASE_INSENSITIVE_ORDER);
        for (int i = 0; i < noOfColumns; i++)
        {
            Object value = null;  
            if (rsmd.getColumnType(i) == Types.CLOB)
            {
                value = rs.getString(i); // workaround for TEXT datatype
            }
            else
            {
                value = rs.getObject(i);
            }
            if (rs.wasNull())
            {
                value = null;
            }
            columnMap.put(columnNames[i-1], value);
        }
        result.add(columnMap);
    }
    return result;
}

Pour accéder à cette page j'ai un lien depuis une autre page. J'ai remarqué que chaque fois que je clique sur ce lien, la mémoire utilisé pas Tomcat (gestionnaire de taches) augment à chaque fois, elle ne rédiminue plus quand je change de page. Je crois que c'est ça la cause mon problème.
Si je clique sur ce lien plusieurs fois (genre utilisateur stressé qui veut vite ouvrir et qui ne sais pas que cliquer plusieurs fois ne sert à rien ) et que la table est grosse (~60 lignes) j'ai une erreur de "Java heap space".

Code :

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

1
2
3
4
5
6
7
8
9
10
11
javax.servlet.ServletException: Java heap space
    javax.faces.webapp.FacesServlet.service(FacesServlet.java:125)
    org.apache.myfaces.component.html.util.ExtensionsFilter.doFilter(ExtensionsFilter.java:112)
 
javax.faces.FacesException: Java heap space
 
org.apache.jasper.JasperException: Java heap space
 
javax.servlet.ServletException: Java heap space
 
java.lang.OutOfMemoryError: Java heap space

à partir de ce moment la, je ne peux presque plus rien ouvrir de mon site web, j'ai toujours cette erreur. A part des toutes petites tables (2lignes).

J'ai remarqué que si j'ouvre des pages contenant des tables plus petites (~40 lignes), j'ai une autre erreur:

Code :

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

1
2
3
4
5
6
7
8
9
javax.servlet.ServletException: org.apache.jasper.JasperException
    javax.faces.webapp.FacesServlet.service(FacesServlet.java:125)
    org.apache.myfaces.component.html.util.ExtensionsFilter.doFilter(ExtensionsFilter.java:112)
 
javax.faces.FacesException: org.apache.jasper.JasperException
 
org.apache.jasper.JasperException
 
java.lang.ArrayIndexOutOfBoundsException

Je peux vous donner la trace complète de la cause mère de cette erreur, si nécessaire. Et d'autres parties de code?

Ma question est: comment gérer ça? Comment éviter l'erreur, ou au moins éviter de devoir redémarrer Tomcat si cela arrive??

Je ne veux pas augmenter la taille de la mémoire alloué, en cas normal c'est suffisant...et le mieux serais que la mémoire utilisé par Tomcat n'augmente sans cesse...

J'espère que quelqu'un ici puisse m'éclairer un peut...

[karement]

tu peux déjà fermer tes statement et voir ce que ça donne

[sinok]

Et plus généralement la connexion aussi: http://java.developpez.com/faq/jdbc/...etureStatement

Et puis tant que tu y es tu pourras fermer le resultset que ça ne ferait aucun mal.

[Bobble]

Merci pour les réponses! Je suis en train de reprendre le code pour tout fermer et voir la différence, mais il me faudra un moment...

J'ai encore une question: quand on fais des

Code :

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

stmt.executeUpdate();

on dois fermer aussi le PreparedStatement, après?

merci

[Altenide]

non, on ferme le Resulset, la connexion et le Statement.

[tchize_]

oui

et eviter de faire une conneexion systématique dans ton getter. JSF va appeler plusieurs fois le get pour faire un seul affichage, donc si tu va chercher dans la DB à chaque fois, ça peux vite être lourd.

[Altenide]

une question dans le sujet, lorsque l'on ferme le Statement çà ne ferme pas le prépare PreparedStatement du même coup ??

[Bobble]

Merci tchize_ !

Mais, comment faire pour ne pas mettre ma connexion dans le getter? J'imagine que je devrais l'ouvrir au chargement de la page et la fermer à la fermeture de la page... est-ce qu'il y a une exemple quelque part? (je ne trouve pas)

Ou simplement avec un if sur mon ResultSupport (not null) ??

Ensuite, si j'ai bien compris, l'interface PreparedStatement étend Statement. C'est donc la "même chose", je ne comprend pas la question de Altenide...
je pense que ou on utilise un, ou l'autre. Juste?

Moi j'utilise des PreparedStatement partout...car très souvent j'ai des paramètres variables dans ma query (avec des ?)

Code :

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

1
2
PreparedStatement stmt = connection.prepareStatement(SQL_SELECT_LOG_DETAILS);
stmt.setString(1, sessionID);

Est-ce que la solution d'utiliser des PreparedStatement est la meilleure? Même si j'en ai plusieurs dans une même fonction? Ou dans certains cas c'est mieux de créer un seul Statement (mais plusieurs query ) ?

[tchize_]

stocker le résultat après le premier appel, tout simplement. Et mettre le bean en question en request scope.

[Bobble]

Désolée mais je crois qu'il me manque une petite case pour comprendre ta dernière réponse

Mon bean est déjà en request scope.

Si je comprend bien, je dois stocker mon

Code :

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

ResultSupport.toVector(rs)

dans une variable globale (ex. monItems). et faire simplement

Code :

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

return monItems

dans le getter.

Mais dans quelle partie du code dois-je implémenter la connexion avec le select pour que se soit exécutée uniquement une première fois?
Si je crée un constructeur de mon bean, lui, il est appelé implicitement uniquement au chargement de la page?

[tchize_]

au meme endroit:

Code :

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

1
2
3
4
if (monitem==null) {
   // récupérer monitem de la db
}
return monitem;

[Bobble]

ok

Merci!!!!!