Problème OutOfMemoryError avec des Bitmap sur Android

Version imprimable

Bonjour,

Je suis actuellement en train de développer une application android permettant de tracer des lignes sur un plan quadriller. Seulement après avoir tracer ces lignes j'effectue une sauvegarde de mon plan, après rechargement de celui ci, des fois le message suivant apparaît :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
05-13 15:29:53.831: E/AndroidRuntime(5074): FATAL EXCEPTION: main
05-13 15:29:53.831: E/AndroidRuntime(5074): java.lang.OutOfMemoryError
05-13 15:29:53.831: E/AndroidRuntime(5074): 	at android.graphics.Bitmap.nativeCreate(Native Method)
05-13 15:29:53.831: E/AndroidRuntime(5074): 	at android.graphics.Bitmap.createBitmap(Bitmap.java:640)
05-13 15:29:53.831: E/AndroidRuntime(5074): 	at android.graphics.Bitmap.createBitmap(Bitmap.java:620)
05-13 15:29:53.831: E/AndroidRuntime(5074): 	at com.example.ligne.indoor.Source.drawSource(Source.java:158)
05-13 15:29:53.831: E/AndroidRuntime(5074): 	at com.example.ligne.indoor.PlanSimulation.draw(PlanSimulation.java:103)
05-13 15:29:53.831: E/AndroidRuntime(5074): 	at com.example.ligne.indoor.Floor.draw(Floor.java:128)
05-13 15:29:53.831: E/AndroidRuntime(5074): 	at com.example.ligne.indoor.Plan.onDraw(Plan.java:364)
05-13 15:29:53.831: E/AndroidRuntime(5074): 	at android.view.View.draw(View.java:13776)
05-13 15:29:53.831: E/AndroidRuntime(5074): 	at android.view.View.getDisplayList(View.java:12727)
05-13 15:29:53.831: E/AndroidRuntime(5074): 	at android.view.View.getDisplayList(View.java:12771)
05-13 15:29:53.831: E/AndroidRuntime(5074): 	at android.view.ViewGroup.dispatchGetDisplayList(ViewGroup.java:3205)

on m'indique donc des erreurs à chaque fois au niveau des bitmaps, par exemple pour Plan.onDraw

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
	protected void onDraw(Canvas canvas) {
		canvas.save();
		paint.setStyle(Style.FILL_AND_STROKE);
		Bitmap tmp = drawEchelle();
		Bitmap ori = drawOrigin();
		if (ori != null)
			canvas.drawBitmap(ori, 0, 0, paint);
		if (tmp != null)
			canvas.drawBitmap(tmp, 0, 0, paint);
 
		currentFloor.draw(getWidth(), getHeight());
		canvas.drawBitmap(currentFloor.getBitmap(), 0, 0, paint);
	}

Je ne sais pas comment éviter ce problème, si quelqu'un a des idées ?

13/05/2015, 16h42
dinobogan

Ton image est trop grande et ne peux pas tenir en entier en RAM.
Pour ce cas il n'y a pas 36 solutions : soit tu charges une petite partie de l'image (par exemple la taille de l'écran plus une marge autour s'il faut scroller) soit tu construits à la volée l'image en la dessinant à l'écran (si les éléments composants l'image sont calculable).
13/05/2015, 17h00
lass7

Je vous remercie de votre réponse je comprends bien ce que vous me dites, mais j'ai du mal a voir en quoi dans mon exemple j'ai se soucis, du fais que je ne fais que dessiner un plan quadriller et sinon ajouter des lignes
13/05/2015, 23h02
flarin

salut,
cela ne résoudra pas ton problème de fond, mais tu peux demander à avoir de la mémoire Heap en plus.

Pour cela, ajoute dans le fichier AndroidManifest.xml, à l'intérieur des tags <application>, la ligne suivante (si ce n'est déjà fait) :

Code:

android:largeHeap="true"

.

ça ne fonctionne pas avec les anciennes version d'android.

L'augmentation de mémoire dépendra du terminal Android utilisé, et des applications lancés dessus (ex: de 32 à 64mo, 64 à 96mo, etc...)
Android n'alloue qu'une petite partie de la mémoire (8-12-16-24-32mo...) pour chaque activité, ce qui rends toute application manipulant les bitmaps assez délicate à réaliser.

Merci pour ta réponse je viens de rajouter cela au manifest mais j'ai toujours le même probleme, mon code est le suivant :

