Discussion :

[philippe13]

bonjour,

Je viens de modifier 2 petites choses dans un programme et voilà qui plante au bout de 3 jours.
Il y a un thread qui fait des opérations sur des bufferedImage.
J'avais :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
public BufferedImage buffer1 = null;
public BufferedImage buffer2 = null;
public BufferedImage buffer3 = null;
 
public void run()
{
        while(....)
        {
              buffer1 = capt.newBuffer();
              buffer2 = capt.newBuffer();
              buffer3 = capt.newBuffer();
        }
}

pas de problème

puis j'ai fait cela :

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
 
public BufferedImage buffer1 = null;
public BufferedImage buffer2 = null;
public BufferedImage buffer3 = null;
 
public void run()
{
        while(....)
        {
              buffer1 = newBufferedImage(tab1,w,h);
              buffer2 = newBufferedImage(tab2,w,h);
              buffer3 = newBufferedImage(tab3,w,h);
        }
 public BufferedImage newBufferedImage(int[] tab,w,h)
    {
        BufferedImage bufIm = new BufferedImage(w, h, BufferedImage.TYPE_INT_RGB); 
        bufIm.setRGB(0,0,w,h,tab,0,w);
        return bufIm;
    }
}

Est ce que ma fonction newBufferedImage peut provoquer un outofmemory ?
Est ce qu'il faut mettre buffer1 = null;buffer2 = null;buffer3 = null à chaque passage pour libérer la mémoire ? Perso, je pensais que dans mon cas ça écrasait.

Merci d'avance de votre aide

[gifffftane]

Selon la taille de l'image, un new BufferedImage peut provoquer un OutOfMemory.

Inutile sur le code que tu nous présentes de mettre buffer1 = null pour libérer la mémoire.

Attention que le traitement d'image est un des domaines où il faut faire attention dans l'usage de la mémoire, non seulement parce qu'il consomme beaucoup, mais que en plus il consomme des zones mémoires pas forcément libérées par le ramasse miettes (il faut bien penser à faire les dispose ou close), et que souvent il traine des buffers partiels qui accumule la mémoire pour peu qu'on les réutilise.

[philippe13]

bonsoir,
merci de ta réponse.

Effectivement j'avais un inputstream qui sert au traitement dans un thread qui n'était mal fermé.... Et cette class est appelé assez souvent.

Le fait de passer les buffer1 à null ne peut il pas aider le GC ?

[gifffftane]

Non le fait de passer le buffer à null n'aide pas GC, au contraire. Ce qui aide le GC est que les variables soient placées dans les blocs où elles sont vraiment utiles, et non comme attributs d'instances alors qu'elles ne sont utilisées que dans une boucle while (par exemple).

Et, bien sûr et ofcourse, que l'on fasse les close des streams, particulièrement ceux qui servent à lire les images