Discussion :

[QuantumSoul]

Bonjour,

j'ai un petit problème dans mon programme que je n'arrive pas a comprendre : le principe est que au début je créé un nouveau thread qui exécute une fonction bien précise et, dans le main, une interface attend un évènement de la souris. Si celui-ci est détecté, on arrête le thread courant et on le lance à nouveau sous le même nom avec un argument différent. Ce qui donne (succinctement) :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
pthread_t MusicThread;
pthread_create(&MusicThread, NULL, play_musique, &fichier);
 
if (bouton_clic())
{
   pthread_cancel(MusicThread);
   fichier.position_rep++;
   pthread_create(&MusicThread, NULL, play_musique, &fichier);
}

Mon problème étant que, pour une raison inconnue, quand l'évenement est détecté le thread ne se relance pas du tout. Tout est bien détecté et la variable est bien modifiée mais impossible de créer à nouveau le thread.

Avez-vous une idée ?
Merci d'avance.



PS: j'utilise Code::Blocks et MinGW sous Windows.

[imperio]

Dans l'exemple donné dans le man, la fonction est appelée à l'intérieur du thread, mais comme elle prend un identifiant, je ne suis pas sûr que ce soit requis. Par-contre, il serait bien que tu attendes la fin de l'exécution de ton thread pour être sûr que tout est bon (avec pthread_join). Peux-tu ajouter ça et voir si ça résout ton problème ?

[QuantumSoul]

Bonjour imperio,

merci de ta réponse j'ai bien ajouté pthread_join, ce me semble en effet un peu plus clean. ceci dit le programme continue de planter
Mais je peut être un peu plus précis: j'ai l'impression que c'est lors de l'annulation du thread qu'il y a un problème, il plante à peu près à cet endroit…

[imperio]

Je pense que c'est le code qui se situe dans ton thread qui pose problème. Pourrais-tu mettre tout ton code ?

[QuantumSoul]

C'est ce que je me suis dis pendant un moment mais je n'ai jamais pu trouver ce qui pouvait causer ce problème.
Voici le code contenu dans le thread:

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
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
 
#include <stdlib.h>
#include <stdio.h>
#include <fmod.h>
#include <dirent.h>
#include <pthread.h>
#include <SDL2/SDL.h>
#include "main.h"
 
