Discussion :

[oliv928]

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

[Médinoc]

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

[Emmanuel Delahaye]

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

Commence déjà par corriger ceci :

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
 
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

[Skyounet]

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

Ah si si je pense qu'il devait en avoir

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.

[oliv928]

et il faut que je le rende début janvier...

[diogene]

Tout au commencement :

Code :

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

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!

[oliv928]

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 !

[oliv928]

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

[Skyounet]

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

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

[oliv928]

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.

[akoril]

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 :

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

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 :

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

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 :

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
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.