Gestion image et fuite mémoire

Version imprimable

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.

11/05/2011, 10h44
Feanorin

Bonjour,

Citation:

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à

Oui , tu as juste ;).

Citation:

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.

Le Handler

Citation:

A Handler allows you to send and process Message and Runnable objects associated with a thread's MessageQueue. Each Handler instance is associated with a single thread and that thread's message queue. When you create a new Handler, it is bound to the thread / message queue of the thread that is creating it -- from that point on, it will deliver messages and runnables to that message queue and execute them as they come out of the message queue.

http://developer.android.com/referen...s/Handler.html

Citation:

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

Oui c'est pour préciser à ton thread que tu aimerais qu'il tourne directement sur l'UIThread ce qui n'est pas le cas si tu utilises la procédure run .

Par contre cela implique qu'il utilise déjà un thread existant (UIThread) et non un nouveau thread .

Citation:

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.

Si ils servent à plein de chose pas seulement pour l'affichage de la vue .
La gestion des messages (envoie, réception), les observers sur le changement de donnée , ...

Citation:

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.

Tu peux utiliser les handlers ou the thread (run), par contre comme tu l'as souligné il faudra utilisé le postinvalidate sur toutes modifications de la vue , pour indiquer que cette méthode doit bien se faire dans l'UIThread et non dans le thread actuel .

En gros
invalidate -> appel d'un UIThread
postinvalidate-> appel d'un Non UI Thread

Citation:

public void postInvalidate ()
Since: API Level 1

Cause an invalidate to happen on a subsequent cycle through the event loop. Use this to invalidate the View from a non-UI thread.
See Also

* invalidate()

tu as les autres méthodes à ce lien
http://developer.android.com/guide/t...d-threads.html

Citation:

* Activity.runOnUiThread(Runnable)
* View.post(Runnable)
* View.postDelayed(Runnable, long)

Citation:

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.

:ccool: ,essayes alors de réduire la taille de la première avant de télécharger la seconde, à mois que tu ne sois obligé de garder la qualité de l'image . Sinon c'est là

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

Au pire tu peux toujours la compresser,
http://developer.android.com/referen...utputStream%29

C'est de nouveau moi !!

Je ne suis pas sur d'avoir tout bien compris, mais j'ai essayé de mettre en application les choses vu sur ce sujet.

J'ai regardé ce lien http://android-developers.blogspot.c...threading.html et vu qu'avec les Thread je n'ai pas vraiment d'accroche, j'ai décidé de tenter avec les AsyncTask même si on m'avait dit que ça s'utilisait plutôt dans les services, est ce vrai ?

Bien évidemment cela ne marche pas, ca serait trop beau...
Voici mon code : (j'ai enlevé ce qui est dans le try car c'est la même chose depuis le début.

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
private class DownloadImageTask extends AsyncTask<String,Void,Bitmap> {
 
		@Override
		protected Bitmap doInBackground(String... urls) {
			Bitmap b = null;
			try {
				//Récupération image
			} catch (IOException e) {
				e.printStackTrace();
			}  catch (OutOfMemoryError e) {
				Log.d("Pas assez de mémoire", "dommage");
			}
			return b;
		}
 
		@Override
		protected void onPostExecute(Bitmap result) {
			BitmapDrawable dw = (BitmapDrawable) myImageView.getDrawable();
			if (dw != null) {
				Bitmap b = dw.getBitmap();
				if (b != null)
					b.recycle();
			}	
			myImageView.setImageBitmap(result);
		}
 
		@Override
		protected void onPreExecute() {
 
		}
 
 
	}

Donc là, ça marche, merci j'y serais jamais arriver sans toi, mais il reste quand même un problème. Je télécharge des images vraiment grosse. L'application ne peut pas en avoir plus d'une à la fois. Du coup quand je télécharge une nouvelle photo vu qu'il n'a pas encore recyclé l'ancienne, il passe dans l'exception outOfMemoryError.

Je vais vous expliquer toute ma démarche.
J'ai décidé de faire le recycle dans le onPreExecute. Et là gros crash

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
05-11 15:15:06.316: ERROR/AndroidRuntime(31802): FATAL EXCEPTION: main
05-11 15:15:06.316: ERROR/AndroidRuntime(31802): java.lang.RuntimeException: Canvas: trying to use a recycled bitmap android.graphics.Bitmap@405a8440
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.graphics.Canvas.throwIfRecycled(Canvas.java:955)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.graphics.Canvas.drawBitmap(Canvas.java:1044)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.graphics.drawable.BitmapDrawable.draw(BitmapDrawable.java:325)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.widget.ImageView.onDraw(ImageView.java:872)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.view.View.draw(View.java:6880)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.view.ViewGroup.drawChild(ViewGroup.java:1646)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.view.ViewGroup.dispatchDraw(ViewGroup.java:1373)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.view.ViewGroup.drawChild(ViewGroup.java:1644)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.view.ViewGroup.dispatchDraw(ViewGroup.java:1373)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.view.View.draw(View.java:6883)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.widget.FrameLayout.draw(FrameLayout.java:357)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.view.ViewGroup.drawChild(ViewGroup.java:1646)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.view.ViewGroup.dispatchDraw(ViewGroup.java:1373)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.view.View.draw(View.java:6883)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.widget.FrameLayout.draw(FrameLayout.java:357)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at com.android.internal.policy.impl.PhoneWindow$DecorView.draw(PhoneWindow.java:1862)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.view.ViewRoot.draw(ViewRoot.java:1522)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.view.ViewRoot.performTraversals(ViewRoot.java:1258)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.view.ViewRoot.handleMessage(ViewRoot.java:1859)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.os.Handler.dispatchMessage(Handler.java:99)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.os.Looper.loop(Looper.java:130)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at android.app.ActivityThread.main(ActivityThread.java:3683)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at java.lang.reflect.Method.invokeNative(Native Method)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at java.lang.reflect.Method.invoke(Method.java:507)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:839)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:597)
05-11 15:15:06.316: ERROR/AndroidRuntime(31802):     at dalvik.system.NativeStart.main(Native Method)