void play_musique(MUSIC_FILE *Fichier)
{
 
    FMOD_SYSTEM *Systeme;
    FMOD_System_Create(&Systeme);
    FMOD_System_Init(Systeme, 1, FMOD_INIT_NORMAL, NULL);
    FMOD_BOOL Lecture = 1;
 
    unsigned int longeur;
 
    if (strstr(Fichier->chemin, ".mp3")!=NULL || strstr(Fichier->chemin, ".wav")!=NULL || strstr(Fichier->chemin, ".wmv")!=NULL)
    {
        FMOD_SOUND *Musique=NULL;
        if (FMOD_System_CreateStream(Systeme, Fichier->chemin, FMOD_2D | FMOD_LOOP_NORMAL, 0, &Musique)==-1) exit(EXIT_FAILURE);
        FMOD_Sound_GetLength(Musique, &longeur, FMOD_TIMEUNIT_MS);
        FMOD_Sound_SetLoopCount(Musique, Fichier->Loop_Status);
        FMOD_System_PlaySound(Systeme, Musique, 0, 0, &Fichier->cannal);
 
        SDL_Delay(longeur);
        while (Lecture!=0) FMOD_Channel_IsPlaying(Fichier->cannal, &Lecture);
    }
 
    else
 
    {
 
        DIR *repertoire=NULL;
        struct dirent *fichier_musique = NULL;
        repertoire = opendir(Fichier->chemin);
        if (repertoire==NULL) exit(EXIT_FAILURE);
        char nom_fichier[128];
        int i;
        if (Fichier->liste_fichiers == NULL)
        {
            if ((Fichier->liste_fichiers=malloc(2*sizeof(long))) == NULL) exit(EXIT_FAILURE);
            Fichier->liste_fichiers[0] = 3;
            for (i=0; i<2; i++) fichier_musique = readdir(repertoire);
            i = 1;
            while ((fichier_musique=readdir(repertoire)) != NULL)
            {
                if (strstr(fichier_musique->d_name, ".mp3")!=NULL || strstr(fichier_musique->d_name, ".wav")!=NULL || strstr(fichier_musique->d_name, ".wmv")!=NULL)
                {
                    Fichier->liste_fichiers[i] = telldir(repertoire);
                    if ((Fichier->liste_fichiers = realloc(Fichier->liste_fichiers, sizeof(long))) == NULL) exit(EXIT_FAILURE);
                    i++;
                }
            }
            Fichier->liste_fichiers[i] = -1;
        }
        seekdir(repertoire, Fichier->liste_fichiers[Fichier->position_rep]-1);
        do
        {
            fichier_musique = readdir(repertoire);
            if (fichier_musique!=NULL)
            {
 
                strcpy(nom_fichier, Fichier->chemin);
                strcat(nom_fichier, fichier_musique->d_name);
 
                if (strstr(nom_fichier, ".mp3")!=NULL || strstr(nom_fichier, ".wav")!=NULL || strstr(nom_fichier, ".wmv")!=NULL)
                {
                    FMOD_SOUND *Musique=NULL;
                    if (FMOD_System_CreateStream(Systeme, nom_fichier, FMOD_2D | FMOD_LOOP_NORMAL, 0, &Musique) != -1)
                    {
                        FMOD_Sound_GetLength(Musique, &longeur, FMOD_TIMEUNIT_MS);
                        FMOD_Sound_SetLoopCount(Musique, Fichier->Loop_Status);
                        FMOD_System_PlaySound(Systeme, Musique, 0, 0, &Fichier->cannal);
 
                        SDL_Delay(longeur);
                        while (Lecture!=0) FMOD_Channel_IsPlaying(Fichier->cannal, &Lecture);
                    }
                    Fichier->position_rep++;
                    seekdir(repertoire, Fichier->liste_fichiers[Fichier->position_rep]-1);
                }
            }
        }
        while (fichier_musique != NULL);
 
        free(Fichier->liste_fichiers);
        closedir(repertoire);
    }
    FMOD_System_Close(Systeme);
    FMOD_System_Release(Systeme);
    pthread_exit(0);
}

et voici le main.h

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
 
#ifndef MAIN_H_INCLUDED
#define MAIN_H_INCLUDED
#include <string.h>
#include <fmod.h>
#include <SDL2/SDL.h>
 
typedef struct {
    SDL_Rect Coord;
    SDL_Texture* texture_mvmt;
    SDL_Texture* texture_clic;
} BOUTON;
 
typedef struct {
    char chemin[128];
    long position_rep;
    long *liste_fichiers;
    FMOD_CHANNEL *cannal;
    float Volume_Status;
    unsigned int Loop_Status;
    FMOD_BOOL Pause;
} MUSIC_FILE;
 
#endif // MAIN_H_INCLUDED

[imperio]

Arf, ça commence mal :

Code :

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

1
2
3
FMOD_SYSTEM *Systeme;
FMOD_System_Create(&Systeme);
FMOD_System_Init(Systeme, 1, FMOD_INIT_NORMAL, NULL);

Ça doit être fait une seule et unique fois.

Je ne comprends pas vraiment l'intérêt de cette ligne :

Code :

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

SDL_Delay(longeur);

Un Sleep ça marche bien aussi...

De manière générale, je ne comprends pas l'intérêt de ton thread. On utilise généralement un thread pour effectuer des tâches pour soulager le processus principal (ou parce que lui est occupé à faire autre chose). Je n'ai pas l'impression que ce soit ton cas.