application très lente

**hamham** · 10/01/2007, 15h35

Bonjour à tous

je fais actuellement une application en java, reliée à une base de données.
L'utilisateur passe par des interfaces graphiques.

A un certains moment, l'application se bloque et met beaucoup de temps à répondre. A cet endroit, le programme doit faire quelques requêtes SQL.
Pensez vous que ce sont ces requêtes qui ralentissent autant l'application ou le problème peut il venir d'ailleurs.

Merci
Bye

**qdqdfqfdqdxcwcrzsdfw** · 10/01/2007, 15h40

essaye de faire un system.out.println() a la place de la requete.
Si c'est plus court, est ce que ta requete s'effectue apres un long temps d'execution ou elle plante?
Si elle plante essaye de l'executer dans ton gestionnaire de base

**adiGuba** · 10/01/2007, 15h44

Salut,

Avec si peu de détail ce n'est pas évident, mais je parierais sur un blocage de l'EDT (lire Threads et performance avec Swing par gfx)...

Quel type d'UI tu utilises ? Swing ? AWT ? SWT ?
A quel moment le programme se bloque-t-il ? Toujours dans les mêmes fonctionnalité ou de manière aléatoire ? Si c'est toujours au même endroit il serait utile de poster un peu de code...

a++

in · 10/01/2007, 15h47

dure de te répondre avec si peux de détail.

Tes requetes sont longues à exécuter sur la bdd (sans passer par jdbc) ?

est ce que ton programme attend les resultats pour continuer ou alors c'est dans des threads à part ?

Cette base est accédée par plein de monde en meme temps ?

La base est locale, sur un réseau local, sur le net ?

Enfin bref ... ça peut dépendre de plusieurs paramètres.

Si ce sont les requetes qui sont longues essaie de les optimiser (suivant la bdd que tu utilise).

Au fait c'est quel genre de requete ? insert, update, select ?
Parce que n'oublies pas que tu peux appeler des procédures stockées sur la base de donnée. Tu peux aussi utiliser des traitements par lot (batch) mixés avec des preparedStatement ...
Précise un peu ce point : c'est quoi tes requetes ? Comment tu les execute ?

Sinon, solution de substitution, si il y a des ralentissement essaie de mettre une barre de progression ou un truc du genre pour que l'utilisateur sache que ça travaille et ai moins l'impression que c'est bloqué ... C'est toujours mieux

**®om** · 10/01/2007, 19h48

Envoyé par adiGuba

Salut,

Avec si peu de détail ce n'est pas évident, mais je parierais sur un blocage de l'EDT (lire Threads et performance avec Swing par gfx)...

Quel type d'UI tu utilises ? Swing ? AWT ? SWT ?
A quel moment le programme se bloque-t-il ? Toujours dans les mêmes fonctionnalité ou de manière aléatoire ? Si c'est toujours au même endroit il serait utile de poster un peu de code...

a++

Je pense aussi que ça doit être ça vu la "petite" description du problème...

**hamham** · 11/01/2007, 12h54

Deja merci a tous de m'avoir répondu.

Je vais essayer de répondre à toutes vos questions, mais je ne pense pas pouvoir mettre le code de cette page en entier il serait trop long.

