Discussion :

[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

[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

[®om]

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

[®om]

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

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
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")

[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

[sinok]

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

http://gfx.developpez.com/tutoriel/j...ing-threading/

[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).