Discussion :

[Sebenf0rce]

Bonjour,

J'ai commencé une application android et j'ai un gros soucis de performances. Actuellement j'affiche par l'intermédiaire d'une sufaceView un background et une image (une bulle) qui se déplace. Le déplacement est saccadé et je ne comprends pas pourquoi.
Voici mon code actuel : https://github.com/SebGonzalez/Sumble

La surfaceView est créé dans la classe SurfaceViewBulle et la classe DrawingThread affiche cette surfaceView dans un thread.


Merci d'avance

[hotcryx]

Dans un premier temps, tu pourrais déjà convertir tes images au format webp, notamment le backgroung...

http://tools.android.com/tech-docs/webp

[hotcryx]

ici il pourrait boucler indéfiniment si le join ne se fait pas, sans pouvoir sortir:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
public void surfaceDestroyed(SurfaceHolder holder) {
        mThread.keepDrawing = false;

        boolean joined = false;
        while (!joined) {
            try {
                mThread.join();
                joined = true;
            } catch (InterruptedException e) { //? on ne sort pas }
        }
}

A partir d'ici, je pense que c'est le code à vérifier car l'Update est dans une boucle While, l'update ayant aussi une boucle For et OnDraw également.
Ca fait beaucoup de travail pour le processeur.
N'y a t'il pas une sorte de DoEvents() comme en VB????
Tu ne laisses pas le processeur gérer ses propres messages.
Je ne parle pas de mettre des sleep!!!! (horreur) mais de laisser le processeur "s'auto gérer".

Code :

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

1
2
3
4
5
6
synchronized (mSurfaceHolder) {
                        //on déplace la bulle
                        update();
                        // Et on dessine
                        onDraw(canvas);
}

Pense à réutiliser les variables:
Ex: For(Bulle bulle : ...) => changer la boucle (en for..to par exemple) afin de réutiliser bulle et à le définir en dehors de la boucle.

[Sebenf0rce]

Ok j'ai donc converti mon background en webp et ajouté un " e.printStackTrace();" dans mon catch.
Pour le for du update et du onDraw j'ai essayé de l'enlever et d'affecter le déplacement et l'affichage seulement au premier élément de la liste mais j'ai toujours ce soucis de latence lors du déplacement.

Pour les sleep j'ai vu ça sur un tuto d'openClasseroom, apparemment c'était pour "afficher l'écran à 50fps".

[hotcryx]

Tu utilises beaucoup de mémoire (comme la variable bulle) avec des images.
As-tu bien fait les déréférences? bulle = null; avant de lui réaffecter un objet?

Si ces objets sont visibles à l'écran, n'est ce pas plus simple de changer leurs coordonnées plutôt que de les effacer et redessiner?
Si l'image ne change pas, ce serait une perte de temps d'essayer de le redessiner...

As-tu pensé à l'accélération graphique? (dans le manisfeste).

Quel est ton moteur de rendu?

Les moteurs de rendu ont déjà été confrontés à ce genre de problème...

[Sebenf0rce]

Je ne récréé pas les objet bulle à chaque fois normalement. J'utilise juste la méthode update qui modifie leurs coordonnées et je les ré affiche avec la méthode onDraw.

Je n'ai rien trouvé sur le web pour activer l'accélération graphique.

Pour le moteur de rendu je crois que je n'en utilise pas