Discussion :

[blaster681]

Bonsoir,

J'ai une augmantation importante de la memoire ram avec ce 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
 
 [...]
 
out = new ObjectOutputStream(socket.getOutputStream());
in = new ObjectInputStream(socket.getInputStream());
 
while (true) {  	
 
ImageIcon img = (ImageIcon) in.readObject();
 
 try {
 
canvas.setBackgroundImage(makeSWTImage(d,img.getImage()));	
 
 } catch (OutOfMemoryError e) {
 
System.gc();
} catch (Exception e) {
// TODO Auto-generated catch block
										e.printStackTrace();
 
  }
);			        	        
 
 
 }

Je fais des captures d'ecran dans une boucle while... La mémoire augemente petit a petit... Je crois avoir trouver le coupable, cette fonction: makeSWTImage(d, img.getImage()) //elle permet de convertire une image AWT en une image SWT

Voici cette fameuse fonction:

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
 
 org.eclipse.swt.graphics.Image makeSWTImage(Display display, java.awt.Image ai)
	    throws Exception
	  {
	    int width = ai.getWidth(null);
	    int height = ai.getHeight(null);
	    BufferedImage bufferedImage = null;
	    bufferedImage =  new BufferedImage(width, height, BufferedImage.TYPE_INT_RGB);
	    Graphics2D g2d = null;
	    	g2d = bufferedImage.createGraphics();
	    g2d.drawImage(ai, 0, 0, null);
	    g2d.dispose();
	    int[] data = null;
	      data = ((DataBufferInt)bufferedImage.getData().getDataBuffer())
	      .getData();
	    ImageData imageData = null;
	     imageData = new ImageData(width, height, 24,
	        new PaletteData(0xFF0000, 0x00FF00, 0x0000FF));
	    imageData.setPixels(0, 0, data.length, data, 0);
	    org.eclipse.swt.graphics.Image swtImage = new org.eclipse.swt.graphics.Image(display, imageData);
 
	    return swtImage;
	  }

Apres plusieurs jours de recherche et de galère j'espère que vous pourrez me donner un petit coup de main ^^

Merci.

Olivier

[CheryBen]

Bonjour, à priori, tu ne fermes pas les streams que tu ouvres (out et in). FAQ : Comment libérer proprement les ressources (ou comment utiliser proprement les bloc try/finally) ?

[dinobogan]

Le bout de code

Code :

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

1
2
3
4
catch (OutOfMemoryError e) {
 
System.gc();
}

est parfaitement inutile.

[we.are.the.storm]

Il faut également éviter de créer des variables à l'intérieur des boucles (même temporaires)

Code :

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

1
2
3
4
while (true) {
    ImageIcon img = (ImageIcon) in.readObject(); //va allouer un nouvel espace mémoire pour stocker un ImageIcon à chaque passage dans la boucle. 
    ...
}

La manière vue ci-dessus va donner beaucoup de travail inutile au GarbageCollector. Il est recommandé de procéder à la création de variable en dehors des boucles, comme ceci :

Code :

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

1
2
3
4
5
6
 
ImageIcon img = null;
while (true) {
    img = (ImageIcon) in.readObject(); //va utiliser toujours le même espace mémoire, alloué avant l'entrée dans la boucle. 
    ...
}

