Segmentation fault

Segmentation fault - C - Programmation

Marsh Posté le 09-02-2004 à 17:34:30    

Re bonjour,
Je m'y reprend pour vous expliquer mon pb.
J'ai une fonction :  
 

Code :
  1. int readDateDebut(struct rdv *unRdv, int numLigne ) {
  3.   int i = 0;
  4.   char* uneLigne;
  5.   char unChar;
  6.   int jour;
  7.   int mois;
  8.   int annee;
  9.   int minute;
  10.   int heure;
  12.   (int)uneLigne = malloc(100);
  15.   assert(numLigne > 0 );
  17.   //positionnement au debut du fichier.
  18.   fseek(monFichier,0,SEEK_SET);
  21.   //positionnement sur la bonne ligne.
  22.   while ( i != numLigne && !feof(monFichier) ) {
  23.    
  24.     fgets(uneLigne,100,monFichier);
  25.     i++;
  27.   }
  30.   //si on est sur la bonne ligne
  31.   if ( i == numLigne ) {
  33.     //r�cup�ration du jour de la date de debut.
  34.     unChar = uneLigne[0];
  35.     jour = atoi(&unChar);
  36.     jour = jour * 10;
  37.     unChar = uneLigne[1];
  38.  
  39.     //..je vous passe les détails.. (c pas de la que vient le pb.)
  40.    
  41.     minute = atoi(&unChar);
  42.     minute = minute * 10;
  43.     unChar = uneLigne[15];
  44.     minute = minute + atoi(&unChar);
  46.     unRdv->dateDebut.tm_min = minute; //<-
  47.     unRdv->dateDebut.tm_hour = heure; //<- C la que j'ai un segmentation fault.
  48.     unRdv->dateDebut.tm_mday = jour;  //<-
  49.     unRdv->dateDebut.tm_mon = mois;   //<-
  50.     unRdv->dateDebut.tm_year = annee;;//<-
  52.   } else {//sinon on est a la fin du fichier.
  54.     printf("EOF!!\n" );
  55.     return 1;
  56.   }
  58.   free(uneLigne);
  60.   return 0;
  62. }


 
et mon main() :  

Code :
  1. int main (int argc, char **argv) {
  3.   struct rdv *rdv1;
  5.   monFichier = fopen(argv[1],"r" );//ouvre le fichier pass� en parametre en lecture.
  7.   if ( monFichier == NULL ) {//test si l'ouverture c'est bien passer.
  8.    
  9.     close(monFichier);//on ferme le fichier
  10.     printf("Pas d'acces � %s \n", argv[1]);//et on enonce le probleme.
  12.     return 1;
  14.   } else {
  16.     printf("debut de la lecture du rdv de la ligne 2\n" );
  18.     //fseek(monFichier,0,SEEK_SET);
  20.     readRDV(rdv1,2);
  22.     printf("fin de la lecture du rdv de la ligne 2\n" );
  24.     fclose(monFichier);
  26.   }
  28.   return 0;
  30. }


et j'ai un segmentation fault la ou je l'ai mis en commentaire.
voici la ma struct rdv :

Code :
  1. struct rdv {
  2.   struct tm dateDebut;
  3.   struct tm dateFin;
  4.   char libelle[77];
  5. };


 
si qqun peut m'éclairer ca serait sympa.
Merci d'avance

Reply

Marsh Posté le 09-02-2004 à 17:34:30   

Reply

Marsh Posté le 09-02-2004 à 17:50:31    

Ben tu alloues pas la memoire pour ta structure:
==> struct rdv *rdv1;  
Tu declares un pointeur sur une structure (non allouée).
==> readRDV(rdv1,2);  
Si c'est la meme chose que: int readDateDebut(struct rdv *unRdv, int numLigne )
==> unRdv->dateDebut.tm_min = minute;
rien que la deja ca peut foirer, puisque unRdv pointe sur une zone non allouée.
 
A+,


---------------
There's more than what can be linked! --    Iyashikei Anime Forever!    --  AngularJS c'est un framework d'engulé!  --
Reply

Marsh Posté le 09-02-2004 à 18:02:34    

tu peux te calmer s'il te plait ? deux topics avec 2 titres pourris, deux fois que je perds mon temps pour voir que tu n'a pas alloué une pauvre structure

Reply

Marsh Posté le 09-02-2004 à 18:14:49    

char unChar;
int minute;  
[...]
minute = atoi(&unChar);


C'est pas très propre tout ça.
 
Je pense qu'il y a en fait pas mal de choses à reprendre dans ton code.
 
Quel est le format d'une ligne de ton fichier ?


Message édité par darkoli le 09-02-2004 à 18:16:08
Reply

Marsh Posté le 09-02-2004 à 19:24:55    

taz a écrit :

