Une lecture de fichier midi qui consomme trop de mémoire

Version imprimable

11/04/2008, 22h17
padodanle51

Une lecture de fichier midi qui consomme trop de mémoire

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
11/04/2008, 22h27
slashrod

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

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

Code:

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();

12/04/2008, 09h40
natha

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

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

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
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) 
        {}
    }

12/04/2008, 11h37
natha

Code:

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.
12/04/2008, 11h52
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