Discussion :

[cecile38]

Bonjour,

J'ai une application ou il y a un bouton qui permet d'effacer ce que l'utilisateur a dessiné.

L'application fonctionne bien sauf sur Android 4.1.1 où quand on clique sur le bouton, l'application se ferme.

Voici le code du onClick du bouton :

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
 
            Bitmap precedentDessin;
            private ArrayList<Path> dessin = new ArrayList<Path>();
 
            .......
 
            public void onClick(View v) {
 
            	   precedentDessin = null;
                dessin.clear();
                LinearLayout ll = (LinearLayout) findViewById(R.id.dessin);
                ll.removeAllViews();
                panel = new Panel(DessinActivity.this);
                ll.addView(panel);
            }

Merci d'avance pour votre aide

[Sake_reflex]

Le logcat est ton ami, tout du moins le notre si tu veux qu'on puisse t'aider

[cecile38]

L'erreur est en fait dans la fonction suivante :

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
 
 
@Override
        public void run() {
            Canvas c;
            while (_run) {
                c = null;
                try {
                    c = _surfaceHolder.lockCanvas(null);
                    synchronized (_surfaceHolder) {
                        _panel.onDraw(c);
                    }
                } finally {
                    // do this in a finally so that if an exception is thrown
                    // during the above, we don't leave the Surface in an
                    // inconsistent state if (c != null) {
                    _surfaceHolder.unlockCanvasAndPost(c);
                }
            }
        }

J'obtient l'exception suivante :
java.lang.illegalArgumentException
at android.view.Surface.unlockCanvasAndPost(Native Method)

[nicroman]

heu... à vue de nez je dirait qu'il y a un problème de gestion de ressources...

L'allocation de la ressource doit être faite en dehors du bloc try/finally qui lui protège le code utilisant la ressource:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
c = _surfaceHolder.lockCanvas(null);
try {
    synchronized (_surfaceHolder) {
        _panel.onDraw(c);
    }
} finally {
                    // do this in a finally so that if an exception is thrown
                    // during the above, we don't leave the Surface in an
                    // inconsistent state if (c != null) {
     _surfaceHolder.unlockCanvasAndPost(c);
}

Si le lock du canvas foire, il faut pas faire de unlockCanvasAndPost (d'autant que 'c' est sans doute null)

[cecile38]

Ok. merci!!