Discussion :

[Serialk]

Bonjour à tous.
J'ai commencé à concevoir un programme qui lit des partitions de musiques en faisant des bip sonores. Seulement, l'opération de lecture du fichier contenant un while (berk), j'ai du employer la classe QThread pour éviter que le programme ne freeze pendant la lecture. Avec l'aide de mon cher ami Alpounet, j'ai donc ainsi réussi à pondre ce code, qui ne marche pas plus que l'ancien car le programme freeze également à la manière de l'autre :

Code main.cpp :

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
 
#include <QApplication>
#include <QtGui>
#include "ui_FenBeep.h"
#include "FenBeep.h"
#include <windows.h>
#include <cstdlib>
#include <string>
#include <iostream>
#include <sstream>
#include <fstream>
#include <map>
 
 
int readFile( const char *file )
{
    std::ifstream musicFile(file);
    if (! musicFile.is_open() )
        return 1;
 
//définitions de variables
 
    while ( std::getline( musicFile, line ) )
    {
//berk, c'est ici que j'ai ma boucle qui lit le fichier
    }
    return 0;
}
 
int main(int argc, char *argv[])
{
    QApplication app(argc, argv);
 
    FenBeep fenetre;
 
 
    fenetre.show();
 
    return app.exec();
}

Code FenBeep.h :

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
48
#ifndef HEADER_FENCLIENT
#define HEADER_FENCLIENT
 
#include <QtGui>
#include "ui_FenBeep.h"
 
 
int readFile( const char *file );
 
 
class FenBeep : public QWidget, private Ui::FenBeep
{
    Q_OBJECT
 
    public:
        FenBeep(QWidget *parent = 0);
        QString partition;
 
 
    private slots:
        void on_boutonSelect_clicked();
	void on_boutonPlay_clicked();
 
 
    private:
        Ui::FenBeep ui;
 
 
};
 
 
class ThreadLecture : public QThread
{
    Q_OBJECT
 
    public:
            ThreadLecture(const char*);
            const char* filename;
 
    public slots :
            void run();
 
    signals :
};
 
 
 
#endif

Code FenBeep.cpp :

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
48
 
#include <QApplication>
#include <QString>
#include <QtGui>
#include "FenBeep.h"
#include "ui_FenBeep.h"
 
 
FenBeep::FenBeep(QWidget *parent) : QWidget(parent)
{
    setupUi(this);
    setAttribute(Qt::WA_QuitOnClose);
    boutonPlay->setIcon(QIcon("icone-play.png"));
}
 
void FenBeep::on_boutonSelect_clicked()
{
    partition = QFileDialog::getOpenFileName(this, "Ouvrir un fichier", QString("music\\"), "Partitions Beep (*.bip)");
    if (partition == NULL)
    {
        QMessageBox::warning(this, "Erreur", "La partition n'a pas pu être chargée.");
    } else {
        cheminPartition->setText(partition);
    }
}
 
void FenBeep::on_boutonPlay_clicked()
{
    if (partition == NULL)
    {
        QMessageBox::warning(this, "Erreur", "Veuillez selectionner une partition");
    } else {
        const char *file = partition.toStdString().c_str();
        ThreadLecture monthread(file);
        monthread.run();
 
    }
}
 
void ThreadLecture::run()
{
    readFile(filename);
}
 
ThreadLecture::ThreadLecture( const char *fichier )
{
    filename = fichier;
}

Auriez vous une quelconque idée de la provenance du freeze ? Je vous remercie.

[yan]

Code :

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

1
2
ThreadLecture monthread(file);
monthread.run();

1- il ne faut as appele rub mais start. Si tu appel run tu execute run dans le thread principale et tu fait comme avant

2- attention au scope de ton QThread. SInon ton thread sera détruit avant d'avoir commencé

[Serialk]

Effectivement, je l'avais lu dans la FAQ mais j'ai complètement oublié d'appeler start, comme l'exemple est uniquement avec des connects, je ne savais pas si il fallait faire run ou start.

Sinon c'est quoi un "scope" ? google n'est pas mon ami sur ce coup là ...

