Discussion :

[raspac]

Bonjour,
voilà je dois réaliser un programme fonctionnant avec l'aide des thread de la manière suivante :

- Un processus ( thread principal ) crée un thread
- Ce thread reçoit un nombre en paramètre qu'il affichera à l'écran, à la fin de son traitement, il renvoit ce nombre multiplié par 2.
- le thread principal attend la terminaison du thread "fils".
- Une fois terminé, le thread principal doit afficher à l'écran le nombre renvoyé par le thread fils.

La compilation passe. Ce qui est bizzare c'est que l'adresse retournée par le traitementTh1 et l'adresse reçue est la même. Ca doit coincer au niveau de ces deux lignes :

Code :

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

1
2
3
 
retVal=*((int *)retValTh1);
printf("\nValeur de la variable recu : %d\n", retVal);

Voici mon 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
35
36
37
38
39
 
void * traitementTh1(void * nbParamTh1);
 
int main()
{
        pthread_t idTh1;
        int errTh1, errAttenteTh1;
        int paramTh1; int retVal;
        void * retValTh1;
 
        // création du premier thread
        paramTh1=112;
        errTh1=pthread_create(& idTh1, NULL, traitementTh1, (void *) & paramTh1);
        if(errTh1!=0)
        { perror("Echec creation du premier thread"); exit(1); }
 
 
        // attente de la terminaison du premier thread
        errAttenteTh1=pthread_join(idTh1, &retValTh1);
        if(errAttenteTh1!=0)
        { perror("Echec attente terminaison thread 1"); exit(1); }
        else
        {
                retVal=*((int *)retValTh1);
                printf("\nValeur de la variable recu : %d\n", retVal);
        }
}
 
void * traitementTh1(void * nbParamTh1)
{
        int nb, nbMult;
 
        // conversion du type en entier du paramètre
        nb=*((int *) nbParamTh1);
        printf("\nParametre du thread1 : %d \n", nb);
 
        nbMult=nb * 2;
        pthread_exit((void *) &nbMult);
}

Merci d'avance.

[Emmanuel Delahaye]

Voici mon code :

On pourrait avoir la version qui compile, parce que là...

Project : Forums
Compiler : GNU GCC Compiler (called directly)
Directory : C:\dev\forums2\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `main':
main.c:6: error: `pthread_t' undeclared (first use in this function)
main.c:6: error: (Each undeclared identifier is reported only once
main.c:6: error: for each function it appears in.)
main.c:6: error: syntax error before "idTh1"
main.c:13: warning: implicit declaration of function `pthread_create'
main.c:13: error: `idTh1' undeclared (first use in this function)
main.c:13: error: `NULL' undeclared (first use in this function)
main.c:15: warning: implicit declaration of function `perror'
main.c:15: warning: implicit declaration of function `exit'
main.c:19: warning: implicit declaration of function `pthread_join'
main.c:25: warning: implicit declaration of function `printf'
main.c: In function `traitementTh1':
main.c:38: warning: implicit declaration of function `pthread_exit'
Process terminated with status 1 (0 minutes, 2 seconds)
6 errors, 6 warnings

A part ça, ceci est horrible :

Code :

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

1
2
3
4
5
6
7
 
void * traitementTh1(void * nbParamTh1)
{
        int nb, nbMult;
<...>
        pthread_exit((void *) &nbMult);
}

Tous ces casts sont inutiles
Tu ne peux pas transmettre l'adresse d'une variable qui n'existe plus...

1 - On définit un variable dans le main()
2 - On en passe l'adresse au thread
3 - Le thread récupère l'adresse, modife la variable.
C'est tout.

Ceci fonctionne (il faut rester simple...)

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
 
#include <pthread.h>
#include <stdio.h>
#include <stdlib.h>
 
void *traitementTh1 (void *nbParamTh1)
{
   int *p_nbParamTh1 = nbParamTh1;
 
/* lecture de la variable */
   printf ("\nParametre du thread1 : %d \n", *p_nbParamTh1);
 
/* modification de la variable*/
   (*p_nbParamTh1) *= 2;
 
   return NULL;
}
 
int main ()
{
   int ret = EXIT_SUCCESS;
   pthread_t idTh1;
 
   // création du premier thread
   int paramTh1 = 112;
 
   printf ("\nValeur de la variable : %d\n", paramTh1);
   {
      int err = pthread_create (&idTh1, NULL, traitementTh1, &paramTh1);
      if (err)
      {
         perror ("Echec creation du premier thread");
         ret = EXIT_FAILURE;
      }
      else
      {
         // attente de la terminaison du premier thread
         err = pthread_join (idTh1, NULL);
         if (err)
         {
            perror ("Echec attente terminaison thread 1");
            ret = EXIT_FAILURE;
         }
         else
         {
            printf ("\nValeur de la variable : %d\n", paramTh1);
         }
      }
   }
   return ret;
}

Valeur de la variable : 112

Parametre du thread1 : 112

Valeur de la variable : 224

Press ENTER to continue.

[Médinoc]

Et ta procédure de thread retourne l'adresse d'une variable locale.
Très mauvais.