Discussion :

[Nymar]

bonjour,
Je suis arrivé à écrire ce code et je veux vous poser quelques questions.

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
#include <pthread.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
#include <sys/time.h>
#define nbre_cycle 10
#define nbre_trame 2
#define PTU 1
 
////////////////////////////////////////////////////////////////////
typedef struct signaux_thread
{
   int sync ;
  char *(*schedule_table)[nbre_trame];
}signaux_thread;
////////////////////////////////////////////////////////////////////
void *func_thread (void *arg) 
{
  int j;
  struct signaux_thread * signal= (struct signaux_thread *)arg;   
  fprintf(stderr,"sync frame number: %d\n ",signal->sync);
 
  for (j=0;j<nbre_trame;j++)
	    {
	      fprintf(stderr,"la trame %s est envoyée\n",signal->schedule_table[signal->sync][j]);	   
	    }
 
}
///////////////////////////////////////////////////////////////////
int main(void)
{
  int i,j;
 
  pthread_t thread;
  char *ScheduleTable[nbre_cycle][nbre_trame];
  struct signaux_thread arg;
  arg.sync = 0;
  arg.schedule_table = ScheduleTable;
  for ( i=0;i<nbre_cycle;i++)
    {
      for ( j=0;j<nbre_trame;j++)
	{
	  ScheduleTable[i][j]=(char*)malloc(3 * sizeof(char));
	  sprintf(ScheduleTable[i][j],"%d.%d",i,j);
	}
    }
  while(1)
    {
      if(pthread_create(&thread,NULL,func_thread,&arg)!=0)
	{
	  fprintf(stderr,"can't create thread");
	  exit(EXIT_FAILURE);
	}
      arg.sync++;
       if (arg.sync==nbre_cycle)
	 {arg.sync=0;}
      sleep(PTU);
    }
}

1/ je trouve des difficultés a comprendre toutes instructions(surtout les prtotypes) contenant ( void *) ou (int *) et d'autres formes.
comme

• void *func_thread (void *arg)
• struct signaux_thread * signal= (struct signaux_thread *)arg;
• char *(*schedule_table)[nbre_trame];

2/ et pour l’exécution de ce code voila comment ça se passe

Code :

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

1
2
3
4
5
6
7
8
9
sync frame number: 1
 la trame 1.0 est envoyée
la trame 1.1 est envoyée
sync frame number: 2
 la trame 2.0 est envoyée
la trame 2.1 est envoyée
sync frame number: 3
 la trame 3.0 est envoyée
la trame 3.1 est envoyée

alors que je pensais que ça doit commencer par 0 et non par 1

[Metalman]

Le void* signifie souvent que tu peux mettre "ce que tu veux", à l'inverse de void tout seul qui signifie "rien du tout".

Les int* en paramètre sont souvent associés au fait que la fonction/procédure modifiera la valeur... OU qu'il s'agit d'un tableau d'entiers qui est lu et/ou modifié.

Ce qui me frappe le plus c'est ton "char *(*schedule_table)[nbre_trame];"
Alors ça... jamais vu de ma vie...
Bref, ton void* en paramètre d'une fonction est très trompeur car il acceptera TOUS les types... du coup, le compilo sera content quoique tu donnes en paramètre, mais cela risque de faire n'importe quoi tu appellera la fonction avec des paramètres de diverses tailles (si tu envoies un char au lieu d'une structure, etc...).
Le cast de la structure est pas génial....

Tu devrais mieux typer ta fonction pour savoir exactement ce qu'elle peut accepter et faire.

Pour ton problème de 0 :
Tu crées un thread et juste après tu incrémentes ta valeur !
Si le thread a la main en 1er, il verra 0, mais s'il passe en 2e il verra 1 !
A toi d'utiliser des verrous pour bloquer l'accès à la variable.

[Nymar]

merci pour tes explications et est ce que vous pouvez me corriger la ligne que vous n'avez jamais vu ? je voulais juste déclarer un élément matrice dans ma structure.

Rem : Pour l'histoire de l’incrémentation je l'ai déplacée après la ligne sleep (PTU) et ça marche bien.

[kwariz]

[...]
Ce qui me frappe le plus c'est ton "char *(*schedule_table)[nbre_trame];"
Alors ça... jamais vu de ma vie...
[...]

schedule_table est un pointeur sur un tableau de nbre_trame pointeurs sur char, cela a été préconisé fort justement par diogène dans un post précédent.

[Bktero]

Je n'ai pas Linux donc pas de pthread. Je n'ai donc pas lancé ton code. En lisant je peux te dire :

Code :

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

1
2
#define nbre_cycle 10
#define nbre_trame 2

On met en général les noms de constantes crées avec des macros en majuscules. En lisant le code, on sait ainsi que ce sont des macros et non des variables. Personnellement, je me suis tout de suite demandé où étaient définies ces variables au premier coup d'oeil sur ton code.

Code :

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

void *func_thread (void *arg)

Ta fonction prend un pointeur sur une zone de n'importe quel type et renvoie quelque chose de similaire, comme expliqué par metalman.

Code :

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

struct signaux_thread * signal= (struct signaux_thread *)arg;

Comme un void* pointe sur une zone mémoire qui peut être n'importe quoi, tu peux caster ce pointeur vers un pointeur d'un type voulu pour interpréter la zone comme tel. Ici, tu interprètes la zone pointée comme une structure signaux_thread. Note que rien ne te garantit que c'est effectivement une telle structure qui est réellement à cet endroit et tu peux donc lire n'importe quoi.

Code :

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

fprintf(stderr,...)

Ton code ne semble pas être un cas d'erreur, pourquoi avoir choisi le flux stderr et non stdout ?

[Nymar]

je suis en train d'apprendre ce que je fais n'est pas 100 % intentionnel .

[Metalman]

Le sleep n'est pas "une bonne réponse" en soi...

Pourquoi elle semble être bonne ?
Le comportement de sleep fait que le processus rend la main "en général" et se fait réveiller après.
Du coup, la variable n'est pas modifiée par le programme à ce moment.

Pourquoi cela n'est pas la bonne solution ?
Je n'ai pas le man de sleep en tête (surtout pour la partie thread), mais ça n'est pas impossible qu'il endorme "aussi" le thread ! Et l'OS réveille celui qu'il veut en premier.
De plus, sur certains SMP tu risques d'avoir le thread ET l'applications qui tourneront simultanément....

Bref, du comportement parfaitement indéfini.

Pour la ligne bizarre, je ne sais pas trop ce que tu voulais faire, par contre je peux te dire que tu devrais déclarer ta fonction ainsi :

Code :

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

void *func_thread (struct signaux_thread * signal)

Et retirer la ligne qui déclare la variable.

[Nymar]

voila ce que je veux faire

Code :

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

char *schedule_table [nbre_cycle][nbre_trame];

et voila ce que ça donne a la compilation

Code :

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

erreur: incompatible types when assigning to type ‘char *[10][2]’ from type ‘char * (*)[2]’