Gestion image et fuite mémoire

Version imprimable

Voir 40 message(s) de cette discussion en une page

22/11/2010, 13h10
Jay

Gestion image et fuite mémoire
Bonjour,

Je développe actuellement une application qui gère l'affichage de plusieurs image provenant du téléphone ou d'internet.

Mon problème c'est que de temps en temps sans vraiment l'expliquer, l'application "Crash" avec un message du type :
Code:

1 2 3 "java.lang.OutOfMemoryError: bitmap size exceeds VM budget at android.graphics.Bitmap.nativeCreate(Native Method) at android.graphics.Bitmap.createBitmap(Bitmap.java:468)
Existe-t-il un moyen d'analyser les fuites mémoires ou les ressources non utilisées.

Merci

Salut,

Ton problème vient d'un non recycle sur une bitmap.

Il faut les recycler.

Code:

1
2
// désalloue ta mémoire
bitmap.recycle();

Attention celle présente dans les imageview aussi

Code:

1
2
3
4
5
6
7
8
 
ImageView iv = findViewbyId(R.id.ton imageview);
BitmapDrawable dw = (BitmapDrawable) iv.getDrawable();
		if (dw != null) {
			Bitmap bmpold = dw.getBitmap();
			if (bmpold != null)
				bmpold.recycle();
		}

Pour voir ta Mémoire,

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
public static void logHeap(Class clazz) {
    Double allocated = new Double(Debug.getNativeHeapAllocatedSize())/new Double((1048576));
    Double available = new Double(Debug.getNativeHeapSize())/1048576.0);
    Double free = new Double(Debug.getNativeHeapFreeSize())/1048576.0);
    DecimalFormat df = new DecimalFormat();
    df.setMaximumFractionDigits(2);
    df.setMinimumFractionDigits(2);
 
    Log.d(APP, "debug. =================================");
    Log.d(APP, "debug.heap native: allocated " + df.format(allocated) + "MB of " + df.format(available) + "MB (" + df.format(free) + "MB free) in [" + clazz.getName().replaceAll("com.myapp.android.","") + "]");
    Log.d(APP, "debug.memory: allocated: " + df.format(new Double(Runtime.getRuntime().totalMemory()/1048576)) + "MB of " + df.format(new Double(Runtime.getRuntime().maxMemory()/1048576))+ "MB (" + df.format(new Double(Runtime.getRuntime().freeMemory()/1048576)) +"MB free)");
    System.gc();
    System.gc();
 
    // don't need to add the following lines, it's just an app specific handling in my app        
    if (allocated>=(new Double(Runtime.getRuntime().maxMemory())/new Double((1048576))-MEMORY_BUFFER_LIMIT_FOR_RESTART)) {
        android.os.Process.killProcess(android.os.Process.myPid());
    }
}

22/11/2010, 13h47
Jay

Merci pour la réponse aussi rapide.

Je regarde cela et je fais mes tests dans l’après midi.

Il ne me reste plus que ce point avant de mettre l'application sur le Market ;-)
01/04/2011, 15h29
dawadam

Salut, j'ai ce souci aussi, je télécharge des photos du net que je stocke sur la carte SD.
Ensuite j'aimerais les afficher dans une gridView.
J'ai essayé la technique du recyclage mais cela me donne une nouvelle erreur : java.lang.RuntimeException: Canvas: trying to use a recycled bitmap
Ce qui semble logique.
Alors comment faire ?
De plus, j'ai l'erreur de dépassement de mémoire alors que je ne charge qu'une seul image d'a peine 500 ko !
01/04/2011, 15h48
MrDuChnok

Il faut "recycler" une image à partir du moment où elle n'est plus utilisée.

Pourrais tu nous montrer un peu de code afin d'analyser comment tu charges/utilises/décharges ton image ?

Merci.
01/04/2011, 15h54
ZouBi

Citation:

Envoyé par dawadam

