Discussion :

[hogar]

Bonjour,

Mon fonction ci-dessous sort normalement (pas d'erreur lors de la compilation) mais je ne peux que remplir qu'une seule valeur.

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
 
Node* createList()	               
{    
	char ch;
 
	Node *pt_head=NULL, *pt_tail=NULL, *pt_new; 
 
    do
    {
    	//Initialisation du pointeur du nouveau élément (création d'une nouvelle adresse) 
    	pt_new = (Node*)malloc(sizeof(Node));   
        if(!pt_new)  /* Si l'allocation a échoué. */
		{ 
		  printf("Memory not allocated.\n");
		  exit(EXIT_FAILURE);     
		}
 
        //Remplir le champ data du nouveau élément
    	printf("\nGive an integer number : ");
        scanf("%d", &pt_new -> val);        
	    //pt_new -> pt_next = NULL; 
 
	    if(!pt_head)
	       {
	       	  pt_head=pt_new;     
	       	  pt_tail=pt_new;
	       	  pt_tail -> pt_next = NULL; 
		   }
	    else 
	    {
	    	pt_new -> pt_next=pt_head;
			pt_head=pt_new; 	    		
		}
 
		printf("Y-Continue, N-Exit : ");
		fflush (stdin); 		 
		scanf("%c",&ch);   
	}	while (ch=='Y');  
 
	return pt_head;	
}

Merci pour ceux qui peuvent m'aider.

[Sve@r]

Bonjour

Merci pour ceux qui peuvent m'aider.

Avoir la structure Node aurait été un plus mais on peut la retrouver. En revanche, sans main() pour essayer...

Sinon à quoi sert pt_tail qui n'est utilisé que quand il est égal à pt_new ? Autant rester sur pt_new quoi. Et on ne fflushe jamais stdin car la fonction fflush() est faite pour finaliser les écritures en attentes (or stdin est un tampon de lecture donc on ne peut rien lui écrire).

Et on ne quitte pas un programme quand une fonction échoue. Quand une fonction échoue, on renvoie le souci à l'appelant qui prendra une décision (sans avoir le droit de quitter lui non plus ce qui constituera alors une chaine complète de gestion des erreurs). Et ça ne coûte pas plus cher d'écrire explicitement if (pt_new == NULL) (écrire le plus serré possible ne fait pas aller le code plus vite pour autant et question relecture...).

mais je ne peux que remplir qu'une seule valeur

Remplace while (ch=='Y') par while (ch != 'N') et quand ça marchera (*), réfléchis attentivement au contenu exact de stdin juste après le scanf("%d", &pt_new -> val). Puis tu iras voir l'auteur de fflush(stdin) lui demander de justifier son intelligente suggestion.

(*) tu pourras saisir mais en revanche tu ne pourras plus quitter

[hogar]

Merci sve@r pour ta réponse.

C'est toujours un problème lié à scanf() dans une boucle en particulier avec les chaine de caractères. D'ailleurs que je n'ai jamais pu résoudre ni trouver une solution radicale.

[hogar]

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
 
typedef struct Node
	   {
		int val;              //Data
		struct Node* pt_next; //pointeur (lien) vers l'élement suivant de même type (pointeur sur un objet de type Node)
	   } Node;
 
 
 
int main()
{
	const int n=5;	
 
	Node *head=createList();
 
         traverseList(head);
 
	freeMemory(head);
 
	return 0;
}

[Sve@r]

C'est toujours un problème lié à scanf() dans une boucle en particulier avec les chaine de caractères.

Plus précisément une saisie de nombre (int/float) suivie par une saisie de caractère.

D'ailleurs que je n'ai jamais pu résoudre ni trouver une solution radicale.

C'est pour ça que je t'ai dit de réfléchir sur le contenu de stdin. Quand tu tapes ta valeur, style 12, tu le valides par <return>. Le scanf() récupère le 12 et laisse le <return> dans stdin. Ensuite à la saisie suivante...
Solution: char chaine[100]; fgets(chaine, 100, stdin) pour faire saisir la valeur dans une chaine, ainsi stdin est propre. Et ensuite sscanf(chaine, "%d", &val) pour récupérer la valeur depuis la chaine saisie. Et tu peux même vérifier via le retour de sscanf() (nombre d'éléments récupérés) qu'il a bien récupéré un élément (la valeur).

[CGi]

Attention flush(stdin) à un comportement indéterminé UB. (Ça peut fonctionner comme pas).
Une solution rapide pour manger le '\n', c'est de mettre un espace avant le %c de la saisie de caractère.

Code :

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

scanf(" %c", &ch);

[hogar]

Bonjour,

Merci Sve@r j'ai remplacé scanf() par fgets et sscanf() et ça marche.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
char chaine[100];
.......................................................
printf("\nGive an integer number : ");
fgets(chaine, 100, stdin);	    
sscanf(chaine, "%d", &pt_new -> val);
 
......................................................
printf("Y-Continue, N-Exit : ");
fgets(chaine, 100, stdin);   //ou bien
sscanf(chaine,"%c",&ch);
...................................................

Merci.

[Sve@r]

Merci Sve@r j'ai remplacé scanf() par fgets et sscanf() et ça marche.

Faut faire plus que remplacer, faut aussi comprendre ce qui se passe. Ainsi ça permet de rajouter des contrôles...

Code c :

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

1
2
3
4
5
6
while (1) {
	printf("\nGive an integer number : ");
	fgets(chaine, 100, stdin);
	if (sscanf(chaine, "%d", &pt_new -> val) == 1) break;
	printf("Mauvaise saisie, recommencez !!!\n");
}

[hogar]

Merci Sve@r.

[Xelland]

et faire attention, si tu rentres "55toto"
pt_new -> val va contenir 55

sscanf(chaine, "%d" => va retourner 1 si chaine commence par un entier

si tu veux que chaine soit un entier, il faut ajouter d'autres contrôles