Discussion :

[Nyx de Tours]

Bonjour,

Je suis débutant en C. Je dois demander une saisie de chaine de caractères pour la mettre dans une variable. Si l'utilisateur choisit de ne rien mettre et donc de taper directement sur Entrée, je mets une valeur par défaut dans la variable, sinon je récupère la valeur saisie par l'utilisateur.

Mon problème est que je ne connais que la fonction scanf pour la saisie au clavier, et qu'il n'accepte pas que l'on ne saisisse rien (appui direct sur la touche Entrée).

Pourriez-vous me proposer une solution (l'idéal serait de ne pas utiliser les pointeurs )

Merci d'avance

[PRomu@ld]

fgets() tout simplement ...

[Pragmateek]

De toute façon scanf est déconseillé:
http://emmanuel-delahaye.developpez....tes.htm#saisie

[Nyx de Tours]

Merci Je teste ce soir et je reviens donner des nouvelles

De toute façon scanf est déconseillé:
http://emmanuel-delahaye.developpez.com/notes.htm#saisie

J'en ai entendu parler mais c'est une fonction de saisie souvent enseigné en premier au débutant - comme moi .
De plus, certains défendent l'utilisation du scanf
http://xrenault.developpez.com/tutoriels/c/scanf/

[Nyx de Tours]

Merci encore pour vos réponses. J'utlise donc fgets.

Je souhaite maintenant créer une fonction lire() avec ce fgets, mais ce que j'ai fais contient des bétises (surement énormes).

Est-ce que vous pourriez m'aider à corriger (voir compléter) cette fonction ?

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
#include <stdio.h>
 
char lire();
 
main()
{
	char saisie[256];
 
	printf("Saisir \"toto\" : ");
	fflush(stdout);
	strcpy(saisie, lire());
 
	if(!strcmp(saisie, "toto"))
	{
		printf("Texte identique\n");
	}
	else
	{
		printf("Texte different\n");
	}
	printf("Texte saisi : \"%s\"\n", saisie);
}	
 
 
char lire()
{
	char input[256];
 
	fgets (input, sizeof input, stdin);
	/* On recherche le '\n' */
	{
		char *pt = strchr (input, '\n');
		/* S'il y en a un, on le remplace par la valeur "fin de chaîne" */
		if (pt != NULL)
		{
			*pt = 0;
		}
		else
		{
			/* sinon, on purge... */
			int c;
		    while ((c = fgetc (stdin)) != '\n' && c != EOF)
		    {
		    }
		}
	}
}

[hegros]

Par contre bizarrement, ta fonction lire() renvoi un char ? Il est ou le return ?

[PRomu@ld]

Alors comme ça dans le desordre :

main()
{

Même si de vielles normes ne spécifient pas de valeur de retour pour la fonction main, il faut néanmoins (si tu veux suivre les normes actuelles), renvoyer un int. Du coup, à la fin de ta fonction, il faut que tu renvoie une valeur, soit EXIT_SUCCESS, soit EXIT_FAILURE (à la fin de main, c'est toujours success), ces deux constantes sont définies dans l'entête stdlib.h

Ta fonction de saisie (ie : lire) renvoie un char, c'est à dire un caractère or ce que tu va saisir, c'est une chaine de cractère, c'est donc un char * qu'il faut renvoyer.

De plus, dans ta fonction, je ne vois pas d'instruction return, comment vu tu que ta fonction renvoie quelque chose ?

Enfin à quoi celà sert de purger ta chaine ?

[Trap D]

Il faudrait apprendre à régler le compilateur, ce code est bourré d'erreurs.

La fonction lire doit revoyer une adresse valide, donc pas question qu'elle renvoie input; qui est l'adresse d'un tableau local à la fonction lire.

[Emmanuel Delahaye]

Je souhaite maintenant créer une fonction lire() avec ce fgets, mais ce que j'ai fais contient des bétises (surement énormes).

Heureux de voir que mes petits articles ne sont pas inutiles...

• Il manque <string.h> (strcpy() etc.)
• Quand une fonction n'a pas de paramètres, on le précise avec void.
• Le type retourné par main() est int et il doit être explicite en C99. Il faut un return explicite en C90.
• strcpy() attend l'adresse du premier élément d'un tableau de char. Tu lui donne la valeur d'un char. C'est incohérent.
• Il n'y a pas de return dans la fonction lire. Elle ne sait pas quoi retourner... En tout cas, elle ne peut retourner l'adresse du premier élément de input, car c'est une variable locale, donc invalide après exécution. Je recommande de passer l'adresse et la longueur du bloc mémoire à lire(). Celui-ci est crée par l'appelant.
• Une fonction non exportée peut être privée (static) et définie avant d'être utilisée (suppression du prototype séparé). Ca facilite la maintenance.

Code corrigé :

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
 
#include <stdio.h>
#include <string.h>
 
static lire(char *input, size_t size)
{
 
   fgets (input, size, stdin);
   /* On recherche le '\n' */
   {
      char *pt = strchr (input, '\n');
      /* S'il y en a un, on le remplace par la valeur "fin de chaîne" */
      if (pt != NULL)
      {
         *pt = 0;
      }
      else
      {
         /* sinon, on purge... */
         int c;
         while ((c = fgetc (stdin)) != '\n' && c != EOF)
         {
         }
      }
   }
}
 
int main(void)
{
   char saisie[256];
 
   printf("Saisir \"toto\" : ");
   fflush(stdout);
 
   lire(saisie, sizeof saisie);
 
   if (!strcmp(saisie, "toto"))
   {
      printf("Texte identique\n");
   }
   else
   {
      printf("Texte different\n");
   }
   printf("Texte saisi : \"%s\"\n", saisie);
 
   return 0;
}

[hegros]

En fait tu ne pourrais pas faire dans ta fonction lire

Code :

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

1
2
3
4
char * lire() { 
char input[256];
return  fgets(input,sizeof (input) ,stdin);
}

[Emmanuel Delahaye]

Enfin à quoi celà sert de purger ta chaine ?

Ahem...

http://emmanuel-delahaye.developpez....tes.htm#saisie
http://emmanuel-delahaye.developpez....s.htm#fichiers

[PRomu@ld]

Ce n'est pas le concept que je critiquait (loin de là, je fais ceci pour des entrée valides), neanmoins, je voulais rester dans le cadre du programme, dans la mesure où il s'agit d'un test pour comparer deux chaines, la purge me semblait un petit plus peut être inutile.

[Emmanuel Delahaye]

Ce n'est pas le concept que je critiquait (loin de là, je fais ceci pour des entrée valides), neanmoins, je voulais rester dans le cadre du programme, dans la mesure où il s'agit d'un test pour comparer deux chaines, la purge me semblait un petit plus peut être inutile.

Ecrire du code solide et stable n'est jamais inutile...

Je vois bien qu'il a fait un copié collé d'un code que je connais bien, mais ça ne peut pas nuire, notamment, ça résiste au teste du singe,



surtout si il boucle l'affaire... ou si il met un satané getchar() en pause à la fin du code pour ce bloody Dev-C++...

[PRomu@ld]

Ecrire du code solide et stable n'est jamais inutile...

On est d'accord.

Je vois bien qu'il a fait un copié collé d'un code que je connais bien

Ah bon ?

mais ça ne peut pas nuire, notamment, ça résiste au teste du singe,

Tu n'oserais pas comparer tes clients à des singes ?

surtout si il boucle l'affaire...

Alors là OK

ou si il met un satané getchar() en pause à la fin du code pour ce bloody Dev-C++

Oui mais là on rentre dans d'autres considérations, il faut changer d'IDE ...

[Emmanuel Delahaye]

mais ça ne peut pas nuire, notamment, ça résiste au teste du singe,

Tu n'oserais pas comparer tes clients à des singes ?

[Nyx de Tours]

Merci à tous pour votre aide.

Il faudrait apprendre à régler le compilateur, ce code est bourré d'erreurs

J'ai même beaucoup plus à apprendre

Heureux de voir que mes petits articles ne sont pas inutiles...

Bien sur qu'ils ne le sont pas

Je vois bien qu'il a fait un copié collé d'un code que je connais bien

En effet

surtout si il boucle l'affaire...

oui, en fait la fonction sera dans un programme un peu plus important. Il s'agit d'un petit projet scolaire.




Vous serait il possible de m'expliquer comment ceci est fait dans la fonction lire() revue et corrigée par Emmanuel Delahaye et situé dans un messsage ci dessus :

Je recommande de passer l'adresse et la longueur du bloc mémoire à lire()

[Emmanuel Delahaye]

Vous serait il possible de m'expliquer comment ceci est fait dans la fonction lire() revue et corrigée par Emmanuel Delahaye et situé dans un messsage ci dessus :

Je recommande de passer l'adresse et la longueur du bloc mémoire à lire()

Voici ce qui a été fait : L'adresse du premier élément du tableau 'saisie' est passée dans 'input' et sa taille dans 'size'. Qu'est-ce que tu ne comprends pas ?

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
 
static lire(char *input, size_t size)
{
 
   fgets (input, size, stdin);
   /* On recherche le '\n' */
   {
      char *pt = strchr (input, '\n');
      /* S'il y en a un, on le remplace par la valeur "fin de chaîne" */
      if (pt != NULL)
      {
         *pt = 0;
      }
      else
      {
         /* sinon, on purge... */
         int c;
         while ((c = fgetc (stdin)) != '\n' && c != EOF)
         {
         }
      }
   }
}
 
int main(void)
{
   char saisie[256];
   <...>
   lire(saisie, sizeof saisie);

[Nyx de Tours]

Désolé je n'ai pas posé la bonne question... c'est ça de faire plusieurs choses à la fois... oublions la question précédente.

En fait ce que je ne comprend pas c'est comment la valeur saisie grâce au fgets, et stockée dans input, se retrouve dans la variable saisie...

...en tapant je suis en train de comprendre. Est-ce bien ça :
Comme précisé c'est l'adresse de input qui est passée dans la fonction lire().
Quand on déclare la fonction, le premier paramètre *input fait référence à cette même adresse ? et ainsi quand on saisit une chaine avec le fgets, elle est stockée à cette même adresse ? Serais-je en train de comprendre les pointeurs ?

Si c'est bien ça il y a autre chose qui n'est pas clair pour moi : parfois on met une étoile '*' et parfois il n'y en a pas. Pourtant à chaque fois on travaille sur un élément situé à une adresse mémoire, non ? comment interpréter ou expliquer la présence ou non de l'astérisque ?

[hegros]

En fait il existe à ma connaissance deux façons de passer des paramètres à une fonction soit par valeur soit par adresse.

Quand tu passes un paramètre par valeur,donc sans *, tu travailles sur une copie.En fait l'appel à une fonction améne à créer une copie de la variable passée en paramètre sur ce qu'on nomme la pile.Donc pendant toute l'execution de la fonction tu travailles sur une copie ( celle qui est sur la pile).

Par contre lorsque tu précises que tu passes en paramètre une adresse à ta fonction alors tu ne travailles plus sur une copie mais sur la variable elle-même.

Il y a un post où promuld je crois explique trés bien ce mécanisme cela doit être dans un sujet récent sur la première page...

Je sais pas si c'est clair mais n'hésite pas à poser des questions c'est important de bien comprendre cette notion.

[DaZumba]

En fait il existe à ma connaissance deux façons de passer des paramètres à une fonction soit par valeur soit par adresse.

En C, tous les passages de parametres se font par valeur. Pour modifier une variable à l'intérieur d'une fonction, il faut passer la valeur de l'adresse de cette variable (i.e. la valeur d'un pointeur vers cette variable).