Tout d'abords, j'ai encore un peu de mal à faire la différence entre AWT et SWING, mais je ne pense pas que le problème vienne de la. Je pense que ça vient de cette requête :

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
 
    		Statement st=conn.createStatement();
 
    		String query1 = "SELECT Count(*) as nbr FROM film";
		ResultSet rs1 = st.executeQuery(query1);
	   		while (rs1.next())
	   			{film = new String[rs1.getInt("nbr")];}
    		int i = 0;
 
 
    		String query= "SELECT DISTINCT Titre FROM film F, joue J, acteur Ac, appartient Ap, categorie C," +
    				"realisateur Re, realise R";
    		if (paramfilm!=null)
    		{
    			query+= " WHERE (Titre LIKE '% "+paramfilm+" %' OR Titre LIKE '"+paramfilm+" %' OR Titre LIKE '% "+paramfilm+"' AND Titre REGEXP '[[:<:]]tout[[:>:]]') ";
        		i = 1;
    		}
    		if (paramact!=null)
    			{if (i==1){query+=" and";} else {query+=" WHERE";}
    			query+= " CONCAT(PreAct,' ',NomAct) LIKE '"+paramact+"' and Ac.IdAct= J.IdAct and J.IdFilm = F.IdFilm";
    			i = 1;}
    		if (paramreal!=null)
    		{
    			if (i==1){query+=" and";} else {query+=" WHERE";}
    			query+=" CONCAT(PreReal,' ',NomReal) LIKE '"+paramreal+"' and Re.IdReal=R.IdReal and R.IdFilm=F.IdFilm";
    			i=1;
    		}
    		if(paramcat!=null)
    		{
    			if (i==1){query+=" and";} else {query+=" WHERE";}
    			query+=" LibelCat LIKE '"+paramcat+"' and C.IdCat=Ap.IdCat and Ap.IdFilm=F.IdFilm";
    		}
    		query+=" ORDER BY Titre;";
 
    		i=0;
    	        ResultSet rs = st.executeQuery(query);
 
    	        i=0;
    	        while(rs.next())
    	        {
    	        	film[i]=rs.getString("Titre");
    	        	System.out.println (film[i]);
    	        	i++;
    	        }
    	        st.close();
    	}
    	catch (SQLException e){
    		System.out.println("Gloups "+e);
    	}
 
 
 
    	tFilm= new JList(film);
    	tFilm.setVisibleRowCount(8);
    	conteneur.add(new JScrollPane(tFilm));

Les paramflm, paramact.. sont des String qui viennent d'un autre formulaire.

pour information, ma base est en local, et le programme attend tFilm pour afficher la page. J'ai essayer avec un preparedStatement, mais je n'y suis pas arrivée.

je vous remercie

**adiGuba** · 11/01/2007, 13h20

Quelques remarques :

As-tu testé les requêtes via les outils de ta BD pour voir le temps que cela prend ?
Tu ne fermes pas le premier ResultSet (pas bien !). De plus il faudrait le faire dans un bloc finally...
Si la requête SQL prend un certain temps et qu'elle est appelé dans une des méthodes évènementielle (comme actionPerformed() par exemple), alors elle est exécuté dans l'EDT et cela bloquera l'interface graphique...

a++

**OButterlin** · 11/01/2007, 13h37

Si je peux me permettre, pourquoi tu fais :
...Titre like '%mot%' or Titre like '%mot' or Titre like 'mot%'... ?
le premier like te les renvois tous...
Ce que tu demandes (traduit en français), c'est : donne moi tout ce qui contient 'mot' ou tout ce qui termine par 'mot' ou tout ce qui commence par 'mot'
Ensuit, les paramact, paramreal et paramcat contiennent les '%' ? Parce que sinon, mieux vaut mettre '=' que 'like'...
A part ça, tu peux mettre des points d'arrêt dans ton traitement et voir où ça coince...

A+

**xv-mnt** · 12/01/2007, 08h48

Cette requete ne peut pas marcher rapidement dans la plupart des cas. Tu écris le code suivant :

Code :

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

1
2
    		String query= "SELECT DISTINCT Titre FROM film F, joue J, acteur Ac, appartient Ap, categorie C," +
    				"realisateur Re, realise R";

Donc, quels que soient les paramètres passés, tu vas réaliser des jointures sur 7 tables. Or, tu ne fais que des jointures partielles suivant les paramètres passés.
Tu auras donc des produits cartésiens sur potentiellement 7 tables ! Derrière, tu fais un distinct qui peut être potentiellement couteux.
Ce que je te conseille, c'est de rajouter les jointures entre les 7 tables SYSTEMATIQUEMENT à la fin ou au début de ta requête. Les perfs devraient augmenter drastiquement !

