Discussion :

[vitia]

apres l`ecriture de ce mini programme

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
# include<stdio.h>
 
void main()
{
 
char word;
	printf("enter your pasword :\n");
		scanf("%c",&word);
 
		printf("le mot qui tu as ecrit \'%c\' ne corespand pas avec l`original\n");
 
		getch();
 
 
}

et apres il aparu

error C2065: 'getch' : undeclared identifier

.alors quoi faire ??

[Franck.H]

getch n'est pas standard, le mieux est d'utiliser éventuellement getchar.

[Sve@r]

apres l`ecriture de ce mini programme

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
# include<stdio.h>
 
void main()
{
 
char word;
	printf("enter your pasword :\n");
		scanf("%c",&word);
 
		printf("le mot qui tu as ecrit \'%c\' ne corespand pas avec l`original\n");
 
		getch();
 
 
}

et apres il aparu .alors quoi faire ??

Le symbole "%c" indique une saisie d'un caractère. C'est peu pour un mot...

Par ailleurs, si tu mets "%c" dans printf, après faut que t'indiques un élément e type "char" que tu veux afficher à l'endroit du "%c"...

[_SamSoft_]

Je croyais que c'était getch le standard ? Je me suis donc trompé

Code :

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

1
2
 
void main();

lol c'est une caméra cachée n'est ce pas ?

Bon une petite correction s'impose :

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
 
#include <stdlib.h>
#include <stdio.h>
 
int main(void)
{
   char word[256];
 
   printf("Enter your passowrd : ");
   fgets(word, sizeof word, stdin);
   /* normalement on fait fclean(word, stdin); ici */
   /* Verification du mot de passe */
   getchar();
   return 0;
}

Voilà un code presque correct (manque fclean) mais qui marche au poil

[Thierry Chappuis]

Effectivement, c'est plus propre de nettoyer le tampon du flux d'entrée standard après usage de fgets (voir la fonction fclean() dans les archives du forum). Il convient aussi de purger le tampon du flux de sortie standard pour forcer l'affichage de l'invite:

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
#include <stdio.h>
#include <string.h>
 
#define PASSWORD_MAXLEN 32
 
void fclean(char *buff, FILE * fp);
 
int main(void)
{
    char const *my_secret_pw = "developpez";
    char word[PASSWORD_MAXLEN + 1] = "";
 
    printf("Enter your password : ");
    /* -tc- force l'affichage de l'invite */
    fflush(stdout);
    if (fgets(word, sizeof word, stdin) != NULL)
    {
        fclean(word, stdin);
 
        if (strcmp(word, my_secret_pw) == 0)
        {
            printf("Password OK\n");
        }
        else
        {
            printf("Password KO\n");
        }
    }
    /* -tc- l'appel a getchar() n'est pas necessaire */
 
    return 0;
}

Thierry

[Leole]

et si tu veux vraiment utiliser getch pour une quelquonque raison et que tu comptes rester sous windows , inclus <conio.h>

[Emmanuel Delahaye]

et si tu veux vraiment utiliser getch pour une quelquonque raison et que tu comptes rester sous windows , inclus <conio.h>

ou <conio2.h>. Mais il faut aussi lier la bibliothèque qui va bien...

Ceci-dit, je conseille d'abandonner conio au profit de PDCurses qui est beaucoup plus portable...

[Sve@r]

Effectivement, c'est plus propre de nettoyer le tampon du flux d'entrée standard après usage de fgets (voir la fonction fclean() dans les archives du forum).

Moi suis pas d'accord. J'ai eu une discussion assez sérieuse à ce propos sur forum hardware et mon interlocuteur Taz (qui tate sa bille en C) m'a démontré que 1) c'était inutile et 2) ça empêchait un programme de traiter un flux passé par pipe.

En effet, imaginons qu'on tape "ls | pgm" le buffer d'entrée stdin est chargé avec tout le contenu de "ls".
Ensuite dans mon pgm je traite par un "fgets()" mon entrée (le nom du premier fichier qui est effacé du buffer) => il me reste ensuite les autres fichiers à traiter. Mais si juste après mon fgets() je fais un fclean() ben je perds le reste de mes noms de fichiers.
Conclusion: Si on lance un pgm le buffer stdin est vide. Ensuite s'il est rempli c'est pour une bonne raison donc faut traiter ce qu'il contient et non le bazarder...

Voici un copier/coller de son message

si c'est la donnée. tu as tapé le '\n', encore heureux que tu le récupères. tu n'en veux pas ? et bien saute le. maintenant si tu fous en l'air systématiquement les données de l'utilisateur, tu t'exposes à de graves problème.

on va prendre un exemple simple parmi tant d'autres d'un programme à rendre après un TP, le professeur spécifie l'emploi du programme : c'est très courant. Maintenant pas con, ton prof il remplis un fichier avec le jeu de données, et il appelle les programmes à corriger avec. Va lui expliquer que ton programme il marche pas ... il te répondra qu'il en a rin à secouer, qu'il donne les données et que toi tu les balardes, que ce n'est pas son problème si tu ne sais pas analyser des données, et qu'il te mets donc une note conséquente.

[Gruik]

Euh... que ce soit par pipe ou par saisie directe, ça reste l'entrée standard...
Ya un probleme en effet mais c'est pas lié à fclean, c'est lié au fait qu'il faille une taille max à fgets et que si la ligne lue est trop grande bein on perd la fin de la ligne avec fclean.
Mais de toute manière l'algo actuel ne gere pas le reste de la ligne, alors autant faire fclean et ne pas géner la lecture suivante sur l'entrée standard.

