thread(erreur de lecture d'un caractère)

[khadi8]

Salut !
J'essaye d'écrire un programme avec deux thread le 1er permet la lecture d'un caractère et le 2eme l'afficher, j'ai une erreur avec le 1er thread qui permet la lecture voici mon code, je précise que je suis sur Linux

Code :

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<stdlib.h>
#include<stdio.h>
#include<pthread.h>
#include<unistd.h>
 
char c;
 
//fonction pour lire des caractère 
void *lecture(void *k)
{
printf("donner un caractère\n");
scanf("%s",&c);
while(c!='f' || c!='F')
{
printf("donner un caractère\n");
scanf("%s",&c);//l'erreur est ici 
printf("\n");
}
 
}
void *ecriture(void *k)
{
while(c!='f' || c!='F')
{
printf("le caractere est :%c",c);
}
}
 
int main(void)
{
pthread_t pth[2];
 
if(pthread_create(pth,0,lecture,NULL))
{
perror("pthread_create");
exit(EXIT_FAILURE);
}
 
/*if(pthread_create(pth+1,0,ecriture,NULL))
{
perror("pthread_create");
exit(EXIT_FAILURE);
 
}*/
 
if(pthread_join(pth,NULL))
perror("erreur");
 
/*if(pthread_join(pth+1,NULL))
perror("erreur");*/
}

voici l'erreur

Code :

erreur:Success

[Bktero]

http://www.linux-kheops.com/doc/man/...ad_join.3.html

Voir la section "valeur renvoyée". Il te donne une liste des codes. Regarde la valeur de ce code, que tu n'affiches pas avec le code actuel.

[Médinoc]

Pour commencer, utilise scanf("%c"), pas %s.

[Red Sno]

Tu fais erreur dans le traitement des erreurs

Code :

if(pthread_create(pth,0,lecture,NULL))

en cas d'echec pthread_create renvoi -1, et c'est la ou tu dois exécuter perror, de meme pour pthread_join

ce que je te conseil c'est d'enlever le traitement des erreurs vu que tu utilises de simple fonction bien définie

Une autre remarque, il faut synchroniser la fonction lecteur et la fonction rédacteur, il y a plusieurs manière de le faire, des sémaphores, ou de simple variables booléenne ce qui serait l’idéal
si tu ne les synchronise pas, la fonction afficher s’exécute plusieurs fois en affichant le même caractère

Pour terminer, il faut séparer la variable ou tu récupères le caractère, en utilisant deux variable, une pour récuperer le caractère, une autre pour faire le test (!='F' || ='f') et d'envoyer première variable à la fonction afficher

Bon courage

[Bktero]

ce que je te conseil c'est d'enlever le traitement des erreurs vu que tu utilises de simple fonction bien définie

Euh.... Pourquoi ?

[LittleWhite]

Bonjour,

Envoyé par Red Sno

ce que je te conseil c'est d'enlever le traitement des erreurs vu que tu utilises de simple fonction bien définie

Mais si bien définie et sûre, il faut vérifier les erreurs, car il arrive toujours que l'utilisateur ne sache pas les utiliser correctement, ou encore, qu'il insère de mauvais paramètres et que cela provoque crash ou autre.
Ce n'est pas parce que l'on vous donne des outils surs, que vous n'allez pas essayer de me planter un clou avec une perceuse (c'est une image bien sur).