[JAVA / Out Of Memory] Aide pour optimiser du code

**shaun_the_sheep** · 26/01/2007, 08h05

Bonjour,

J'ai développé une classe permettant de parcourir un répertoire (sur un serveur), dans lequel est stocké entre 2000 à 3000 photos en vu de les redimenssionner.

voici mon 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
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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
import java.awt.Color;
import java.awt.Graphics2D;
import java.awt.RenderingHints;
import java.awt.geom.AffineTransform;
import java.awt.image.AffineTransformOp;
import java.awt.image.BufferedImage;
import java.io.File;
import java.io.IOException;
import javax.imageio.ImageIO;
 
public class TraitementsImage{   
 
 /**
 
 * Type de compression BMP
 
 */
 
   public final static String IMAGE_TYPE_BMP = "BMP";
 
 
 
 /**
 
 * Type de compression WBMP
 
 */
 
   public final static String IMAGE_TYPE_WBMP = "WBMP";
 
 
 
 /**
 
 * Type de compression JPEG
 
 */
 
   public final static String IMAGE_TYPE_JPEG = "JPEG";
 
 
 
 /**
 
 * Type de compression PNG
 
 */
 
   public final static String IMAGE_TYPE_PNG = "PNG";
 
 
 
 
 
 /**
 
 * Conversion sans modification de taille d'un fichier
 
 * @param fichierSource File
 
 * @param fichierDestination File
 
 * @param compressionType String
 
 * @throws IOException 
 
 */
  public static String cvtImage(File fichierSource, File fichierDestination, String compressionType)
 
  {
 
 
 
    // buffer
 
    BufferedImage buf = null;
 
    BufferedImage bufFinal = null;
 
 
 
    // récuperation de l''image dans le buffer
 
    try {
 
    	buf = ImageIO.read(fichierSource);
 
      	}
 
    catch (IOException ex)
 
    	{
 
    	ex.printStackTrace();
        return "non ok";
 
    	}
 
 
 
    //calcul de l''echelle
 
    int width=buf.getWidth();
 
    int height=buf.getHeight();
 
    int widthMsq=130;
 
    int heightMsq=170;
 
    double echelle=0;
 
 
 
    double echelleW=(double) widthMsq / width;
 
    double echelleH=(double) heightMsq / height;
 
    if(echelleW<echelleH) echelle=echelleW; else echelle=echelleH;
 
    echelle=(Math.round(echelle*Math.pow(10,2)) )/ (Math.pow(10,2));
 
 
 
    // CrÃ¯Â¿Âœtion du buffer final widthMsq, heightMsq buf.getWidth(), buf.getHeight()
 
    bufFinal = new BufferedImage((int) (buf.getWidth()* echelle), (int)(buf.getHeight()*echelle),buf.getType());//.TYPE_INT_RGB); // masque
 
 
 
    // Redimensionnement de l''image si volonté
 
    Graphics2D g = (Graphics2D) bufFinal.getGraphics();
 
    g.setRenderingHint(RenderingHints.KEY_INTERPOLATION,RenderingHints.VALUE_INTERPOLATION_BILINEAR);
 
    g.drawImage(buf, 0,0, (int) (buf.getWidth()* echelle), (int)(buf.getHeight()*echelle), null);
 
 
 
    g.dispose();
 
 
 
    // Ecriture du fichier destination
 
    try {
 
    	ImageIO.write(bufFinal, compressionType,fichierDestination);
 
    	}
 
    catch (IOException e)
 
    	{
 
    	e.printStackTrace();
 
         return "non ok";
 
    	}
 
 
 
 
    return fichierDestination.toString();
 
  }
 
 
 
  public  static String lstImages(String strRepertoire,String destRepertoire)
 
  {     	
 
	 String resultat=""; 
 
	 File repertoire=new File(strRepertoire);
 
	 File[] listFile = repertoire.listFiles();
 
	 if(listFile !=null)
 
	       for ( int i = 0; i < listFile.length; i++) 
 
	       {
 
               File file = listFile[i];
 
               if( file.toString().substring(file.toString().length()-3,file.toString().length()).equals("jpg"))
 
               {     resultat=cvtImage(file,file,IMAGE_TYPE_JPEG);
 
                     File destination = new File(destRepertoire+"/"+file.toString().split(strRepertoire)[1]);
 
                     file.renameTo(destination);
 
 
               }
 
 
 
           } 
 
 
 
 
 
	  return "ok";    
  }  
}