Je suis repartis sur le premier code avec tout dans le onPostExecute. Vu que je passais dans le catch, le onPostExecute recevait null en valeur de Bitmap, ce qui avait pour effet de supprimer la photo (normal). Si je recliquais sur l'image il arrivait à la télécharger vu qu'il y avait de l'espace. Mais c'est pas top de devoir cliquer deux fois.

Donc j'ai tenté un truc et là je vous jure que je ne vous mens pas, que c'est pas le fruit du hasard. J'ai mis ça

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
@Override
		protected void onPostExecute(Bitmap result) {
			myImageView.setImageBitmap(result);
 
		}
 
		@Override
		protected void onPreExecute() {
			BitmapDrawable dw = (BitmapDrawable) myImageView.getDrawable();
			if (dw != null) {
				Bitmap b = dw.getBitmap();
				if (b != null)
					b.recycle();
			}
			myImageView.setImageBitmap(null);
		}

Et là ça marche... Mais mais pourquoi ??
Je ne comprends vraiment pas ce qui se passe.

Merci encore de m'avoir aidé. Je range cette discussion dans mes favoris, comme si un jour j'ai trop le moral je viendrai la relire :)

11/05/2011, 15h43
Feanorin

Bonjour,

Citation:

Et là ça marche... Mais mais pourquoi ??
Je ne comprends vraiment pas ce qui se passe.

Hmm , tu ne peux pas recycler ta bitmap tant que tu n'as pas changer celle-ci .
Sinon elle perds les données et elle ne sait pas où les retrouver.
Donc cela marche dans ton deuxième cas car tu changes la bitmap de suite après l'avoir recycler .

par contre juste une question quand tu parles de charger des images vraiment grosses c'est quelle taille à peu près ?
11/05/2011, 16h42
AndroChristophe

Je me suis peut être un peu emporté sur le terme "vraiment grosses" :roll:. Elles font plus d'un Mo :aie:. Mais pour moi elles grosses car le téléchargement prend du temps.
Ok merci pour ta réponse

Je me permet de faire remonter cette question car j'ai le même problème que je n'arrive absolument pas à resoudre

Voici un bout de ma classe permettant de redimensionner une image. Pour permettre de se déplacer sur une image plus grande que l'écran

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
49
50
 
public class ResizableBitmap
{
	// bitmap large enough to be scrolled
	private static Bitmap bmLargeImage; 
 
	// zoom leve
	private float zoomLevel;
 
    public ResizableBitmap( Context context, Bitmap drawable, float zoomLevel)
    {
        // load bitmap
        bmLargeImage = drawable;
		this.zoomLevel = zoomLevel;
 
        // initialiastion par defaut
        Display display = ( (WindowManager) context.getSystemService( Context.WINDOW_SERVICE ) ).getDefaultDisplay();
        onSizeChanged( display.getWidth(), display.getHeight(), 0, 0 );
    }
 
	public void onSizeChanged( int w, int h, int oldw, int oldh )
    {
		... some code
 
        // redimensionnement
        bmLargeImage = resize( bmLargeImage, w );
 
		... some code
	}
 
	private Bitmap resize( Bitmap bm, int w )
    {
        // initial size
        int width = bm.getWidth();
        int height = bm.getHeight();
 
        // zoom level (zoom is always up to 1)
        w *= zoomLevel;
 
        // scale
        float scaleWidth = ( (float) w ) / width;
        float scaleHeight = ( (float) w ) / width;
 
        Matrix matrix = new Matrix();
        matrix.postScale( scaleWidth, scaleHeight );
        Bitmap resizedBitmap = Bitmap.createBitmap( bm, 0, 0, width, height, matrix, true );
 
        return resizedBitmap;
    }
}

Mais j'ai une outofmemory erreur sur certain téléphone. Uniquement ceux avec un heapspace de 16Mb.

Une idée ?

19/12/2011, 13h43
Feanorin

Bonjour,

Est ce que tu recycle tes bitmaps que tu n'utilises plus ?