**OButterlin** · 12/01/2007, 09h18

Envoyé par xv-mnt

Donc, quels que soient les paramètres passés, tu vas réaliser des jointures sur 7 tables. Or, tu ne fais que des jointures partielles suivant les paramètres passés.
Tu auras donc des produits cartésiens sur potentiellement 7 tables ! Derrière, tu fais un distinct qui peut être potentiellement couteux.

C'est certain... ça m'avait fait tiquer aussi, mais je n'avais pas percuté...
Pour rajouter une couche, en fonction des bases de données, tu peux augmenter les performances en utilisant des 'join' ou 'left join' plutôt que ce genre de jointure

A+

**hamham** · 12/01/2007, 15h13

déja merci à tous,

j'ai finalement opté pour le code suivant, donnez moi votre avis :

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
int i =0;
String querysuite="";
    		String query= "SELECT DISTINCT Titre" +
    				" FROM film F";
    		if (paramfilm!=null)
    		{
    			querysuite+= " WHERE (Titre LIKE '% "+paramfilm+" %' OR Titre LIKE '"+paramfilm+" %' OR Titre LIKE '% "+paramfilm+"' AND Titre REGEXP '[[:<:]]tout[[:>:]]') ";
    			i=1;
    		}	
    		if (paramact!=null)
    		{	
    			if (i==0)querysuite+=" WHERE"; else querysuite+=" AND";
    			query+=", acteur Ac, joue J";
    			querysuite+= " CONCAT(PreAct,' ',NomAct) LIKE '"+paramact+"' AND Ac.IdAct=J.IdAct" +
    				" AND J.IdFilm=F.IdFilm";
    		}
    		if (paramreal!=null)
    		{	
    			if (i==0)querysuite+=" WHERE"; else querysuite+=" AND";
    			query+=", realisateur Re, realise R";
    			querysuite+=" CONCAT(PreReal,' ',NomReal) LIKE '"+paramreal+"' AND Re.IdReal=R.IdReal " +
    				" AND R.IdFilm=F.IdFilm";
    		}
    		if(paramcat!=null)
    		{	
    			if (i==0)querysuite+=" WHERE"; else querysuite+=" AND";
    			query+=", appartient Ap, categorie C"; 
    			querysuite+=" LibelCat LIKE '"+paramcat+"'AND C.IdCat=Ap.IdCat " +
    				" AND Ap.IdFilm=F.IdFilm";
    		}
    		query+=querysuite+" ORDER BY Titre;";
 
    		i=0;
    	        ResultSet rs = st.executeQuery(query);
 
    	        i=0;
    	        while(rs.next())
    	        {
    	        	film[i]=rs.getString("Titre");
    	        	System.out.println (film[i]);
    	        	i++;
    	        }
    	        st.close();
    	        rs.close();

de plus, comme je dois beaucoup travailler sur java, si quelqu'un si connait vraiment et qu'il veut bien répondre à mes questions, qu'il m'envoie un message privé. Merci

Bye

**schniouf** · 12/01/2007, 15h38

Ce que tu peux faire, c'est que partout où tu le jugeras nécessaire tu faits un System.out.println( un chiffre ) en partant de 1 à X et tu vois entre quels chiffres c'est le plus long... Une fois que t'as trouvé une zone qui est lente, tu refais pareil au sein de cette même zone, etc. En gros, tu resserres l'étau sur la ou les lignes qui ralentissent !

**xv-mnt** · 12/01/2007, 15h40

a priori ça devrait être mieux puisque tu n'utilise que les tables nécessaires à chaque fois. A tester chez toi pour les perfs.

Remarque : Pour la lisibilité, tu peux remplacer

Code :

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

if (i==0)querysuite+=" WHERE"; else querysuite+=" AND";

par

Code :

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

querysuite+= (i==0) ? " WHERE" : " AND";

**hamham** · 17/01/2007, 14h41

merci pour tous vos conseils et a très vite
bye

application très lente

Java

Discussions similaires

Partager

Partager