tu peux te calmer s'il te plait ? deux topics avec 2 titres pourris, deux fois que je perds mon temps pour voir que tu n'a pas alloué une pauvre structure


 
monseigneur à daigné regarder le topic, monseigneur est trop bon.
 
Arrêtez de poser des questions vous autres, monseigneur attend de la question de qualité, qui vaille le coup de bouger son lard

Reply

Marsh Posté le 09-02-2004 à 19:28:29    

darkoli a écrit :

char unChar;
int minute;  
[...]
minute = atoi(&unChar);


C'est pas très propre tout ça.

ça fait même « boom »

Reply

Marsh Posté le 09-02-2004 à 20:23:55    

Désolé pour ceux que j'ai ofensé...
 
Mais le problème est que ca passait tres bien chez moi comme ca.
Et je suis un grand débutant en C.
donc il faut vraiemnt que j'alloue de la mémoire pour chaque pointeur? Comment ca se fait que ca passait chez moi et pas a mon ecole ?
 
Pour ce qui est de la propreté du code est ce que qqun peut me conseillier qqchose car je vois pas pkoi c pas propre (je débute ..)...
 
Voici le format d'une ligne de mon fichier :

Code :
  1. 15/08/2003 12:00-13:00 Aller chez le coiffeur


Message édité par Khyna le 09-02-2004 à 20:26:02
Reply

Marsh Posté le 09-02-2004 à 20:53:37    

non
parce que t'avais du bol, tu te balladais dans uns egment de mémoire à tois

Reply

Marsh Posté le 09-02-2004 à 21:02:36    

Donc si je comprend bien a chaque pointeur il faut un malloc et un free ?...
Mais dans ce cas la comment je fais pour mon pointeur sur FILE (FILE *monFichier) je lui alloue quoi comme et mémoire ? idem pour rdv1 ?

Reply

Marsh Posté le 09-02-2004 à 21:13:24    

c'est fopen qui alloue en interne un ptr FILE, et fclose qui le libère.
pour rdv1 tu dois bien utiliser malloc et free.


---------------
-( BlackGoddess )-
Reply

Marsh Posté le 09-02-2004 à 21:13:24   

Reply

Marsh Posté le 09-02-2004 à 21:15:56    

khyna a écrit :

Donc si je comprend bien a chaque pointeur il faut un malloc et un free ?...
Mais dans ce cas la comment je fais pour mon pointeur sur FILE (FILE *monFichier) je lui alloue quoi comme et mémoire ? idem pour rdv1 ?

non. un pointeur est une chose complètement distincte des allocations dynamiques

Reply

Marsh Posté le 09-02-2004 à 21:30:39    

khyna a écrit :

Donc si je comprend bien a chaque pointeur il faut un malloc et un free ?...
Mais dans ce cas la comment je fais pour mon pointeur sur FILE (FILE *monFichier) je lui alloue quoi comme et mémoire ? idem pour rdv1 ?


 
Tu confonds deux choses:
Le pointeur et ce sur quoi il pointe.
- Tu n'alloues pas de la memoire a un pointeur, c'est fait quand tu le declares (puisque c'est juste un truc de taille fixe contenant une adresse memoire, le compilo sait tres bien allouer ca de maniere automatique).
Mais ton pointeur, il doit pointer sur quelque chose qui a ete alloué en memoire (et ca le compilo ne peut pas savoir a l'avance de quelle taille ca va etre, puisque tu peux tres bien faire pointer ton pointeur sur plusieurs objets successifs du type pointé: int *p, tu peux faire pointer ca sur un entier, ou sur une suite de n entiers; d'autre part, le compilo peut pas savoir a l'avance si tu veux pointer sur un nouvel objet, ou sur un objet existant deja). Donc ce sur quoi ton pointeur va pointer (ie stocker l'adresse memoire)
- Soit c'est deja alloué (par exemple, quand tu vas faire ton FILE *monFichier tu declares juste le pointeur, et quand tu fais monFichier = fopen(argv[1],"r" ); tu fais pointer ton pointeur sur un truc alloué par le systeme et retourné par la fonction fopen. (et dans le cas de trucs alloués par le systeme, il y a soit des fonctions systeme pour desallouer, soit c'est explicitement indiqué dans la documentation de la fonction systeme qui t'a fourni l'objet sur lequel ton pointeur pointe que c'est a toi de le desallouer apres utilisation).
- Soit c'est pas alloué, et c'est a toi de faire l'allocation a coup de malloc puis apres usage, de faire la desallocation a coup de free.
 
A+,


Message édité par gilou le 09-02-2004 à 21:37:48

---------------
There's more than what can be linked! --    Iyashikei Anime Forever!    --  AngularJS c'est un framework d'engulé!  --
Reply

Marsh Posté le 09-02-2004 à 21:46:43    

Merci bcp, je vais avancer grace a ca.
J'ai du boulot devant moi et je dois revoir tout mon programme alors...
Merci encore pour votre aide

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed