Discussion :

[Flash_Over]

bonjour
mon probleme est le suivant ,je dois creer une communication entre un processus pere et k processus fils le pere doit communiquer avec chaque fils par un tube séparé mon code est le suivant

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
 
int tab[k][2] ;
for(i=0 ;i<k ;i++){ 
                pipe(tab[i]);
		pid=fork();
	  		if (pid==0){
                        close(tab[i][1]);
                        for(j=0;j<k;j++){
				if (j!=i){
				close(tab[j][0]);
				}
			}
}
else {
			close(tab[i][0]);
 
		       }}
char comp[256]; 
         int w=5;              
	sprintf(comp,"%d",w);
	write(tab[2][1],comp,strlen(comp)+1);

j ai pris au pif le tube 2 pour ecricre la valeur 5 qaund j essais de faire marcher la programme il tourne dans le vide meme si la compilation s est bien passé

alors que quand j evite de fermer tab[i][0] au niveau du pere ca marche

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
 
int tab[k][2] ;
for(i=0 ;i<k ;i++){ 
                pipe(tab[i]);
		pid=fork();
	  		if (pid==0){
                        close(tab[i][1]);
                        for(j=0;j<k;j++){
				if (j!=i){
				close(tab[j][0]);
				}
			}
}
else {
			//close(tab[i][0]);
 
		       }}
char comp[256]; 
         int w=5;              
	sprintf(comp,"%d",w);
	write(tab[2][1],comp,strlen(comp)+1);

le probleme c est que je crois que les pipes ne sont pas bien branché avec les fils
et que quand ca plante c est que pq le au niveau de l ecriture dans le tube il ne voit que la sortie du pere sans voir celles des fils
Merci

[Sve@r]

bonjour
mon probleme est le suivant ,je dois creer une communication entre un processus pere et k processus fils le pere doit communiquer avec chaque fils par un tube séparé mon code est le suivant

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
 
int tab[k][2] ;
for(i=0 ;i<k ;i++){ 
                pipe(tab[i]);
		pid=fork();
	  		if (pid==0){
                        close(tab[i][1]);
                        for(j=0;j<k;j++){
				if (j!=i){
				close(tab[j][0]);
				}
			}
}
else {
			close(tab[i][0]);
 
		       }}
char comp[256]; 
         int w=5;              
	sprintf(comp,"%d",w);
	write(tab[2][1],comp,strlen(comp)+1);

j ai pris au pif le tube 2 pour ecricre la valeur 5 qaund j essais de faire marcher la programme il tourne dans le vide meme si la compilation s est bien passé

alors que quand j evite de fermer tab[i][0] au niveau du pere ca marche

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
 
int tab[k][2] ;
for(i=0 ;i<k ;i++){ 
                pipe(tab[i]);
		pid=fork();
	  		if (pid==0){
                        close(tab[i][1]);
                        for(j=0;j<k;j++){
				if (j!=i){
				close(tab[j][0]);
				}
			}
}
else {
			//close(tab[i][0]);
 
		       }}
char comp[256]; 
         int w=5;              
	sprintf(comp,"%d",w);
	write(tab[2][1],comp,strlen(comp)+1);

le probleme c est que je crois que les pipes ne sont pas bien branché avec les fils
et que quand ca plante c est que pq le au niveau de l ecriture dans le tube il ne voit que la sortie du pere sans voir celles des fils
Merci

Le problème de tes boucles imbriquées (je crée un pipe et en même temps je crée un fils) c'est que le fils lui, va fermer tous les pipe (même ceux qui n'ont pas encore été créés avec le "for (j=0; j < k; j++)").
Donc tu passes à close() des variables non initialisées => comportement indéterminé.

Faut faire ça en 2 étapes
1) je crée tous les pipe
2) je crée tous les fils

Puis chaque fils ferme tous les cotés "lecteur" et tous les cotés écrivains (sauf le sien) tandis que le père ferme le coté écrivain du fils créé et lit le fils

PS: si tu pouvais te forcer à donner des noms significatifs à tes variables ("tab" ???) ben ce serait pas mal. Et si, au lieu d'utiliser un tableau 2D tu utilisais un tableau de structures où chaque structure va gérer un pipe, ça gagnerait encore plus en clarté. Surtout que tu peux y ajouter d'autres trucs à l'intérieur comme par exemple le pid du fils (ce qui pourra être utile plus tard)...

Bon, un truc tapé vite fait mais qui devrait fonctionner

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
typedef struct {
    int tube[2];
    int pid;
} t_fils;
 
t_fils tab[k];
 
for (i=0; i < k; i++)
    pipe(tab[i].tube);
 
for (i=0; i < k; i++)
{
    tab[i].pid=fork();
    if (tab[i].pid == 0)
    {
         // Fils
         // Fermeture de tous les lecteurs et tous les écrivains sauf lui
         for (j=0; j < k; j++)
         {
             close(tab[j].tube[0]);
             if (j != i)
                 close(tab[j].tube[1]);
         }
 
         // Ecriture
         sprintf(comp, "%d", 5);
         write(tab[i].tube[1], comp, strlen(comp + 1));
    }
    else
    {
         // Pere
         // Fermeture de l'écrivain
         close(tab[i].tube[1]);
 
         // Lecture et affichage
         read(tab[i].tube[0], comp, sizeof comp);
         printf("Fils %d (pid=%d) - Lu %s\n", i + 1, tab[i].pid, comp);
    }
}