(Depuis ma 1ère modification le programme s'arrète au while .

[yan]

c'est la durée de vie de l'objet.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void FenBeep::on_boutonPlay_clicked()
{
    if (partition == NULL)
    {
        QMessageBox::warning(this, "Erreur", "Veuillez selectionner une partition");
    } 
    else 
    {
        const char *file = partition.toStdString().c_str();
        ThreadLecture monthread(file); <- creation de monthread
        monthread.run();
    }<- destruction de monthread
}

[Serialk]

Oui, alp m'a expliqué, mais ou il faudrait que j'instancie alors ? et comment ?

<Alpounet> dans FenBeep
<Alpounet> tu mets un attribut ThreadLecture

Code :

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

1
2
3
4
5
6
7
8
class FenBeep : public QWidget, private Ui::FenBeep
{
    Q_OBJECT
 
    public:
        FenBeep(QWidget *parent = 0);
        QString partition;
        ThreadLecture;

dnas le constructeur tu l'initialises avec les listes d'initialisation

Comment je suis censé initialiser alors que j'ai besoin de mon attribut "fichier" à envoyer à mon thread ?

[Serialk]

Bon grâce à Alp j'ai pu énormément avancer dans mon code.
seulement, mon Thread ne fait qu'appeler une fonction externe, readFile. Or, pour la stopper, j'ai besoin qu'elle fasse partie du Thread. Voici ce qui se passe lorsque je remplace readFile(filename); par la fonction en elle même :

Code cpp :

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
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
#include <QApplication>
#include <QString>
#include <QtGui>
#include "FenBeep.h"
#include "ui_FenBeep.h"
 
 
FenBeep::FenBeep(QWidget *parent) : QWidget(parent)
{
    setupUi(this);
    setAttribute(Qt::WA_QuitOnClose);
    boutonPlay->setIcon(QIcon("icone-play.png"));
    boutonStop->setIcon(QIcon("icone-stop.png"));
}
 
void FenBeep::on_boutonSelect_clicked()
{
    partition = QFileDialog::getOpenFileName(this, "Ouvrir un fichier", QString("music\\"), "Partitions Beep (*.bip)");
    if (partition == NULL)
    {
        QMessageBox::warning(this, "Erreur", "La partition n'a pas pu être chargée.");
        cheminPartition->setText("");
    } else {
        //QMessageBox::information(this, "Partition sélectionée.", "Vous avez sélectionné la partition :\n" + partition);
        cheminPartition->setText(partition);
    }
}
 
void FenBeep::on_boutonPlay_clicked()
{
    if (partition == NULL)
    {
        QMessageBox::warning(this, "Erreur", "Veuillez selectionner une partition");
 
    } else {
        const char *file = partition.toStdString().c_str();
 
 
        monthread.setFileName(file);
        monthread.start();
    }
}
 
void ThreadLecture::run()
{
    std::ifstream musicFile(file);
    if (! musicFile.is_open() )
        return 1;
 
    int octave = 0;
    double duration = 0;
    std::string duration_str;
    std::string line;
    std::string note;
 
 
    while ( std::getline( musicFile, line ) )
    {
        note = "";
        octave = 0;
        duration = 0;
        duration_str = "";
 
        std::stringstream lineStream(line);
        lineStream >> note >> octave >> duration_str;
 
        // Empty line skipping
        if( note == "" )
            continue;
 
        // Tempo management
        std::string::size_type pos = note.find("TEMPO");
        if( pos  != std::string::npos )
        {
            if( octave != 0)
            {
                TEMPO = octave;
                DURATION = TEMPO;
                continue;
            }
        }
 
        // Octave management
        if( octave )
        {
            OCTAVE = octave;
        }
 
        // Duration management
        // (this looks dirty because of fraction interpretation)
        if( duration_str != "" )
        {
            pos = duration_str.find('/');
            if( pos != std::string::npos)
            {
                std::istringstream numerateur( duration_str.substr(0,pos+1) );
                std::istringstream denominateur (duration_str.substr(pos+1, duration_str.size()-(pos + 1)) );
                int num = 1;
                int den = 1;
                numerateur >> num;
                denominateur >> den;
                DURATION = (int) (TEMPO * num / den);
            }
            else
            {
                std::istringstream stream(duration_str);
                stream >> duration;
                DURATION = (int) (TEMPO * duration);
            }
        }
 
        // Silence management
        if (noteMap[note.c_str()] == SILENCE)
            Beep(1, DURATION);
        else
            // Finally, play the note !
            // This calls the "Beep" system function under windows
            // Or the custom one under Linux
            Beep(NOTES[ noteMap[note.c_str()] ][ OCTAVE ], DURATION);
    }
}
 
void ThreadLecture::setFileName(const char* file)
{
    filename = file;
}
 
ThreadLecture::ThreadLecture()
{
    //filename = fichier;
}
 
void FenBeep::on_boutonStop_clicked()
{
        monthread.quit();
}

FenBeep.cpp:46: error: variable `std::ifstream musicFile' has initializer but i
complete type
FenBeep.cpp:48: error: return-statement with a value, in function returning 'vo
d'
FenBeep.cpp:57: error: no matching function for call to `getline(<type error>,
td::string&)'
FenBeep.cpp:64: error: variable `std::stringstream lineStream' has initializer
ut incomplete type
FenBeep.cpp:77: error: `TEMPO' undeclared (first use this function)
FenBeep.cpp:77: error: (Each undeclared identifier is reported only once for ea
h function it appears in.)
FenBeep.cpp:78: error: `DURATION' undeclared (first use this function)
FenBeep.cpp:86: error: `OCTAVE' undeclared (first use this function)
FenBeep.cpp:96: error: variable `std::istringstream numerateur' has initializer
but incomplete type
FenBeep.cpp:97: error: variable `std::istringstream denominateur' has initializ
r but incomplete type
FenBeep.cpp:106: error: variable `std::istringstream stream' has initializer bu
incomplete type
FenBeep.cpp:113: error: `noteMap' undeclared (first use this function)
FenBeep.cpp:113: error: `SILENCE' undeclared (first use this function)
FenBeep.cpp:114: error: `Beep' undeclared (first use this function)
FenBeep.cpp:119: error: `NOTES' undeclared (first use this function)

Voici le code de mon main.cpp :

Code cpp :

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
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
#include <QApplication>
#include <QtGui>
#include "ui_FenBeep.h"
#include "FenBeep.h"
#include <windows.h>
#include <cstdlib>
#include <string>
#include <iostream>
#include <sstream>
#include <fstream>
#include <map>
 
 
 
        int NOTES[12][9] =
        {
            {16, 33, 65, 131, 262, 523, 1046, 2093, 4186},
            {17, 35, 69, 139, 277, 554, 1109, 2217, 4435},
            {18, 37, 73, 147, 294, 587, 1175, 2349, 4699},
            {19, 39, 78, 155, 311, 622, 1244, 2489, 4978},
            {21, 41, 82, 165, 330, 659, 1328, 2637, 5274},
            {22, 44, 87, 175, 349, 698, 1397, 2794, 5588},
            {23, 46, 92, 185, 370, 740, 1480, 2960, 5920},
            {24, 49, 98, 196, 392, 784, 1568, 3136, 6271},
            {26, 52, 104, 208, 415, 831, 1661, 3322, 6645},
            {27, 55, 110, 220, 440, 880, 1760, 3520, 7040},
            {29, 58, 116, 233, 466, 932, 1865, 3729, 7459},
            {31, 62, 123, 245, 494, 988, 1975, 3951, 7902}
        };
 
        enum
        {
            SILENCE = -1,
            DO = 0,
            DO_ = 1,
            RE = 2,
            RE_ = 3,
            MI = 4,
            FA = 5,
            FA_ = 6,
            SOL = 7,
            SOL_ = 8,
            LA = 9,
            LA_ = 10,
            SI = 11
        };
 
        enum
        {
            OCTAVE_0 = 0,
            OCTAVE_1 = 1,
            OCTAVE_2 = 2,
            OCTAVE_3 = 3,
            OCTAVE_4 = 4,
            OCTAVE_5 = 5,
            OCTAVE_6 = 6,
            OCTAVE_7 = 7,
            OCTAVE_8 = 8,
        };
 
 
        std::map<std::string, int> noteMap;
 
        // Global variables
        int TEMPO = 250;
        int DURATION=TEMPO;
        int OCTAVE = 0;
 
 
 
 
//la fonction readFile(filename) se trouvait ici auparavant.
 
 
int main(int argc, char *argv[])
{
    QApplication app(argc, argv);
 
        noteMap["DO"] = DO;
        noteMap["DO_"] = DO_;
        noteMap["RE"] = RE;
        noteMap["RE_"] = RE_;
        noteMap["MI"] = MI;
        noteMap["FA"] = FA;
        noteMap["FA_"] = FA_;
        noteMap["SOL"] = SOL;
        noteMap["SOL_"] = SOL_;
        noteMap["LA"] = LA;
        noteMap["LA_"] = LA_;
        noteMap["SI"] = SI;
        noteMap["_"] = SILENCE;
 
    FenBeep fenetre;
 
 
 
    fenetre.show();
 
    return app.exec();
}

Une explication ?