Discussion :

[Vestigo]

Bonjour tous le monde, pouvez vous m'aidez à résoudre mon code svp?

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
 
#include <stdio.h>
#include <stdlib.h>
#include<string.h>
 
int main(int argc, char *argv[])
{
   char villeHabite[200];
 
   printf("Ou habite tu?");
   scanf("%s",villeHabite);
   if(villeHabite==Rennes)
   {
      printf("Ah je connais bien");
   }
   else
   {
      printf("je connais pas");
   }
 
   system("PAUSE");
   return 0;
}

[Emmanuel Delahaye]

Bonjour tous le monde, pouvez vous m'aidez à résoudre mon code svp?

Tu peux le poster sans les numéros de ligne ?

[Freed0]

Salut,

J'ai corrigé ton code, et je l'ai commenté... Pose des questions si 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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main(int argc, char *argv[])
{
    char villeHabite[200];
 
    printf("Ou habite tu ?\n");
    /* Je te conseille plutot d'utiliser fgets pour la saisie de chaine
    de caracteres */
    /* scanf("%s",villeHabite); */
    fgets (villeHabite, sizeof villeHabite, stdin);
    /* Pour fclean, fais une recherche sur le forum, on en a déjà parlé */
    fclean (villeHabite, stdin);
 
    /* La comparaison entre 2 chaines de caractères se fait à l'aide
    de strcmp (ou strncmp) */
    if (strcmp (villeHabite, "Rennes") == 0)
        printf ("Ah je connais bien.\n");
    else
        printf ("Je ne connais pas.\n");
 
    system("PAUSE");
 
    return 0;
}

Pense à regarder les prototypes des fonctions que tu ne connais pas..

[Emmanuel Delahaye]

Pense à regarder les prototypes des fonctions que tu ne connais pas..

Sauf que fclean() n'est pas standard...

[Vestigo]

Merci à tous pour vos réponses, et excusez moi pour les numéros de lignes.
Alors par contre pourquoi dois je utiliser fgets plutot que scanf? Je sais que tous le monde le dit mais qui pourait m'expliquer? Il y'a plein de mots en plus maintenant dans mon code que est ce vraiment nécessaire?

Merci.

[Vestigo]

Je viens de tester le code mais mon compilateur detecte 2 erreurs; les voici:

Code :

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

  [Linker error] undefined reference to `fclean'

et

Code :

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

  ld returned 1 exit status

Pouvez vous m'aidez?

[_SamSoft_]

Tu n'as pas du inclure le header contenant la fonction fclean. De plus, Emmanuel a dit que ce n'était pas standard

[Emmanuel Delahaye]

Merci à tous pour vos réponses, et excusez moi pour les numéros de lignes.

Tu ne les a toujours pas enlevées...

Alors par contre pourquoi dois je utiliser fgets plutot que scanf? Je sais que tous le monde le dit mais qui pourait m'expliquer?

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

Il y'a plein de mots en plus maintenant dans mon code que est ce vraiment nécessaire?

La sécurité a un cout...

[Vestigo]

OK merci, je vais voir ça... Merci pour le lien il à l'air très clair. Et que faut il que je fasse pour les entrées sécuriseé?installation?

[gl]

Et que faut il que je fasse pour les entrées sécuriseé?i

Suivre ce qui est explique dans le lien.
C'est a dire ne pas faire confiance a l'utilisateur, ne pas utiliser les fonctions ne verifiant pas la taille des entrees (gets(), scanf(), etc.), verifier la validite de la saisie avant de l'utiliser, etc.

[_SamSoft_]

En gros, utiliser fgets, puts...

[Vincent Rogier]

Décidemment, ce soir, c'est le lynchage de la fonction scanf()....

Dire que scanf ne vérifie pas la taille de l'entrée, c'est faux !!!

Scanf() le fait si le lui demande....

Celui qui utilise scanf("%s") est suicidaire et joue à la roulette russe.

Il faut toujours spécifier la taille du buffer à scanf(), eg scanf("%256s"), par exemple...

[gl]

Dire que scanf ne vérifie pas la taille de l'entrée, c'est faux !!!

Certes, il est possible de verifier la taille de l'entree avec scanf (il est vrai que dans mon message precedent, j'ai fait un raccourci rapide). Mais ce n'est pas aussi simple que peut l'etre l'utilisation de fgets() et tellement souvent mal fait que, personnellement, je preconise plutot d'oublier scanf() et de le remplacer par fgets() pour la saisie de chaine.

[Emmanuel Delahaye]

Décidemment, ce soir, c'est le lynchage de la fonction scanf()....

Comme d'habitude...

Dire que scanf ne vérifie pas la taille de l'entrée, c'est faux !!!

Scanf() le fait si le lui demande....

Celui qui utilise scanf("%s") est suicidaire et joue à la roulette russe.

Il faut toujours spécifier la taille du buffer à scanf(), eg scanf("%256s"), par exemple...

En espérant que celui-ci fasse 257 char (ou plus)... C'est difficile à gérer de façon automatique. Franchement, oublie scanf(), il y a mieux et plus simple.

[Vestigo]

Merci, mais faut il que j'installe quelques chose pour faire marcher le code que l'on m'a donné?? Parce qu'il ne marche mon compilateur détecte des erreurs.

Merci de m'aider.

[Vincent Rogier]

Je ne suis pas accro à scanf, je l'utilise même très rarement voir jamais.

bien sûr que fgets est de loin plus pratique que scanf dans les exemples cités...

mais un chat est un chat ...

[Vestigo]

Merci mais ça m'aide pas beaucoup.

[_SamSoft_]

C'est sûr que si tu dis : "ca compile pas " on peut pas t'aider. Envoie nous la sortie du compilateur et ton code !

[Emmanuel Delahaye]

Je viens de tester le code mais mon compilateur detecte 2 erreurs; les voici:

Code :

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

  [Linker error] undefined reference to `fclean'

et

Code :

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

  ld returned 1 exit status

Voici un exemple d'implémentation de fclean() :

http://www.developpez.net/forums/sho...01&postcount=2

[Emmanuel Delahaye]

C'est sûr que si tu dis : "ca compile pas " on peut pas t'aider. Envoie nous la sortie du compilateur et ton code !

Tout a été dit au-dessus, mais certains on 'parasité' son message... attention à ne pas squatter les discussions...