Salut, j'ai ce souci aussi, je télécharge des photos du net que je stocke sur la carte SD.
Ensuite j'aimerais les afficher dans une gridView.
J'ai essayé la technique du recyclage mais cela me donne une nouvelle erreur : java.lang.RuntimeException: Canvas: trying to use a recycled bitmap
Ce qui semble logique.
Alors comment faire ?
De plus, j'ai l'erreur de dépassement de mémoire alors que je ne charge qu'une seul image d'a peine 500 ko !

Est ce que, dans la vrai vie, tu utilises tes objets après les avoir recyclés, ou avant? ;)
01/04/2011, 19h30
dawadam

Citation:

Envoyé par MrDuChnok

Il faut "recycler" une image à partir du moment où elle n'est plus utilisée.

Pourrais tu nous montrer un peu de code afin d'analyser comment tu charges/utilises/décharges ton image ?

Merci.

Je viens de tester avec un code très épuré et ça marche...
Ça bug quant j'utilise un Adapter pour une GridView.
Je posterais un code plus approprié dès que possible.

Citation:

Envoyé par ZouBi

Est ce que, dans la vrai vie, tu utilises tes objets après les avoir recyclés, ou avant? ;)

Tu vois, c'est pour ça que j'ai mis que ça semblait logique... :scarymov:

Voici un joli code qui plante :lol:

Activity :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public class TestLoad2 extends Activity {
 
	public void onCreate(Bundle savedInstanceState) {
		super.onCreate(savedInstanceState);
		this.setContentView(R.layout.menu_gridview);
		// Adapter
		AdaptTest adapt = new AdaptTest(this);
		// GridView
		GridView gridview = (GridView) findViewById(R.id.gridview);
		gridview.setAdapter(adapt);
	}
}

Adapter :

Code:

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
 
public class AdaptTest extends BaseAdapter {
	private Context context;
 
	public AdaptTest(Context c) {
		context = c;
	}
 
	public int getCount() {
		return 1;
	}
 
	public Object getItem(int position) {
		return null;
	}
 
	public long getItemId(int position) {
		return 0;
	}
 
	// retourne la view affichée
	public View getView(int position, View convertView, ViewGroup parent) {
		ImageView imageView;
		if (convertView == null) {
			imageView = new ImageView(context);
			imageView.setLayoutParams(new GridView.LayoutParams(85, 85));
			imageView.setScaleType(ImageView.ScaleType.CENTER_CROP);
			imageView.setPadding(8, 8, 8, 8);
		} else {
			imageView = (ImageView) convertView;
		}
 
		try {
			String fileName = "test.jpg";
			String chemin = Environment.getExternalStorageDirectory()
					+ "/photos/" + fileName;
			FileInputStream in = new FileInputStream(chemin);
			BufferedInputStream buf = new BufferedInputStream(in);
			Bitmap bmp = BitmapFactory.decodeStream(buf);
			imageView.setImageDrawable(new BitmapDrawable(bmp));
 
		} catch (FileNotFoundException e) {
			e.printStackTrace();
		}
 
		return imageView;
	}
}

GridView :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
<?xml version="1.0" encoding="utf-8"?>
<GridView xmlns:android="http://schemas.android.com/apk/res/android" 
    android:id="@+id/gridview"
    android:layout_width="fill_parent" 
    android:layout_height="fill_parent"
    android:columnWidth="90dp"
    android:numColumns="auto_fit"
    android:verticalSpacing="10dp"
    android:horizontalSpacing="10dp"
    android:stretchMode="columnWidth"
    android:gravity="center"
/>

C'est la ligne :
Bitmap bmp = BitmapFactory.decodeStream(buf);
Qui cause l'erreur :
java.lang.OutOfMemoryError: bitmap size exceeds VM budget
Une idée ? :?

04/04/2011, 13h38
Feanorin

Bonjour,

Citation:

Bitmap bmp = BitmapFactory.decodeStream(buf);
Qui cause l'erreur :
java.lang.OutOfMemoryError: bitmap size exceeds VM budget

Les images que tu essayes de charger est trop grande pour la mémoire.

Essayes de réduire la taille des images soit directement , soit via un scale avant de la positionner dans l'imageview.
04/04/2011, 13h48
dawadam

Citation:

Envoyé par Feanorin

Les images que tu essayes de charger est trop grande pour la mémoire.

Si c'est ça, pourquoi cela fonctionne quand je la charge simplement dans une ImageView ?

En plus de cela, l'erreur survient à la création de la bitmap, je ne peux donc pas la redimensionner.

Citation:

Si c'est ça, pourquoi cela fonctionne quant je la charge simplement dans une ImageView ?

Les images que tu essayes de charger sont trop grande pour la mémoire.

N'oublie pas que tu as une GridView qui garde en mémoire toutes les images que tu affiches ;). Tu n'en as pas qu'une en mémoire .

Fais un scale à ce moment cela te permettra de réduire la mémoire de chaque image donc de les afficher .

Après fais bien attention à recycler les images .

Pour connaître la mémoire utilisé par ton application :
Code:

1 2 3 4 5 6 Double allocated = new Double(Debug.getNativeHeapAllocatedSize())/new Double((1048576)); Double available = new Double(Debug.getNativeHeapSize())/new Double((1048576)); Double free = new Double(Debug.getNativeHeapFreeSize())/new Double((1048576)); DecimalFormat df = new DecimalFormat(); df.setMaximumFractionDigits(2); df.setMinimumFractionDigits(2);

04/04/2011, 14h26
dawadam

Citation:

Envoyé par Feanorin

N'oublie pas que tu as une GridView qui garde en mémoire toutes les images que tu affiches ;). Tu n'en as pas qu'une en mémoire .

Je fait le test avec une seul image pour le moment et ça plante.
En plus de ça, quand je met la photo dans les ressources et que je charge la view avec :

Code:

imageView.setImageResource(R.drawable.photo);

À la place de charger la bitmap, ça fonctionne !
04/04/2011, 14h52
dawadam
Bon, j'ai remarqué que la méthode getView de l'Adapter était appelée 4 fois, même avec une seul photo...

Alors en utilisant un redimentionnement et en recyclant l'image, ça marche :
Code:

1 2 3 Bitmap bmp = loadBitmap("photo.jpg"); imageView.setImageDrawable(new BitmapDrawable(changeTaille(bmp))); bmp.recycle();
Merci !
:ccool:

