probleme sur un fscanf vers une structure dynamique

Version imprimable

30/12/2006, 12h39
oliv928

1 pièce(s) jointe(s)

probleme sur un fscanf vers une structure dynamique

bonjour,
j'ai écrit avec un collègue un programme qui doit gérer une petite "base de données" d un cabinet médical. Ce programme repose sur la gestion de deux files de structures autoreferencees par pointeurs sur la fiche suivante. Les structures sont : Consultation et Ordonnances. Elles sont elles meme liées entre elles : chaque fiche de Consultation pointe vers une premiere Ordonnance.
Le probleme :
Lors de la lecture du fichier (un fichier par patient ou No INSEE ), le fscanf fait tout planter: "violation d'acces". Alors qu'à la compilation il n y a que des warnings. (devC++).Par contre pour la sauvegarde ca marche.

Le probleme semble se trouver dans la fonction load_file à la ligne du fscanf.

si quelqun a une idée ca me dépannerait bien.

bonnes fetes à tous. Olivier
30/12/2006, 13h30
Médinoc

J'ai vu un scanf("%s") et j'ai arrêté de lire.

Citation:

Envoyé par oliv928

bonjour,
j'ai écrit avec un collègue un programme

Commence déjà par corriger ceci :

Code:

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums2\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c:35: warning: function declaration isn't a prototype
main.c:36: warning: function declaration isn't a prototype
main.c:45: warning: function declaration isn't a prototype
main.c:78: warning: function declaration isn't a prototype
main.c:95: warning: function declaration isn't a prototype
main.c:111: warning: function declaration isn't a prototype
main.c: In function `new_medic':
main.c:187: warning: assignment from incompatible pointer type
main.c:221: warning: return from incompatible pointer type
main.c: In function `Save_client_file':
main.c:235: warning: assignment from incompatible pointer type
main.c: In function `load_file':
main.c:264: warning: assignment discards qualifiers from pointer target type
main.c: In function `sim_sauv':
main.c:312: warning: passing arg 1 of `printf' from incompatible pointer type
main.c:314: warning: assignment from incompatible pointer type
main.c:315: warning: passing arg 1 of `printf' from incompatible pointer type
main.c: In function `new_medic':
main.c:152: warning: 'last_med' might be used uninitialized in this function
main.c: In function `load_file':
main.c:249: warning: 'LeFichierClient' might be used uninitialized in this function
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 6 seconds)
0 errors, 15 warnings

Si ton compilateur ne dit rien, c'est qu'il est probablement mal reglé...

http://emmanuel-delahaye.developpez....tm#cfg_compilo

30/12/2006, 15h55
Skyounet

Citation:

Envoyé par Emmanuel Delahaye

Si ton compilateur ne dit rien, c'est qu'il est probablement mal reglé...

Ah si si je pense qu'il devait en avoir

Citation:

Alors qu'à la compilation il n y a que des warnings.

<malloc.h> ca existe po.

On dirait plus un TP d'étudiant, plutôt qu'un programme destiné à un usage professionnel. :roll: :roll:
30/12/2006, 21h22
oliv928

c est un tp d etudiant

et il faut que je le rende début janvier...:oops:
30/12/2006, 22h15
diogene
Tout au commencement :
Code:

1 2 3 4 5 6 char* lire_String(){ char * chaine_a_lire = calloc(1, sizeof *chaine_a_lire); scanf("%s",chaine_a_lire); return chaine_a_lire; }
Cette fonction est à l'évidence vérolée : sizeof *chaine_a_lire = sizeof char =1
On alloue de la place pour 1 char. Si la chaîne lue est plus grande que 0 caractère(s) (LE caractère étant utilisé pour '\0') la fonction va planter.
Ca commence mal!
30/12/2006, 23h29
oliv928

c est à la ligne 259 que ca plante

soit dans la fonction de chargement des données :
soit ici :
fscanf (LeFichierClient,"%d\t%d\t%d\t%s\n",
&UneFiche->jour,
&UneFiche->mois,
&UneFiche->annee,
UneFiche->nom_maladie);
ca fait des jours que je suis dessus et rien à faire. je vais me taper une taule !:oops:
31/12/2006, 01h10
oliv928

2 pièce(s) jointe(s)

plus precis

pour essayer de voir ce qui deconne jai ajouté une fonction qui simule la lecture fichier mais avec un scanf (donc lit au clavier) et des indicateurs.

printf("entrez fiche No %d\n",j);
printf("jour\t"); scanf ("%d\n",&jj); printf("mois\t"); scanf ("%d\n",&mm); printf("annee\t"); scanf ("%d\n",&aa);

et bien je ne comprends pas mais apres la saisie du 1er jour il demande une autre donnée ! des la deuxieme fiche tout rentre dans l ordre . c est incroyable je comprends pas ! j y passe mes journées .

aidez moi svp !

bonnes fetes
31/12/2006, 01h46
Skyounet

Ben commence par ne plus utiliser scanf mais fgets. Pour saisir un nombre, fgets + strtol.

http://emmanuel-delahaye.developpez....tes.htm#saisie
31/12/2006, 02h07
oliv928

1 pièce(s) jointe(s)

pourquoi ce program ne marche pas ?

dans le meme ordre d idee pourquoi le programme ci joint "ajoute une demande de saisie" supplémentaire apres le 1er jour ? il me semble que l erreur vient de la meme chose.

comme diogene je pense qu'il y a un pb. avec la variable char *nom_maladie de la structure CONSULTATION. l'allocation mémoire dans la fonction lire_String n'alloue qu'un seul caractère.
Evidement ça plante quand on s'en sert et qu'on tape dans une zone mémoire qui n'est pas alloué au programme.
- Alors soit:
Code:

1 2 3 4 5 6 #define _Max_Str_Len_ 256 char* lire_String(){ char * chaine_a_lire = calloc(1, _Max_Str_Len_); scanf("%s",chaine_a_lire); // ou sncanf(...) return chaine_a_lire; }
- Sinon dans la fonction new_consult () ajoute

Code:

chaine_a_lire = (char*)calloc(1, _Max_Str_Len_);

et ne plu le faire ailleur, mais remplacer par des strcpy.
- Ou déclare char nom_maladie[_Max_Str_Len_]; dans la structure...

C'est pas ce qu'il y a de mieux, mais pour un début c'est suffisant (et utilise des strcpy() ).

Je crois bien qu'il y a un Gros Bug dans ta gestion de liste consultation
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 consultation* new_consult (){ consultation *tempo_consult = malloc (sizeof(consultation)); consultation *last; int nb_consult = 1; ... if (nb_consult == 1){ Toujour = 1 ! last = tempo_consult; tempo_consult = NULL; nb_consult++; } else if (nb_consult > 1){ tempo_consult->suivant = last; last n'est pas initialisé! last = tempo_consult; tempo_consult = NULL; nb_consult++; } return last; }
Je crois que les variable last et nb_consult ne devrais pas être dans la fonction mais déclarées en global et initialisé une première fois (dans le main).

Tu es sur des tabulations ?
*%[ \t] devrais pouvoir remplacer les \t
Et je virerais par la même occasion le \n final, quitte à contrôler que neFiche->nom_maladie ne finis pas par un '\n', auquel cas on le remplace par '\x0'
Salut, et bon apprentissage. 8-)