De la même manière comme la méthode makeSWTImage(...) est appelée dans la boucle il faut éviter au possible d'y créer des variables temporaires. BufferedImage, Graphics2D, ImageData, etc pourrait être stockées en tant qu'attribut de la classe qui contient la méthode. (Quitte à ne pas y mettre d'accésseur public puisque c'est pour un usage d'optimisation interne à la classe).

[jowo]

Bonjour,

Il faut également éviter de créer des variables à l'intérieur des boucles (même temporaires)

Code :

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

1
2
3
4
while (true) {
    ImageIcon img = (ImageIcon) in.readObject(); //va allouer un nouvel espace mémoire pour stocker un ImageIcon à chaque passage dans la boucle. 
    ...
}

La manière vue ci-dessus va donner beaucoup de travail inutile au GarbageCollector. Il est recommandé de procéder à la création de variable en dehors des boucles, comme ceci :

Code :

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

1
2
3
4
5
6
 
ImageIcon img = null;
while (true) {
    img = (ImageIcon) in.readObject(); //va utiliser toujours le même espace mémoire, alloué avant l'entrée dans la boucle. 
    ...
}

De la même manière comme la méthode makeSWTImage(...) est appelée dans la boucle il faut éviter au possible d'y créer des variables temporaires. BufferedImage, Graphics2D, ImageData, etc pourrait être stockées en tant qu'attribut de la classe qui contient la méthode. (Quitte à ne pas y mettre d'accésseur public puisque c'est pour un usage d'optimisation interne à la classe).

@we.are.the.storm: Tes deux codes sont identiques. Sauf que dans le premier, la variable img est seulement visible que dans le bloc 'while' tandis que le deuxième img est visible dans le bloc parent du while (donc aussi dans le bloc 'while').

[we.are.the.storm]

@we.are.the.storm: Tes deux codes sont identiques. Sauf que dans le premier, la variable img est seulement visible que dans le bloc 'while' tandis que le deuxième img est visible dans le bloc parent du while (donc aussi dans le bloc 'while').

Le but ici n'est pas de modifier le comportement du code mais bien d'optimiser la consommation en ressources mémoire.

[jowo]

Tu n'as compris mon message.

Ta solution n'économise pas de ressource mémoire. Tes deux codes consomment la même quantité de mémoire.....

Code :

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

1
2
3
4
5
 
while (true) {
    ImageIcon img = (ImageIcon) in.readObject(); //va allouer un nouvel espace mémoire pour stocker un ImageIcon à chaque passage dans la boucle. 
    ...
}

La charge de travail du "Garbage Collector" sera la même si tu déclares la variable hors de la boucle. Seule la visibilité de la variable est changée.

[dinobogan]

Le but ici n'est pas de modifier le comportement du code mais bien d'optimiser la consommation en ressources mémoire.

Attention, la solution que tu proposes est pire, car tu as oublié un détail.
Exemple concret, ça revient à peut près à ce que tu proposes :

Code :

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

1
2
3
4
5
6
int[] tab = null;
for( int i = 0 ; i < 3 ; ++i )
{
	tab = new int[ 14 * 1024 * 1024 ];
	System.out.println( i );
}

Ce code plante à la deuxième itération de boucle (si la mémoire allouée à la JVM reste par défaut, 64Mo).
Par contre, le code suivant est bon :

Code :

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

1
2
3
4
5
6
7
int[] tab = null;
for( int i = 0 ; i < 3 ; ++i )
{
	tab = null;
	tab = new int[ 14 * 1024 * 1024 ];
	System.out.println( i );
}

La mémoire allouée à "tab" est d'abord libérée, puis un nouveau tableau est construit.

Mis à part ce détail, je suis d'accord avec toi : autant conserver un pointeur unique en le déclarant en dehors de la boucle. C'est pas grand chose de gagné, mais c'est toujours bon à prendre. Par contre, il faut vraiment mettre un commentaire pour la mise à null de la variable avant une nouvelle déclaration.

[adiGuba]

La réponse t'a déjà été donné :

Bonjour, à priori, tu ne fermes pas les streams que tu ouvres (out et in). FAQ : Comment libérer proprement les ressources (ou comment utiliser proprement les bloc try/finally) ?

a++

[blaster681]

Je sais sa mais je peut pas fermer mon output dans la boucle while... et vu que c'est une boucle infini le .close() externe a la boucle dans le finally se déclenchera jamais

ou je dois a nouveau faire un new ObjectOutputStream(con.getOutputStream()); ?

Ou je me plante entièrement

[blaster681]

la seul solution pour vider les ressources utiliser par out.writeObject(...); est de fermer le flux. Mais comment faire si il est dans une boucle infini sens sortir de celle-ci et ne pas avoir une erreur "java.net.SocketException: Socket closed"?

J'ai vraiment besoin d'aide je bloque sur ce problème depuis deux semaines

Merci.

[blaster681]

Merci pour ta reponse adiGuba.

Je vous mets le code complet:

Class start:

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
 
public class start {
 
	/**
         * @param args
         */
 
	public static void main(String[] args) {
		final Display d = new Display();
		final Shell shell = new Shell(d);
		shell.setSize(800, 800);
		shell.setLayout(new FillLayout());
 
		 final Canvas canvas = new Canvas(shell, SWT.NONE);
 
			// Thread t = new Thread() {
			       // public void run() {        
			robot r = new robot();
			r.srceen(d, canvas);
			  //      }
		    //  };
		      //t.start();
 
		shell.open();
 
		while(!shell.isDisposed()) {
			if (d.readAndDispatch())
				d.sleep();
		}
		d.dispose();
	}
 
}

Class robot:

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
 
public class robot {
 
	java.awt.Image image =null;	
	BufferedImage img=null;
	GC gc = null;
	public void srceen(final Display d, final Canvas canvas) {
		final Rectangle  rec = new Rectangle(Toolkit.getDefaultToolkit().getScreenSize());
 
		 Thread t = new Thread() {
		       public void run() {   
 
		 while( true ) {
			 Robot robot = null;
			try {
				robot = new Robot();
			} catch (AWTException e1) {
				// TODO Auto-generated catch block
				e1.printStackTrace();
			}
				img = robot.createScreenCapture(rec);
				image = img;		
			robot = null;
				try {
					d.asyncExec(new Runnable() {
						public void run() {
							try {
								gc = null;
								 gc = new GC(canvas);
								 aToS t = new aToS();
								gc.drawImage(t.makeSWTImage(d, image), 0, 0);					
							} catch (Exception e) {
								// TODO Auto-generated catch block
								e.printStackTrace();
							} finally {					
									gc.dispose();
									img.flush();
									image.flush();
							}
						}
					}
						);
 
				} catch (Exception e) {
					// TODO Auto-generated catch block
					e.printStackTrace();
				}
		 }
		       }
		 };
		 t.start();   
		 }
}

et enfin la class aToS:

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
 
public class aToS {
	Image makeSWTImage(Display display, java.awt.Image ai)
    throws Exception
  {
    int width = ai.getWidth(null);
    int height = ai.getHeight(null);
    BufferedImage bufferedImage =
      new BufferedImage(width, height, BufferedImage.TYPE_INT_RGB);
    Graphics2D g2d = bufferedImage.createGraphics();
    g2d.drawImage(ai, 0, 0, null);
    g2d.dispose();
    int[] data =
  ( (DataBufferInt)bufferedImage.getData().getDataBuffer())
      .getData();
 
    ImageData imageData =
      new ImageData(width, height, 24,
        new PaletteData(0xFF0000, 0x00FF00, 0x0000FF));
    imageData.setPixels(0, 0, data.length, data, 0);
    Image swtImage = new Image(display, imageData);
 
    bufferedImage = null;
    g2d = null;
    data = null;
    imageData = null;
 
    return swtImage;
  }
}

Voila vous avez tout Il y a certainement des Object = null; qui servent a rien... mais depuis deux jours j'ai tourner le code dans tout les sens ^^
(Ce n'est pas mon projet, mais une simple simulation du probleme)

@ ++

[blaster681]

Personne pour me filer une piste?

[blaster681]

Bon je me répond a moi même

Espèse d'idiot tu aurrais dû faire sa:

Code :

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

1
2
3
4
5
6
7
8
9
 
[...]
 
gc = new GC(canvas);
aToS c = new aToS();
Image i = c.makeSWTImage(d, image);
gc.drawImage(i, 0, 0);
i.Dispose();     // De préférence dans un bloc finally mais je voulais pas remettre tout le code :mrgreen:
[...]

Voila ce qui arrive apres avoir fais une bonne grasse mat'

[blaster681]

Bonjour,

J'ai le même genre de problème avec un autre code mais sans aucun message d'erreur, la mémoire augmente constamment...

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
 
public class serverStart {
 
	ServerSocket ser;
	Socket con;
	ObjectOutputStream out;
	ObjectInputStream in;
 
	public serverStart() {
 
	}
 
	void ejecutar(){
 
	try{
	ser = new ServerSocket( 5700,1 );
	con = ser.accept();
	out= new ObjectOutputStream(con.getOutputStream());
	out.flush();
	in= new ObjectInputStream(con.getInputStream());
	procesar();
	}
	catch(IOException e){}
	}
 
	void procesar() throws IOException{
 
	while(true){
	try{
 
		Robot r = new Robot();
 
		BufferedImage buffer = r.createScreenCapture(new Rectangle(200, 200, 800, 800));
		ImageIcon i = new ImageIcon(buffer);
		out.writeObject( (ImageIcon) i );
		buffer = null;
	}catch(IOException e){} catch (AWTException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	} finally {
		out.reset();
		System.out.println("finally");
	}
	}
 
	}
 
	public static void main(String args[]) {
		serverStart srv = new serverStart();
	srv.ejecutar();
	}

Je suis pas sur mais je pense que sa vien du-faite que le out.writeObject( (ImageIcon) i ); est dans une boucle infinie... et qu'il ne se ferme que une fois l'application fermer. Mais je suis obliger d'envoyer en boucle les images...

Mon projet consiste a faire un logiciel client/serveur pour faire du contrôle a distance.

Vous avez une solution?
Merci.

Olivier