Le code fonctionne correctement, seulement il me fait un java out of memory après avoir redimensionné 700 photos, je supose que le nombre d'objets crées y soit pour quelque chose.

Je cherche donc a optimiser mon code pour ne plus avoir cette erreur.

Quelqu'un a t'il une idée?

Merci

**®om** · 26/01/2007, 08h23

Pour ton pb, je n'ai rien vu qui pouvait l'expliquer.

Par contre:

Code :

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

echelle=(Math.round(echelle*Math.pow(10,2)) )/ (Math.pow(10,2));

fais plutôt:

Code :

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

echelle = ((int)(echelle * 100)) / 100;

Ça sert à rien de calculer 10^2 au runtime, surtout que c'est des puissances de nombre flottants qui passent par l'exponentielle...

**chtig** · 26/01/2007, 14h34

Pourrais-tu essayer de mettre à la fin de ta boucle :

Code :

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

1
2
3
 
Runtime runtime = Runtime.getRuntime();
System.out.println("Total memory : "+runtime.totalMemory()+"\tfree : "+runtime.freeMemory());

Ca devrait donner une indication de comment ça se passe (genre il ne passe pas le garbage collector, ou il le passe mais à chaque fois il récupère moins etc ...)

A essayer (ça pourrait se tuner) de mettre un simple

Code :

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

1
2
 
System.gc();

A la fin de ta boucle. Ca va te tuer les perfos, mais après on pourrait par exemple le faire toutes les 100 images ou quelquechose du genre ...

**thibaut** · 26/01/2007, 14h54

Envoyé par chtig

Code :

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

1
2
 
System.gc();

Non c'est une mavaise idée, car ça va risque de ralentir considérablement ton appli sans te garantir que les ressources seront libérées.

A+

**chtig** · 26/01/2007, 15h21

Faut lire avant de critiquer .... J'ai dit explicitement que ça allait tuer les perfos. Mais il s'agit là d'analyser le problème pour le moment, pas de le résoudre....
Maintenant si tu as de meilleures idées, je suis preneur

Etre constructif, encore et toujours c'est important

**thibaut** · 26/01/2007, 15h32

Envoyé par chtig

Faut lire avant de critiquer .... J'ai dit explicitement que ça allait tuer les perfos. Mais il s'agit là d'analyser le problème pour le moment, pas de le résoudre....
Maintenant si tu as de meilleures idées, je suis preneur

Etre constructif, encore et toujours c'est important

Certes ça va tuer les performances, mais qui plus est tu n'es pas certain que ça va effectivement libérer la mémoire.

**vbrabant** · 02/02/2007, 22h44

Envoyé par beegood

Bonjour,

J'ai développé une classe permettant de parcourir un répertoire (sur un serveur), dans lequel est stocké entre 2000 à 3000 photos en vu de les redimenssionner.

voici mon 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
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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
import java.awt.Color;
import java.awt.Graphics2D;
import java.awt.RenderingHints;
import java.awt.geom.AffineTransform;
import java.awt.image.AffineTransformOp;
import java.awt.image.BufferedImage;
import java.io.File;
import java.io.IOException;
import javax.imageio.ImageIO;
 
public class TraitementsImage{   
 
 /**
 
 * Type de compression BMP
 
 */
 
   public final static String IMAGE_TYPE_BMP = "BMP";
 
 
 
 /**
 
 * Type de compression WBMP
 
 */
 
   public final static String IMAGE_TYPE_WBMP = "WBMP";
 
 
 
 /**
 
 * Type de compression JPEG
 
 */
 
   public final static String IMAGE_TYPE_JPEG = "JPEG";
 
 
 
 /**
 
 * Type de compression PNG
 
 */
 
   public final static String IMAGE_TYPE_PNG = "PNG";
 
 
 
 
 