J'ai une classe Mur qui dessine mes lignes de couleur grise et rouge lorsque je les selectionne

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
51
52
53
54
55
56
57
58
 
 
public void drawElement(int width, int height) {
 
		double nbPixelByTwo = (2 * Wall.getNbMetre_by_pixel());
		double scaleModify = (Plan.getScale() / Plan.getScaleStd());
 
		if (isSelected()) {
 
			if (bitmapWall != null) {
				bitmapWall.recycle();
				bitmapWall = null;
			}
 
			bitmapWall = Bitmap.createBitmap(width, height, Config.ARGB_8888);
			canvasWall = new Canvas(bitmapWall);
			paintWall = new Paint();
 
			/** Couleur du Mur selectionne **/
			paintWall.setColor(Color.RED);
 
			/** Epaisseur du trait **/
			paintWall.setStrokeWidth(getEpaisseur());
 
			/** Dessine le Mur **/
			canvasWall.drawLine(
					(float) ((getPosX_DebElt() / nbPixelByTwo) * scaleModify),
					(float) ((getPosY_DebElt() / nbPixelByTwo) * scaleModify),
					(float) ((getPosX_FinElt() / nbPixelByTwo) * scaleModify),
					(float) ((getPosY_FinElt() / nbPixelByTwo) * scaleModify),
					paintWall);
 
		} else {
 
			if (bitmapWall != null) {
				bitmapWall.recycle();
				bitmapWall = null;
			}
 
			bitmapWall = Bitmap.createBitmap(width, height, Config.ARGB_8888);
			canvasWall = new Canvas(bitmapWall);
			paintWall = new Paint();
 
			/** Couleur du Mur **/
			paintWall.setColor(Color.GRAY);
 
			/** Definir une épaisseur de trait **/
			paintWall.setStrokeWidth(getEpaisseur());
 
			/** Dessine le Mur **/
			canvasWall.drawLine(
					(float) ((getPosX_DebElt() / nbPixelByTwo) * scaleModify),
					(float) ((getPosY_DebElt() / nbPixelByTwo) * scaleModify),
					(float) ((getPosX_FinElt() / nbPixelByTwo) * scaleModify),
					(float) ((getPosY_FinElt() / nbPixelByTwo) * scaleModify),
					paintWall);
		}
	}

et sinon j'utilise egalement des bitmaps lorsque j'effectue le tracer de mon plan quadrillé

18/05/2015, 11h49
flarin

je n'ai pratiquement jamais utilisé l'objet canvas, du coup je sais pas comment il fonctionne.

Faudrait isoler un peu plus le problème... tu pourrait sortir (via logcat) des valeurs de width et height de ta fonction drawElement ?
pour voir déjà à ce niveau la résolution du bitmap :

Code:

bitmapWall = Bitmap.createBitmap(width, height, Config.ARGB_8888);
18/05/2015, 11h55
Altak
Lut,

Pour éviter que ton appli plante sur le chargement de tes bitmap, englobe le avec un try/catch :
Code:

1 2 3 4 5 6 7 try { ... } catch (OutOfMemoryError e) { e.printStackTrace(); return null; }
Ca ne supprime pas le probleme mais ca évite que ton appli crash en plein utilisation.
18/05/2015, 13h48
eulbobo
Citation:
Envoyé par Altak

Code:

1 2 3 4 5 6 7 try { ... } catch (OutOfMemoryError e) { e.printStackTrace(); return null; }
TRES MAUVAIS IDEE !
Si à un moment on a une erreur de type Error, ça veut dire qu'on est dans un état "non récupérable", donc que quoi qu'il arrive, ce qui se passe après risque de péter dans tous les sens. Une Error, ça veut dire que la JVM jette l'éponge. Quand la JVM jette l'éponge, on évite de continuer à lui donner du boulot !

Je t'invite à aller lire la page suivante pour gérer correctement le chargement de Bitmap (avec les options de chargement) :
http://developer.android.com/trainin...aps/index.html
18/05/2015, 14h07
Altak

Je dois avouer ne pas connaitre les best practice pour ce genre de chose mais si l'utilisation de tes bitmap n'est pas critique pour le fonctionnement de ton app, je ne vois pas ou est le probleme... :x
J'utilise ce try/catch pour le chargement d'image que je récupère depuis un web service. Lorsque cela arive (il est possible que je doivent récuperer de trés grande image par exemple...), je clean la mémoire et j'affiche une image par default.
Cela ne change en rien le comportement de mon application...
18/05/2015, 14h22
eulbobo

Citation:

Envoyé par Altak

Je dois avouer ne pas connaitre les best practice pour ce genre de chose mais si l'utilisation de tes bitmap n'est pas critique pour le fonctionnement de ton app, je ne vois pas ou est le probleme... :x

Le problème c'est que tu corriges un problème en utilisant une solution qui "tombe en marche".
Tu n'as strictement aucune garantie que ton application pourra continuer de fonctionner normalement quand ta JVM te balance une Error. Pour l'instant ça marche, tant mieux pour toi. Mais j'ai envie de dire que tu as de la chance.

Tu as de la chance parce que le "je clean la mémoire" fonctionne. Normalement, en java, la gestion de la mémoire est entièrement pilotée par la JVM, et l'appel à System.gc() "peut" nettoyer la mémoire.. Mais ça peut aussi ne rien faire du tout !
Et il y a des cas dans lesquels quoi qu'il arrive tu ne pourras pas nettoyer la mémoire (le cas où la JVM passe plus de temps à essayer de faire de la place avec le garbage collector qu'à faire avancer le programme).

De manière générale, personne n'a aucune garantie du fonctionnement interne des fonctions natives qui dépendent complètement du système sur lequel elles tournent (et donc de la version de Java installée, sur quel OS). Donc il faut éviter de trop compter dessus parce que ça marche chez toi !
18/05/2015, 15h59
Altak

Re,

Dans tout les cas, je n'ai aucune idée de la taille des images que je peut devoir afficher.
Ce catch est comme une sécurité supplémentaire puisque de toute façon, je ne peux pas éviter le outOfMemory et comme l'affichage de ces images n'est pas indispensable au fonctionnement de mon appli, Cela m’évite de planter et l'utilisateur a juste une image par défaut (j'ai tester sur 3 devices différents qui ont des caractéristiques complétement différente).

Pour mes prochaines appli, je passerai sans doute par des lib du genre Picasso. Ca m’évitera de gérer ca moi-même :p

GL HF