Discussion :

[seb__]

Bonjour,
dans un programme j'utilise des threads, aucunne erreur de compilation ou quoi que ce soit, mais j'ai le problème suivant:

mon processus principal lit un fichier ligne par ligne, une fois la ligne obtenue, un thread va la traiter. Seulement, la procédure associée au thread reçoit apparemment la dernière ligne du fichier à chaque fois...

en effet, pour un fichier de 3 lignes par exemple, si je teste la valeur de chaque ligne avant de passer la ligne en arguments, les 3 lignes ont bien été lues.

Dans la procédure de l'argument si je demande qu'il m'imprime successivement toutes les lignes qu'elle reçoit, elle m'affichera 3 fois la ligne numéro 3.

Mon code est le suivant, rien d'exeptionnel..

au niveau de l'envoi de la ligne lue

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
 
        <...>      
 
	while(fgets(line,256,f)!= NULL)
	{
		/* traitement de la ligne, lancement du premier groupe de threads (un seul pour l'instant) */
		printf("ligne envoyée 1 = %s", line);
		pthread_create( &tid[cpt], NULL, lecture_aux, (void*)line);	
		cpt++;
	}
 
 
         <...>

au niveau de la réception de la ligne:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
void* lecture_aux(void * line){
	char* path = (char*) line; /* recuperation de l'argument */
	FILE *f;
	int i,size;
	char *fileName; /* cope de path sans '\n' ni buffer vide après */
 
	printf("ligne recue = : %s\n",path);
 
       <...>

mes threads se terminent bien, le problème ne devrait pas venir de là.
Les threads ayant parcontre en commun le même code que le processus principal, le problème ne viendrait-il pas de là?

J'espere avoir été clair dans mes explications.

D'avance je vous remercie

[zooro]

Salut,

Le problème ne viendrait-il pas du fait que la variable line est partagée par les threads ? Et comme ta boucle while continue à lire les lignes suivantes (stockées à l'adresse que tu as passée à ton thread)...

Peut-être faudrait-il passer à ton thread une copie de la ligne.

[seb__]

enfait, à mon sens, le pointeur étant une zone de mémoire allouée, elle est la même pour chaque thread, donc en effet si j'appelle une procédure x avec un pointeur pointant vers une zone mémoire, et que dans un autre thread, je modifie le contenu de cette zone mémoire, tous les threads se référant à cette même zone mémoire, ils auront tous accès à la même valeur. Zut, ça fonctionne pas comme en java =/

mais alors, comment pourrais-je remédier à ce problème? je peux quand même pas créer autant de pointeurs que de fois où je créerai un thread? c'est pas très propre

[EDIT]
enfait, il faudrait que je puiss envoyer le contenu du pointeur et non le pointeur lui même... mais avec ce cast en void, pas moyen.. j'obtiens des erreurs de segmentation

[zooro]

mais alors, comment pourrais-je remédier à ce problème? je peux quand même pas créer autant de pointeurs que de fois où je créerai un thread? c'est pas très propre

[EDIT]
enfait, il faudrait que je puiss envoyer le contenu du pointeur et non le pointeur lui même... mais avec ce cast en void, pas moyen.. j'obtiens des erreurs de segmentation

Passe au thread un pointeur sur une copie de la chaîne.
Ou alors revois la conception de ton programme.

[Mat.M]

je pense que l'intégralité de la lecture du fichier devrait être effectuée dans un thread pas une partie !
Ce n'est pas intégre :si le thread se bloque ou quoique ce soit des handle de FILE vont être ouverts inutilement et incorrectement fermés

Bonjour,
dans un programme j'utilise des threads, aucunne erreur de compilation ou quoi que ce soit, mais j'ai le problème suivant:

Il faudrait mettre cela en gros dans une FAQ : ce qui peut passer et valide en conception et en "grammaire" du langage peut bloquer en exécution !
Tu peux très bien déclarer int *p; *p=10; et cela plante

[seb__]

c'est à dire que la lecture se fait dans le même thread, les threads se chargent juste du traitement de la ligne.

Pour l'erreur de compilation, je voulais dire que mon erreur ne venait pas d'une compilation

ok je vais tenter de changer l'architecture du programme.. cela dit si quelqu'un a une idée pour passer le contenu d'un pointeur au thread..