Discussion :

[theniaky]

Bonjour à tous !

Je suis sur un projet pour mon école, et je dois utiliser une SGDB Access...
Lors d'un traitement d'un tableau dynamique croisé, j'envoie des fois plusieurs centaines de requêtes, et j'ai un erreur qui ressemble à une saturation du driver ODBC :

Code :

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

java.sql.SQLException: [Microsoft][Pilote ODBC Microsoft Access] Trop de tâches client.

J'ai tenté des "tempo" entre chaque requête avec un thread :

Code :

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

1
2
3
4
5
6
try{
Thread.sleep(2000);
}
catch (InterruptedException e) {
e.printStackTrace();
}

Ca n'a malheureusement rien changé et ça plante toujours... Je précise que tout fonctionne parfaitement lorsqu'il n'y a pas beaucoup de requêtes à envoyer.
Je précise également que je ne malheureusement pas passer à une SGBD plus solide du genre mysql...

Merci d'avance !

[e1lauren]

Tu utilise les PreparedStatment pour tes requetes ?

[remika]

tu pourrais mettre un bout de code avec une requête stp ?

[theniaky]

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
 
        try
        {
            // On se connecte à la base de données
            Class.forName(conn);
            Connection connexion = DriverManager.getConnection(url,user,pass);
 
            // On récupère tous les enregistrements qui contient les deux variables
            Statement result = connexion.createStatement();
            result.executeQuery("SELECT "+ Variable1 +" FROM donnees WHERE " + Variable1 + " = '" + ValeurVariable1 + "' AND " + Variable2 + " = '" + ValeurVariable2 + "'");
            ResultSet ligne = result.getResultSet();
 
            int i=0;
            while(ligne.next())
            {
                i++;
            }
 
            ligne.close();
            result.close();
            connexion.close();
 
            return i;
        }
        catch (Exception e) {e.printStackTrace();System.err.println("Erreur lors du traitement du tableau dynamique");return 0;}

Sachant que cette fonction est relancée en boucle suivant les besoin (plusieurs dixaine, voire centaines de fois à la suite)

[EDIT] J'oubliais de préciser les champs qui précèdent tout ça :

Code :

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

1
2
3
4
5
 
    private String connexion = "sun.jdbc.odbc.JdbcOdbcDriver";
    String url= "jdbc:odbc:Driver={Microsoft Access Driver(*.mdb)};DBQ=./Bretagne.mdb";
    private String user = "";
    private String pass = "";

[remika]

normalement ça ne devrait pas être un problème de lancer 20000 requêtes comme tu fais ça en boucle et que tu close (à part en cas d'exception)

sinon utilise plutôt des PreparedStatement (en particulier pour les String/VARCHAR)

[theniaky]

Ben ouais je comprends pas trop...
Pourtant le message d'erreur est très clair...

[zais_ethael]

Si ton programme est multithread c'est surement ca le hic (access doit être limité à 5 ou 6 connexion maxi je crois).
Sinon, je te conseillerais aussi les PreparedStatement, c'est un bien meilleur réflexe à avoir. Il est encore mieux de conserver ces objets entre 2 requètes, le gain de vitesse est alors tout sauf négligeable.
Le must restant bien sur d'utiliser un pool de connexions, mais je me doute que tu n'as pas trop envie de te casser la tête avec ca pour un dossier.

[theniaky]

Je viens de tenter le preparedStatement mais ça me fait la même erreur... Peut-être que je ne l'ai pas bien utilisée : voici le code :

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
 
String sql = "SELECT " + Variable1 + " FROM donnees WHERE [" + Variable1 + "] = ? AND [" + Variable2 + "] = ?";
PreparedStatement preparedStatement = connexion.prepareStatement(sql);
 
preparedStatement.setString(1,ValeurVariable1);
preparedStatement.setString(2,ValeurVariable2);
 
ResultSet ligne = preparedStatement.executeQuery();
 
int i=0;
while(ligne.next())
{
      i++;
}
 
ligne.close();
preparedStatement.close();
connexion.close();
 
return i;

[xfacq]

Salut,

Essayes de mettre ton objet Connexion en variable membre de la classe
et d'utiliser la même connexion pour faire plusieurs PreparedStatements.
Là, le problème c'est que tu te connectes et déconnectes autant de fois
que tu appelles ta méthode... Alors si tu enchaines ca rapidement, le driver
à peut être du mal au bout d'un moment !

Sinon, regardes pour mettre en place un pool de connexion...
a+
Xavier

[remika]

rien à voir mais pourquoi tu fais pas un COUNT directement dans ta requête ?

[zais_ethael]

Tu ne sembles pas avoir compris le principe du PreparedStatement.
Si tu recomposes à chaque fois ta requete ca fout le bordel. Il faut utiliser des requetes statiques genre "select id from tableClients where nom=?", cette requete sera ensuite précompilée et pourra être réutilisée en chageant simplement les paramètres.
A part ca, xfacq n'a peut-être pas tort. Même si tu fermes correctement tes connexions, on pourrait imaginer que le driver odbc prenne un certain temps avant de s'en rendre compte. Comme ta requete s'exécute en boucle, tu créerais alors des connexions sans laisser le temps aux anciennes de se fermer.
Essaie de conserver ta connexion, mais n'oublie pas de fermer ton resultset. Sinon pool de connexions++.

[remika]

Il faut utiliser des requetes statiques genre "select id from tableClients where nom=?"

ba si ta restriction se fait sur des champs que tu as en paramètre par exemple (ce qui est apparemment le cas), il faut bien que tu construises ta String de requête toi-même

[theniaky]

En tout cas, merci de vous pencher sur le problème !

Je bosse sur mon projet quasiement tout l'après midi donc je vais essayer de ne pas fermer la connexion... Je vais me renseigner aussi sur la fonction COUNT sql que je ne connais pas.

Et si ça continue de merder, je vous demanderai certainement des conseils sur le "pool de connexions" dont je n'ai jamais entendu parlé !

Je vous tiens au courant

[theniaky]

Et bien je vous remercie tous, mon souci est réglé : j'ai utilisé un preparedStatement en faisant tout mon traitement sans fermer la connexion.

Merci encore !