Ainsi donc faut se poser la question de si on est prêt à imposer une longueur max pour les lignes lues sur l'entrée standard ou non. Et si non, faudra surement tater du realloc par exemple.

[Sve@r]

Euh... que ce soit par pipe ou par saisie directe, ça reste l'entrée standard....

Oui, mais quand on se sert d'un pipe on remplit l'entrée standard d'un très grand nombre de lignes qu'il faut ensuite traiter "comme il faut" et sans en perdre. A mon avis, un programme qui supporte qu'on l'appelle par un pipe supportera aisément la frappe clavier alors qu'un soft qui gère la frappe clavier via fclean() parce que le programmeur n'a pas compris ce que contient son buffer (ou n'a pas envie de se fatiguer à mieux le gérer) risque de mal tenir la route sur un pipe...

Et si non, faudra surement tater du realloc par exemple.

Il y a la fonction getline() qui cumule le fgets() et le realloc() pour renvoyer une zone contenant une ligne entière quelle que soit sa taille
http://manpagesfr.free.fr/man/man3/getline.3.html

[Gruik]

Oui, mais quand on se sert d'un pipe on remplit l'entrée standard d'un très grand nombre de lignes qu'il faut ensuite traiter "comme il faut" et sans en perdre. A mon avis, un programme qui supporte qu'on l'appelle par un pipe supportera aisément la frappe clavier alors qu'un soft qui gère la frappe clavier via fclean() parce que le programmeur n'a pas compris ce que contient son buffer (ou n'a pas envie de se fatiguer à mieux le gérer) risque de mal tenir la route sur un pipe...

Bein je me repete mais, pipe ou clavier et c'est la meme chose.

Il y a la fonction getline() qui cumule le fgets() et le realloc() pour renvoyer une zone contenant une ligne entière quelle que soit sa taille
http://manpagesfr.free.fr/man/man3/getline.3.html

Connaissais pas.. de toute façon c'est une extension gnu donc pas du tout standard

[gl]

Oui, mais quand on se sert d'un pipe on remplit l'entrée standard d'un très grand nombre de lignes qu'il faut ensuite traiter "comme il faut" et sans en perdre. A mon avis, un programme qui supporte qu'on l'appelle par un pipe supportera aisément la frappe clavier alors qu'un soft qui gère la frappe clavier via fclean() parce que le programmeur n'a pas compris ce que contient son buffer (ou n'a pas envie de se fatiguer à mieux le gérer) risque de mal tenir la route sur un pipe...

Non, un pipe ou une saisie clavier c'est exactement la meme chose et c'est transaprent pour le progrmme.
Au vue de tes explications et objections, je pense surtout que tu n'as pas du tout compris ce dont parlait les differents intervenant en parlant de nettoyer le flux d'entree : il n'a jamais ete question de purger l'intergralite du flux a venir, mais de gerer correctement la ligne en cours afin de ne pas perturber les entrees ulterieures, ce qui me semble etre la moindre des choses.

Pretendre qu'il ne faut pas nettoyer le flux suite a une saisie est une grave erreur.

[Sve@r]

Au vue de tes explications et objections, je pense surtout que tu n'as pas du tout compris ce dont parlait les differents intervenant en parlant de nettoyer le flux d'entree

Probable (je connais pas fclean())...

il n'a jamais ete question de purger l'intergralite du flux a venir, mais de gerer correctement la ligne en cours afin de ne pas perturber les entrees ulterieures, ce qui me semble etre la moindre des choses.

Exact. Faut gérer correctement la ligne en cours (dans l'hypothèse que la ligne ne soit pas entièrement traitée). Si fclean() le fait alors pas de pb. Sinon on peut aussi faire du fgetc() à répétition jusqu'au "\n" suivant ou alors aller voir du coté de getline(). Là je crois qu'on est d'accord...

[Emmanuel Delahaye]

Probable (je connais pas fclean())...

C'est une fonction, disons 'communautaire' qui circule sur ce forum C... Je n'y suis pas complètement étranger...

[Sve@r]

Je n'y suis pas complètement étranger...

Arf ça m'aurait étonné

Je présume alors que ce doit être un truc qui traite et élimine les octets jusqu'au '\n' suivant

[Emmanuel Delahaye]

Arf ça m'aurait étonné

Je présume alors que ce doit être un truc qui traite et élimine les octets jusqu'au '\n' suivant

Entre autres...

[Thierry Chappuis]

Arf ça m'aurait étonné

Je présume alors que ce doit être un truc qui traite et élimine les octets jusqu'au '\n' 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
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
/**
 * Elimine le caractere de fin de ligne saisi par fgets() et nettoie le tampon 
 * du flux entrant si la saisie a ete tronquee. Si NULL est passe en 1er 
 * argument, le flux entrant est purge.
 *
 * @return code d'erreur valant 1 si la ligne saisie par fgets() a ete tronquee,
 *         et 0 sinon
 */
static int fclean(char *buffer, FILE * fp)
{
    int err = 0;
 
    if (fp != NULL)
    {
        char *pc = NULL;
 
        if (buffer != NULL)
        {
            pc = strchr(buffer, '\n');
        }
 
        if (pc != NULL)
        {
            /* La presence du caractere de fin de ligne indique que la
               saisie n'a pas ete tronquee */
            *pc = 0;
        }
        else
        {
            /* buffer est NULL OU le caractere de fin de ligne
               n'a pas ete trouve (saisie tronquee): on purge le
               tampon du flux entrant */
            int c;
 
            while ((c = fgetc(fp)) != '\n' && c != EOF)
            {
            }
 
            if (buffer != NULL)
            {
                err = 1;
            }
        }
    }
 
    return err;
}

Thierry