Discussion :

[Neuromancien2]

Bonjour,

Je cherche à remplir un tableau de chaîne à partir des lignes d'un fichier.
J'ai le code suivant :

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
char *OKLines = NULL;
int OKLinesCount = 0;
 
while (fgets(line, LINE_SIZE, fp) != NULL) {
	addLine(line, &OKLines, OKLinesCount);
	OKLinesCount ++;
}
 
void addLine(char *line, char **lines, int count) {
	*lines = (char*)realloc (*lines, (count + 1) * sizeof (line));
	if (NULL == lines) {
		printf("Mémoire insuffisante.\n");
		free (lines);
		exit (EXIT_FAILURE);
	}
	strcpy((*lines + count), line);
}

Cela provoque un coredump. Je pense que sizeof (line) ne fonctionne pas.
Quelle serait la bonne syntaxe ?

[diogene]

Tu cherches à construire par allocation dynamique un tableau de chaines, chaque chaine étant allouée de façon dynamique. Chaque chaine correspondra à un pointeur char*. Le tableau de chaines correspondra donc à un pointeur char**. On aura donc

Code :

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

1
2
3
4
5
6
char ** OKLines = NULL; // le tableau est vide
int OKLinesCount = 0;   // ...
while (fgets(line, LINE_SIZE, fp) != NULL) {
	addLine(line, &OKLines, OKLinesCount); // Comment savoir que l'insertion a reussi ?
	OKLinesCount ++;
}

Pour savoir si l'insertion a réussi, il faut que la fonction renvoie un indicateur. Par exemple :

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
void * addLine(char *line, char ***lines, int count)
{
   char ** tab = NULL;
   // Allouer de la place pour la nouvelle chaine :
   char * chaine = malloc(strlen(line)+1);
   if(chaine != NULL) {
      // Allouer un élément de plus au tableau : 
      tab = realloc(*lines, (count+1)* sizeof *tab);
      if(tab != NULL){
         *lines = tab ;
          // Copier la chaine et son pointeur : 
          strcpy(chaine,line);
          tab[count] = chaine;
      }
      else free(chaine), chaine = NULL; // échec de la réallocation
   } 
   return chaine;
}

La fonction retourne NULL en cas d'échec d'allocation et le tableau de chaines et les chaines sont alors inchangés.
La séquence d'appel devient par exemple :

Code :

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

1
2
3
4
5
6
7
8
9
char ** OKLines = NULL; // le tableau est vide
int OKLinesCount = 0;   // ...
while (fgets(line, LINE_SIZE, fp) != NULL) {
   if(addLine(line, &OKLines, OKLinesCount)!= NULL) OKLinesCount ++;
   else {
      printf("Mémoire insuffisante.\n");
      break;
   }
}

[Neuromancien2]

Merci.