 /**
 
 * Conversion sans modification de taille d'un fichier
 
 * @param fichierSource File
 
 * @param fichierDestination File
 
 * @param compressionType String
 
 * @throws IOException 
 
 */
  public static String cvtImage(File fichierSource, File fichierDestination, String compressionType)
 
  {
 
 
 
    // buffer
 
    BufferedImage buf = null;
 
    BufferedImage bufFinal = null;
 
 
 
    // récuperation de l''image dans le buffer
 
    try {
 
    	buf = ImageIO.read(fichierSource);
 
      	}
 
    catch (IOException ex)
 
    	{
 
    	ex.printStackTrace();
        return "non ok";
 
    	}
 
 
 
    //calcul de l''echelle
 
    int width=buf.getWidth();
 
    int height=buf.getHeight();
 
    int widthMsq=130;
 
    int heightMsq=170;
 
    double echelle=0;
 
 
 
    double echelleW=(double) widthMsq / width;
 
    double echelleH=(double) heightMsq / height;
 
    if(echelleW<echelleH) echelle=echelleW; else echelle=echelleH;
 
    echelle=(Math.round(echelle*Math.pow(10,2)) )/ (Math.pow(10,2));
 
 
 
    // CrÃ¯Â¿Âœtion du buffer final widthMsq, heightMsq buf.getWidth(), buf.getHeight()
 
    bufFinal = new BufferedImage((int) (buf.getWidth()* echelle), (int)(buf.getHeight()*echelle),buf.getType());//.TYPE_INT_RGB); // masque
 
 
 
    // Redimensionnement de l''image si volonté
 
    Graphics2D g = (Graphics2D) bufFinal.getGraphics();
 
    g.setRenderingHint(RenderingHints.KEY_INTERPOLATION,RenderingHints.VALUE_INTERPOLATION_BILINEAR);
 
    g.drawImage(buf, 0,0, (int) (buf.getWidth()* echelle), (int)(buf.getHeight()*echelle), null);
 
 
 
    g.dispose();
 
 
 
    // Ecriture du fichier destination
 
    try {
 
    	ImageIO.write(bufFinal, compressionType,fichierDestination);
 
    	}
 
    catch (IOException e)
 
    	{
 
    	e.printStackTrace();
 
         return "non ok";
 
    	}
 
 
 
 
    return fichierDestination.toString();
 
  }
 
 
 
  public  static String lstImages(String strRepertoire,String destRepertoire)
 
  {     	
 
	 String resultat=""; 
 
	 File repertoire=new File(strRepertoire);
 
	 File[] listFile = repertoire.listFiles();
 
	 if(listFile !=null)
 
	       for ( int i = 0; i < listFile.length; i++) 
 
	       {
 
               File file = listFile[i];
 
               if( file.toString().substring(file.toString().length()-3,file.toString().length()).equals("jpg"))
 
               {     resultat=cvtImage(file,file,IMAGE_TYPE_JPEG);
 
                     File destination = new File(destRepertoire+"/"+file.toString().split(strRepertoire)[1]);
 
                     file.renameTo(destination);
 
 
               }
 
 
 
           } 
 
 
 
 
 
	  return "ok";    
  }  
}

Le code fonctionne correctement, seulement il me fait un java out of memory après avoir redimensionné 700 photos, je supose que le nombre d'objets crées y soit pour quelque chose.

Je cherche donc a optimiser mon code pour ne plus avoir cette erreur.

Quelqu'un a t'il une idée?

Merci

Oui.
2 possibilités: Utilise le profiler de NetBeans, et tu verras tout de suite où se situe le problème.
Ou alors JConsole. Qui est pas mal aussi pour cela.

J'ai fait tourner le programme en local sur mon Pc avec JCOnsole, et j'ai rien vu d'anormal.

Tout se libère bien comme il faut. Peut-être qu'une photo exigeait plus de mémoire qu'il ne pouvait donner. Mais en tout cas, chez moi, pas de problème.

Vincent

**billynirvana** · 06/02/2007, 09h58

Idem. Ton code est correct. Augmente à la rigueur la taille de la JVM au lancement de ton application .

[JAVA / Out Of Memory] Aide pour optimiser du code

Java

Discussions similaires

Partager

Partager