Discussion :

[padodanle51]

Bonjour,
Voilà je dois réaliser un piano virtuel et à chaque fois que j'appuie sur une touche, je lis un fichier midi (correspondant à la note)
Problème: A chaque fois que je lis, je créai un nouvel objet qui d'après moi devrai être détruit quand je sors de la fonction. Pourtant la mémoire prise par mon prog ne fait que augmenter!
Merci de bien vouloir m'aider

[slashrod]

Tu peux poster la portion qui t'embêtes?

[padodanle51]

Bien sur. Donc voilà la portion qui pose prob:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
switch (arg0.getKeyChar()) 
{
	case 'a':
		lectureMidi pm = new lectureMidi("do.mid");
		pm.start();
		break;
	case 'z':
		lectureMidi pm1 = new lectureMidi("re.mid");
		pm1.start();
		break;
}
System.gc();

[natha]

C'est plutôt le code de la classe LectureMidi qu'il va falloir là

[padodanle51]

En effet, désolé. Donc voilà le code:

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
Thread cadenceur;
    Sequence fichierSon;
    Sequencer lecteur;
    String cheminFichier;
 
    public lectureMidi(String son) 
    {       
        this.cheminFichier = son;
     }
 
   public void run() 
    {
        try 
        {
            File file = new File(this.cheminFichier);
            this.fichierSon = MidiSystem.getSequence(file);
            this.lecteur = MidiSystem.getSequencer();
            this.lecteur.open();
            this.lecteur.setSequence(fichierSon);
            this.lecteur.start();
            while (this.lecteur.isRunning()) 
            {
                try
                {
                    Thread.sleep(10000);
 
                } catch (InterruptedException e) { }
            }
            this.lecteur.stop();
            this.lecteur.close();	
        } 
        catch (Exception e) 
        {}
    }

[natha]

Code :

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

Thread.sleep(10000);

Ca me parait un peu long, attendre 10s. Imaginons un virtuose qui joue 20 notes en 5 secondes, tu auras 20*2 = 40 threads qui tournent au bout de 10s. Un peu énorme pour ce que c'est.

Sinon, je ne vois pas à quoi sert Thread cadenceur; ni la déclaration de ta classe lectureMidi.

[padodanle51]

En effet le Thread.Sleep était trop long, c'est ce qui faisait que ça ramé. Concernant la place en mémoire, en fait au fur et a mesure, les thread se fermaient, mais les Sequencer restaient en mémoire. La solution:
utiliser un seul Sequencer que je passe à paramètre à mes Threads.
Merci pour votre aide
Bon Week-end