Bonjour je me permet de ré-ouvrir ce sujet, car j'ai quelques questions et problèmes dessus.
Je voulais faire une galerie d'image provenant du web (comme le tuto proposé sur le site). Mon problème c'est que les images sont très grosses (je peux en afficher qu'une à la fois). Ce que je fais c'est qu'au début je télécharge les images une par une, je les rétrécie beaucoup et les stockes dans une liste. Je peux donc afficher la banderole du bas.
Quand je clique sur une image de la banderole je vais chercher sur internet l'image en taille réelle pour avoir une bonne qualité.

J'ai utilisé le code de Feanorin pour afficher l'espace mémoire de mon application en entrant dans mon activité, à chaque téléchargement et à la destruction de mon activité. J'obtiens des valeurs bizarres :
En entrant :

Citation:

allocated: 9,37
available: 9,44
free: 0,08

Je me dis que j'ai pas énormément d'espace...
Après le téléchargement des petites images j'obtiens ceci :

Citation:

allocated: 10,69
available: 10,89
free: 0,18

Ca augmente légèrement.
et après le téléchargement d'une grosse image :

Citation:

allocated: 25,89
available: 26,14
free: 0,25

et quand je détruis mon activité :

Citation:

allocated: 9,44
available: 26,14
free: 0,25

Alors là je comprends plus rien. Pourquoi mon espace augmente. Enfin je vais pas me plaindre, ca me permet de télécharger mes images. Mais si j'en télécharge un autre j'ai l'erreur outOfMemorryError (logique).
Est ce que vous savez d'où vient cette attitude, est ce normal, comment cela fonctionne t-il exactement ?

Après j'ai un autre problème
Dans ma classe j'ai une Bitmap qui contient l'image de mon ImageView.
Le téléchargement de la première image se passe très bien mais lors de la seconde, j'ai l'erreur de mémoire. Pourtant je stocke ma nouvelle image sur l'ancienne Bitmap. J'ai bien tenté un recycle, mais j'obtiens une autre erreur.
Voici le code que j'utilise (assez banal) :
Code:

1 2 3 4 5 6 7 URLConnection conn = myURL.openConnection(); conn.connect(); InputStream is = conn.getInputStream(); BufferedInputStream bis = new BufferedInputStream(is); currentPicture = BitmapFactory.decodeStream(bis); bis.close(); is.close();
Si vous avez une idée pour m'aider à résoudre ce problème, je serai ravi de l'entendre.
Merci d'avance

Bonjour,

Citation:

Le téléchargement de la première image se passe très bien mais lors de la seconde, j'ai l'erreur de mémoire. Pourtant je stocke ma nouvelle image sur l'ancienne Bitmap. J'ai bien tenté un recycle, mais j'obtiens une autre erreur.

Ton problème provient bien de là , tu as des bitmaps que tu n'as pas libérées.

Essaye de voir avec l'allocation que tu as utilisé pour savoir où tu as une fuite .

Pour information le recyclage d'image se fait lorsque tu remplace une bitmap par une autre il faut d'abord libéré l'espace mémoire prise par la première cela se fait comme tel :
Code:

1 2 3 4 5 6 7 ImageView iv = findViewbyId(R.id.ton imageview); BitmapDrawable dw = (BitmapDrawable) iv.getDrawable(); if (dw != null) { Bitmap bmpold = dw.getBitmap(); if (bmpold != null) bmpold.recycle(); }
Citation:

Alors là je comprends plus rien. Pourquoi mon espace augmente. Enfin je vais pas me plaindre, ca me permet de télécharger mes images. Mais si j'en télécharge un autre j'ai l'erreur outOfMemorryError (logique).
Est ce que vous savez d'où vient cette attitude, est ce normal, comment cela fonctionne t-il exactement ?

Ton espace augmente car tu demandes à ton application plus d'espace mémoire (heap).

Essaye aussi de recycler les bitmap que tu alloues dans une procédure sans les réutiliser par la suite (Bitmap temporaire) .

Citation:

Envoyé par Feanorin

Ton problème provient bien de là , tu as des bitmaps que tu n'as pas libérées.

Essaye de voir avec l'allocation que tu as utilisé pour savoir où tu as une fuite .

Pour information le recyclage d'image se fait lorsque tu remplace une bitmap par une autre il faut d'abord libéré l'espace mémoire prise par la première cela se fait comme tel :

Code:

1
2
3
4
5
6
7
ImageView iv = findViewbyId(R.id.ton imageview);
BitmapDrawable dw = (BitmapDrawable) iv.getDrawable();
		if (dw != null) {
			Bitmap bmpold = dw.getBitmap();
			if (bmpold != null)
				bmpold.recycle();
		}

Ok, je pensais que si on écrasait les données, on avait pas besoin de recycler l'ancienne bitmap.
J'ai appliqué ton code mais je me retrouve avec l'erreur que j'avais déjà eu auparavant :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
05-10 16:49:54.750: ERROR/AndroidRuntime(19683): FATAL EXCEPTION: main
05-10 16:49:54.750: ERROR/AndroidRuntime(19683): java.lang.RuntimeException: Canvas: trying to use a recycled bitmap android.graphics.Bitmap@40599770
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at android.graphics.Canvas.throwIfRecycled(Canvas.java:955)
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at android.graphics.Canvas.drawBitmap(Canvas.java:1044)
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at android.graphics.drawable.BitmapDrawable.draw(BitmapDrawable.java:325)
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at android.widget.ImageView.onDraw(ImageView.java:872)
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at android.view.View.draw(View.java:6880)
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at android.view.ViewGroup.drawChild(ViewGroup.java:1646)
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at android.view.ViewGroup.dispatchDraw(ViewGroup.java:1373)
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at android.view.ViewGroup.drawChild(ViewGroup.java:1644)
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at android.view.ViewGroup.dispatchDraw(ViewGroup.java:1373)
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at android.view.View.draw(View.java:6883)
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at android.widget.FrameLayout.draw(FrameLayout.java:357)
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at android.view.ViewGroup.drawChild(ViewGroup.java:1646)
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at android.view.ViewGroup.dispatchDraw(ViewGroup.java:1373)
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at android.view.View.draw(View.java:6883)
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at android.widget.FrameLayout.draw(FrameLayout.java:357)
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at com.android.internal.policy.impl.PhoneWindow$DecorView.draw(PhoneWindow.java:1862)
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at android.view.ViewRoot.draw(ViewRoot.java:1522)
05-10 16:49:54.750: ERROR/AndroidRuntime(19683):     at android.view.ViewRoot.performTraversals(ViewRoot.java:1258)

Voici comment je l'ai implémenté :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
BitmapDrawable dw = (BitmapDrawable) myImageView.getDrawable();
if (dw != null) {
	Bitmap b = dw.getBitmap();
	if (b != null)
		b.recycle();
}
URLConnection conn = myURL.openConnection();
conn.connect();
InputStream is = conn.getInputStream();
BufferedInputStream bis = new BufferedInputStream(is);
currentPicture = BitmapFactory.decodeStream(bis);
bis.close();
is.close();

Vu que je fais cette opération dans un thread le myImageView.SetImage(currentPicture) est dans le handler.

J'ai bien essayé de gruger en mettant

Code:

1
2
if(myImageView.getDrawingCache() != null)
				myImageView.getDrawingCache().recycle();

pour éviter l'erreur mais là c'est pire ça ne recycle pas l'image ^^

En tout cas merci d'essayer de m'aider

Citation:

Envoyé par Feanorin

Essaye aussi de recycler les bitmap que tu alloues dans une procédure sans les réutiliser par la suite (Bitmap temporaire) .

Merci pour le conseil, je vais essayer de le faire partout et pas juste sur le onDestroy();

Je n'avais pas trouvé cela évident non plus quand il fallait l'implémenter . Mais en tout cas c'est bien ce qui faut faire . Tu ne dois pas le mettre au bon endroit , personnellement je nettoie le cache de mon ImageView juste avant de mettre une nouvelle bitmap.

Pour les Bitmap temporaire je les force à se recycler à la fin de la procédure .
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 BitmapDrawable dw = (BitmapDrawable) myImageView.getDrawable(); if (dw != null) { Bitmap b = dw.getBitmap(); if (b != null) b.recycle(); } URLConnection conn = myURL.openConnection(); conn.connect(); InputStream is = conn.getInputStream(); BufferedInputStream bis = new BufferedInputStream(is); currentPicture = BitmapFactory.decodeStream(bis); bis.close(); is.close();
Par exemple ne fais pas le recyclage ici fais le juste avant ton myImageView.SetImage(currentPicture) je pense que cela reste le meilleur endroit. Ton Handler est dans l' UIThread ?

Citation:

Citation:
Envoyé par Feanorin Voir le message
Essaye aussi de recycler les bitmap que tu alloues dans une procédure sans les réutiliser par la suite (Bitmap temporaire) .
Merci pour le conseil, je vais essayer de le faire partout et pas juste sur le onDestroy();

Juste une nouvelle petite précision , ne recycle pas celle que tu utilises dans les imageview sinon tu perdras également les données dans l'imageview également, ne la recycle que lorsque l'imageview n'en a plus besoin .

Je te passe le lien de la doc on sait jamais
http://developer.android.com/referen...#recycle%28%29

10/05/2011, 17h54
AndroChristophe
Oui j'ai mon Handler dans mon UIthread, pourquoi il ne faut pas faire comme ça ?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 public class monActivité extends Activity { @Override public void onCreate(Bundle savedInstanceState) { ... } final Handler mHandler = new Handler() { ... } }
Ok je vais essayer de le mettre juste avant de changer la Bitmap. Suis je obligé de passé par un Drawable ou est ce que je peux directement utiliser le getDrawingCache qui me renvoie une Bitmap ? (enfin c'est pas vraiment important, ca rajoute pas beaucoup de ligne de codes ^^).

Je vais tester ce que tu m'as dit et je re-poste (ça sera surement demain ;))

Merci encore pour ton aide

Edit : je retire ce que j'ai dit sur le getDrawingCache, je n'avais pas lu en entier
10/05/2011, 18h14
Feanorin

Citation:

Oui j'ai mon Handler dans mon UIthread, pourquoi il ne faut pas faire comme ça ?

En fait la partie de ton code qui sera dans l'handler ne sera pas dans l'UIThread , pour remédier à ce problème on passe par un postinvalide sur les objets ou par un RunOnUiThread , et une autre méthode pour pouvoir modifier le graphique. (désolé je n'ai plus la troisième méthode en tête :oops:)

http://developer.android.com/referen...0int,%20int%29

http://developer.android.com/referen...ng.Runnable%29

Après je ne sais pas si cela influence ou pas le Recycle mais cela peut être également une piste.

NB:
UIThread est le thread qui s'occupe des ressources ce n'est pas parce que ton code est dans l'Activity que celui-ci fait partie de l'UIThread .

Citation:

Edit : je retire ce que j'ai dit sur le getDrawingCache, je n'avais pas lu en entier

8-).

Bonjour,

Ouah j'ai l'impression de ne plus rien comprendre (deux mois sur Android pour rien ^^).

Citation:

Envoyé par Feanorin

NB:
UIThread est le thread qui s'occupe des ressources ce n'est pas parce que ton code est dans l'Activity que celui-ci fait partie de l'UIThread .

Mais il y a bien que l'UIThread qui peut modifie les éléments de la vue ? Normalement si j'essaye de changer un élément contenu dans ma view depuis un autre thread, je devrai recevoir une erreur de ce type là

Code:

android.view.ViewRoot$CalledFromWrongThreadException: Only the original thread that created a view hierarchy can touch its views.

non ? Je pensais que le Handler était une sorte d'observer qui faisait la liaison entre l'UIThread et les autres Thread et donc que son code était plus ou moins rattaché à celui de l'UIThread.

L'utilisation de la fonction runOnUiThread se fait directement dans le thread
Code:

1 2 3 4 5 6 7 8 9 10 Thread mTread = new Thread(new Runnable() { @Override public void run() { //Récupération sur internet de l'image runOnUiThread(new Runnable() { public void run() { //mettre à jour l'écran } }); ...
oui alors je le fais dans le Handler ? Si c'est la première solution est la bonne, les Handler ne servent à rien au final s'ils ne peuvent pas interagir sur la vue.

Par contre j'ai un plus de mal à voir l'utilisation de invalidate(), postInvalidate(). D'après ce que j'ai compris, on surcharge la méthode invalidate pour qu'elle fasse ce que l'on veut et après on fait myImageView.postInvalidate(); pour appeler la méthode. Je ne vois pas la différence qu'il y a entre les 2 méthodes.

Désolé et merci de m'avoir consacré de ton temps pour m'expliquer ceci.

Sinon j'ai fait le test ce matin. Effectivement ca marche bien mieux quand je recycle l'image juste avant d'insérer la nouvelle, sauf dans le cas ou ce sont de très grande image... En effet il n'y a pas la place pour deux images sur la mémoire de l'application. Je vais être obligé de recycler mon image avant de télécharger la nouvelle.

Voir 40 message(s) de cette discussion en une page