Tom's Guide > Forum > Programmation > Langage C

Langage C

Forum Programmation : Langage C

TomsGuide.com : 800 000 inscrits répondent à toutes vos questions high-tech et informatique. Pour obtenir de l'aide, inscrivez-vous gratuitement !
Créer un nouveau sujet Répondre
Mot :    Pseudo :           
 
ghost_blue   27-05-2007 à 02:17:07

Bonjour,

Depuis plusieurs mois je fais du C et la j'ai voulu réaliser un carnet d'adresse en C console.
Mais j'ai un probléme, c'est quand je compile le programme et que je le test, tout marche comme je veux sauf arrivée au niveau de "l'adresse", dés que je met un espace dans ce que je rentre, aprés sa beug tout le reste. J'ai éssaié avec plusieur compilateur t'elle que code::block, Dev-C++, Visual basic C++, et j'ai toujours le même probléme, est ce que quelqu'un serait pourquoi ça me fait sa ?

Merci

------------------------------ http://handi-decouverte.site.voila.fr
Répondre
Liens sponsorisés
Inscrivez-vous ou connectez-vous pour masquer ceci.
zoizoboules   27-05-2007 à 12:51:19
- 0 +

Bonjour,
Est-ce qu'on pourrait voir le code source s'il vous plait ?

Répondre à zoizoboules
gryphus71   27-05-2007 à 14:59:39
- 0 +

t'as bien utilisé des gets et puts pour récupérer et afficher les adresses ?

Répondre à gryphus71
ghost_blue   27-05-2007 à 19:49:53
- 0 +

voici le code

Code :
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include <string.h>
  4. #include "adresse.h"
  6. double ajout(char nom, char prenom, char adresse, char ville, char codePostal, char telephone);
  7. double recherche(char nom, char prenom, char adresse, char ville, char codePostal, char telephone);
  9. int main()
  10. {
  11.     long menu = 0;
  12.     char prenom[100], nom[100], adresse[1000], ville[50], codePostal = 0, telephone = 20;
  13.     FILE* fichier = NULL;
  15.     printf("\t\t\tC A R N E T  D ' A D R E S S E\n\n\n" );
  17.     printf("1. Ajout d'adresse\n" );
  18.     printf("2. Recherche d'adresse\n" );
  19.     printf("3. Quitter\n" );
  20.     printf("\n" );
  22.     printf("Votre choix : " );
  23.     scanf("%ld", &menu);
  25.     switch (menu)
  26.     {
  27.     case 1 :
  28.     fichier = fopen("adresse.txt", "w" );
  30.     if (fichier =! NULL)
  31.     {
  32.       ajout(nom, prenom, adresse, ville, codePostal, telephone);
  33.     }
  34.     else
  35.     {
  36.       printf("Impossible d'ouvrir le fichier adresse.txt" );
  37.     }
  39.     break;
  41.     case 2 :
  42.     recherche(nom, prenom, adresse, ville, codePostal, telephone);
  43.     break;
  45.     case 3 :
  46.     break;
  48.     default :
  49.     printf("\nMauvaise entree\n\n" );
  50.     break;
  51.     }
  54.     return 0;
  55. }



Code :
  1. double ajout(char nom, char prenom, char adresse, char ville, char codePostal, char telephone)
  2. {
  4.   printf("\n" );
  5.   printf("Nom : " );
  6.   scanf("%s", nom);
  7.   printf("Prénom : " );
  8.   scanf("%s", prenom);
  9.   printf("Adresse : " );
  10.   scanf("%s", adresse);
  11.   printf("Code postal : " );
  12.   scanf("%ld", codePostal);
  13.   printf("\n" );
  14.   printf("Ville : " );
  15.   scanf("%s", ville);
  16.   printf("\n" );
  17.   printf("Telephone : " );
  18.   scanf("%ld", telephone);
  20.   return 0;
  21. }
  23. double recherche(char nom, char prenom, char adresse, char ville, char codePostal, char telephone)
  24. {
  25.   printf("Nom : " );
  26.   scanf("%s", &nom);
  27.   printf("\n" );
  28.   printf("Prenom : " );
  29.   scanf("%s", &prenom);
  32.   printf("voici le résultat :\n" );
  33.   printf("%s %s\n", nom, prenom);
  34.   printf("%s\n", adresse);
  35.   printf("%ld %s", codePostal, ville);
  36.   printf("\n\n" );
  38. }



non j'ai oublié de mettre gets et puts, et c'est possible que sa vienne de ça mon beug ?

------------------------------ http://handi-decouverte.site.voila.fr
 Répondre à ghost_blue
gryphus71   27-05-2007 à 21:03:37
- 0 +

franchement j'ai pas lu ... mais j'ai vu que tu utilisais pas mal de scanf ... essaye de bannir les scanf qui presentent une faille ... officielle sous linux car si tu programmes sous linux : il te mettera une erreur à chaque utilisation de scanf ;) ... windows préfère continuer à les utiliser (boulets ? qui a dit boulets ?)

 

donc essaye déjà de les remplacer et tu verras si ca fait encore l'erreur revient nous le dire ;)

 

PS : je tiens cette info d'un de mes profs en prog + j'avais lu le rapport officiel mais je le retrouve plus ... même avec google lol


Message édité par gryphus71 le 27-05-2007 à 21:07:59
Répondre à gryphus71
ghost_blue   27-05-2007 à 22:38:27
- 0 +

ok, mais je les remplaces par quoi ?

------------------------------ http://handi-decouverte.site.voila.fr
 Répondre à ghost_blue
gryphus71   27-05-2007 à 22:52:01
- 0 +

ben tu utilises gets !

Répondre à gryphus71
ghost_blue   27-05-2007 à 22:53:49
- 0 +

ouai mais c'est pour écrire dans un fichier donc sa risque de déconner non ?

------------------------------ http://handi-decouverte.site.voila.fr
 Répondre à ghost_blue
gryphus71   27-05-2007 à 22:58:42
- 0 +

heuuu je crois qu'on s'est pas compris ^^ :
le scanf ... tu l'utilise pour récupérer une information ! tu n'utilises pas un scanf pour écrire dans un fichier ! tu utilises ... je sais plus mais c'est une autre fonction !

ben là tu utilise gets pour demander l'info à la personne au clavier, puis une fois que tu auras récupéré cette info (avec gets) tu l'écris dans le fichier avec l'autre fonction

Répondre à gryphus71
gryphus71   27-05-2007 à 23:14:03
- 0 +

si je ne me trompe pas, pour récupérer une donnée tu as plusieurs choix, entre autre :

char DonneeVoulue[30];

/*ca c'est pour récupérer une chaine de longueur inconnue :*/
gets(DonneeVoulue);

/*ca c'est pour récupérer une chaine de longueur définie (ici 12 caractères max : 13-1=12, le -1 est pour le retour chariot '\n') :*/
fgets(DonneeVoulue, 13, stdin);

Répondre à gryphus71
gryphus71   27-05-2007 à 23:16:18
- 0 +

vas lire ca : http://www.linux-kheops.com/doc/ma [...] 3.txt.html
lis toutela page et tu comprendras mieu ;)

Répondre à gryphus71
ghost_blue   27-05-2007 à 23:35:59
- 0 +

quand je disais, c'est pour écrire dans un fichier, je parlais de gets, merci pour le lien je l'irai tout à l'heure

------------------------------ http://handi-decouverte.site.voila.fr
 Répondre à ghost_blue
gryphus71   28-05-2007 à 00:19:36
- 0 +

gets c'est pas pour écrire dans un fichier !
tu dois confondre avec :
fputs : ca c'est pour écrire dans un fichier (enfin je crois me souvenir lol)
ou
fgets : qui est utilisé pour récupérer une donnée d'un fichier

Répondre à gryphus71
chrisz   28-05-2007 à 11:00:15
- 0 +

le proto de ta fonction ajout n'est pas bon, tes paramètres sont de type tableau de char :
double ajout(char* nom, char* prenom, char* adresse, char* ville, char* codePostal, char* telephone);

scanf et gets sont deux fonctions aussi dangereuses l'une que l'autre, puisqu'on ne passe pas la taille maximale du buffer (donc buffer overflow possible).
Tu peux utiliser fgets. (http://www.siteduzero.com/tuto-3-1 [...] fgets.html)


Message édité par chrisz le 28-05-2007 à 11:04:28
Répondre à chrisz
defaultuser   28-05-2007 à 13:39:14
- 0 +

Citation :


si je ne me trompe pas, pour récupérer une donnée tu as plusieurs choix, entre autre :

char DonneeVoulue[30];

/*ca c'est pour récupérer une chaine de longueur inconnue :*/
gets(DonneeVoulue);




Au secours ce code est a bannir , risque de sécurité .

Il faut utiliser fgets pour les saisies clavier , point barre.

------------------------------ powered by gentoo

 

Répondre à defaultuser

gryphus71   28-05-2007 à 14:11:01
- 0 +

Ô_ô
si vous faites du dynamique : vous ne pourvez pas connaitre la taille du texte saisi à l'avance !
c'est juste qu'il faut faire attention d'utiliser la fonction la + appropriée !

Répondre à gryphus71
Créer un nouveau sujet Répondre
Tom's Guide > Forum > Programmation > Langage C
Aller à :

Il y a 2052 utilisateurs connus et inconnus. Pour voir la liste des connectés connus, cliquez ici.

Plan du forum
Forum Bestofmedia ©
2000-2009 Bestofmedia Group
Page générée en 0,398 secondes
Attention

Vous allez répondre sur un sujet resté inactif pendant plus de 6 mois.
Assurez-vous d'apporter des éléments nouveaux à la discussion avant de poursuivre.

Répondre Annuler
Liens