Application bloquée à la première exécution

Version imprimable

19/09/2006, 19h34
chris28200

Application bloquée à la première exécution

Bonjour,

J'ai développé une application libre en Java pour l'apprentissage de la lecture musicale : http://www.jalmus.net

Il s'agit d'executables générés par JBuilder que j'installe avec Izpack. J'ai eu plusieurs retours (XP,MAC) indiquant que le logiciel bloque à la première exécution avec une utilisation à 90% du CPU.

Pouvez-vous la tester et d'où ce problème pourrait-il venir ?

Merci,

Christophe
19/09/2006, 20h38
wizaord

Je viens de télécharger et lancer ton application. Effectivement, au lancement de l'appli, 100% du proc est utilisée.
Mais ca me le fait à chaque lancement. A mon avis, il doit y avoir une boucle infinie quelque part :)
19/09/2006, 20h49
®om

Citation:

Envoyé par chris28200

Bonjour,

J'ai développé une application libre en Java pour l'apprentissage de la lecture musicale : http://www.jalmus.net

Il s'agit d'executables générés par JBuilder que j'installe avec Izpack. J'ai eu plusieurs retours (XP,MAC) indiquant que le logiciel bloque à la première exécution avec une utilisation à 90% du CPU.

Pouvez-vous la tester et d'où ce problème pourrait-il venir ?

Merci,

Christophe

Tu pourrais faire un .zip des sources (je n'ai pas d'outil svn -ou je ne sais pas comment m'en servir-) merci :)

EDIT : j'avais pas vu, ça installe les sources :)

A priori je dirais que ta classe RenderingThread en est la cause (vu le code o_O):

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
while (true) {
             try {
               if (ecranjeu == 1) {
 
                 if (type2 == "NOTES" | type2 == "ALTERATIONS") {
                   sleep(vitesse);
 
                 }
                 else if (type2 == "INTERVALLES")
                   sleep(vitesse * 3 / 2);
                 else if (type2 == "ACCORDS")
                   sleep(vitesse * 2);
                 else
                   sleep(vitesse + 18);
 
                 if (parti) {
                   if (type == "NORMAL" &
                       (type2 == "NOTES" | type2 == "ALTERATIONS"))
                     ncourante.setX(ncourante.getX() + 1);
                   else if (type == "NORMAL" & type2 == "ACCORDS")
                     acourant.avance(1);
                   else if (type == "NORMAL" & type2 == "INTERVALLES")
                     icourant.avance(1);
                   else if (type == "LIGNE" &
                            (type2 == "NOTES" | type2 == "ALTERATIONS"))
                     for (int i = 0; i < ligne.length; i++) {
 
                       ligne[i].setX(ligne[i].getX() - 1);
                     }
 
                   else if (type == "LIGNE" & type2 == "ACCORDS")
                     for (int i = 0; i < ligneacc.length; i++) {
                       ligneacc[i].avance( -1);
                     }
 
                   else if (type == "LIGNE" & type2 == "INTERVALLES")
                     for (int i = 0; i < ligneint.length; i++) {
                       ligneint[i].avance( -1);
 
                     }
                 }
                 panelanim.repaint();
               }
             }
             catch (Exception e) {}
           }
         }

Déjà, un while(true) suivi d'un if, si la condition du if est fausse, ça fait une attente active qui prend énormément de ressources. Ensuite, tu ne devrais pas avoir besoin de demander un rafraichissement basé sur le temps (toutes les 10ms), mais plutôt basé sur des évènements. Et de plus, tout ce qui est affichage graphique, c'est à faire dans l'EDT...

(sinon, tu faisais du C avant, non? je dis ça pour ta classe jalmus.java et les déclarations de variables interminables^^. Et le code est plutôt "impératif" que "objet")

19/09/2006, 22h06
chris28200

Merci beaucoup wizaord et ®om. La condition dans le thread était bien la cause du problème de CPU : je vais demander aux utilisateurs si cela résoud leur blocage.

J'ai d'abord programmé en C et j'apprend le Java au fur et à mesure que j'améliore mon application : je rajoute des fonctionnalités quand le temps me le permet sans vraiment avoir de plan et le code n'est pas très propre (et encore j'ai fait un petit peu le ménage).

Pour l'affichage graphique dan l'EDT peux tu me donner des détails. Merci.

Christophe
19/09/2006, 23h30
sinok

Vas voir le tuto suivant pour ce qui concerne l'EDT:

http://gfx.developpez.com/tutoriel/j...ing-threading/
20/09/2006, 03h20
bouye

Un rafraichissement automatique peut quand meme avoir son utilite mais c'est clair qu'il faut laisser le temps a l'EDT de pouvoir s'executer.
=>une Thread avec un appel a la methode sleep() a la fin de la boucle. Attentions au interractions entre cette thread et l'EDT cependant.
=>ou un Timer Swing qui se repetera a intervalle regulier.

Rajouter une demande de raffraichissement lors de la reception d'evenements c'est tres bien aussi (mais ca demandera plus d'adaptation du code C-like).
20/09/2006, 18h31
chris28200

Merci beaucoup pour votre aide, je vais essayer d'optimiser un peu mon animation.

Christophe.