copier sortie standard dans un pointeur de pointeur de char

**aimad41** · 02/11/2006, 15h42

Bonjour,

Voila, tout est dans le titre, je n'arrive pas a copier de la facon la plus basique qui soit
une chaine de cararacteres de ce type
"ssssssssssssssssss \n
sssssssssssssssssss \n
ssssssssssssssssssss \n\o"

ceci est une chaine de caracteres stockes on va dire dans *tab; comment faire pour stocker chaque ligne dans un char **tab2 du style : 1er ligne tab[0], 2eme ligne tab[1] ect.........
parce que j'ai du segfault de partout

voici le code principale

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
 
temp = malloc(sizeof (char) * 1000);
  av = malloc(sizeof (char) * 1000);
  while (tab[i] != '\0')
    {
      while (tab[i] != '\n')
        {
          temp[pos] = tab[i];
          i++;
          pos++;
        }
      av[n] = temp;
      n++;
      pos = 0;
      i++;
    }
  while (n >= 0)
    {
      write(1, av[n], strlen(av[n]));
      n--;
    }
  my_putchar('\n');

pour le malloc j/ai mis 1000 parce que je ne sait pas comment determiner a l'avance la taille du fichier

**diogene** · 02/11/2006, 19h05

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
temp = malloc(sizeof (char) * 1000);
  av = malloc(sizeof (char) * 1000);
  while (tab[i] != '\0')
    {
      while (tab[i] != '\n')
        {
          temp[pos] = tab[i];
          i++;
          pos++;
        }
      av[n] = temp;
      n++;
      pos = 0;
      i++;
    }

Cette partie est incorrecte puisque av[n] est (ou devrait être) un char et temp est (ou devrait être) un char *. On devrait avoir

Code :

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

char** av = malloc(sizeof (char*) * 1000);

Mettre temp dans av[n] revient à remplir av avec toujours la même valeur : l'adresse du tableau temp.

Si je comprend ce que tu veux faire (mais ce n'est pas sur) :
- tu découpes tab en lignes en mettant chaque ligne dans temp.
- Ensuite, tu alloues de la place pour mettre cette ligne en mettant l'adresse du tableau alloué dans av[n].
- Puis tu copies dans ce tableau la ligne qui est dans temp
- et tu recommence jusqu'à la fin de tab
- Tu fais ce que tu dois faire avec ces chaînes
- puis tu désalloue tous les tableaux dont l'adresse est dans av, puis av et temp et c'est fini.

Ou alors (ce qui dépend de ce que tu veux précisément en faire ensuite):
- tu explore le tableau tab pour trouver les débuts des lignes
- tu places l'adresse (dans tab ) du premier caractère de chaque début de ligne dans av. Ceci évite de recopier les lignes et d'avoir le tableau temp . Terminer le tableau av par NULL quand on est arrivé à la fin de tab
- On peut ainsi retrouver facilement le début de chaque ligne et sa longueur ( par différence avec l' adresse de la ligne suivante ) et le nombre de lignes.
- à la fin, désallouer av.

**D[r]eadLock** · 02/11/2006, 19h06

Montre-nous les types que tu définis (i.e. code compilable sans warnings !), parce que le av[n]=temp est un peu bizarre.

**benhoeil** · 02/11/2006, 22h32

je déteste ce problème, et ton post est l'occase pour moi d'essayer de trouver une bonne solution.

ce que je ferais à ta place : je compterais les retours à la ligne sur ta chaîne, et je créerais un tableau de pointeur de char d'autant d'éléments.
ensuite pour chaque pointeur, il suffit de déterminer la longueur de la ligne correspondante puis d'allouer la mémoire puis de recopier.

ce que j'ai trouvé :

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
char** function()
{
	char** ligne;
	...
	int i, nb;
	char *debut=NULL;
	char *fin=NULL;
 
	for(i=0,nb=0; tab[i]!=0; i++) {
		if(tab[i]=='\n') {
			nb++
		}
	}
	nb++; /* au cas ou la derniere ligne n'est pas suivie d'un retour à la ligne */
 
	ligne = (char**)malloc(sizeof(char*)*nb);
 
	debut=fin=tab
	for(i=0; *debut!=0; i++,fin++) {
		if((fin=='\n')||(fin==0)) {
			ligne[i] = (char*)malloc(sizeof(char)*(fin-debut+1));
			strncpy(ligne[i], debut, fin-debut);
			ligne[i][fin-debut+1] = 0;
			debut = fin;
		}
	}
 
	return ligne;
}

je n'ai pas testé, mais à priori il suffit de peu de modifs pour le faire marcher. pour faire propre, tu devrais tester ce que te renvois malloc à chaque fois.

normalement, tu as alloué juste ce qu'il faut de mémoire.

**aimad41** · 03/11/2006, 11h46

merci pour votre aide j'y reflechit et vous donne mes reactions apres ca

**aimad41** · 03/11/2006, 16h43

alors voila, j'ai choisit ta solution seule probleme une erreur qui type ""cpy_tab.c:35: warning: comparison between pointer and integer"

le code

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
 
#include <stdlib.h>
#include <fcntl.h>
 
char    **function(char *tab)
{
  char  **ligne;
  int i;
  int nb;
  char *debut=NULL;
  char *fin=NULL;
 
  nb = 0;
  for (i=0; tab[i] != 0; i++)
    {
      if (tab[i] == '\n')
        {
          nb++;
        }
    }
  nb++;
  ligne = (char**) malloc (sizeof (char*) *nb);
  debut = fin = tab;
  for (i=0; *debut != 0; i++, fin++)
    {
      if ((fin =='\n') || (fin == 0))
        {
          ligne[i] = (char*) malloc (sizeof (char) * (fin - debut + 1));
          strncpy(ligne[i], debut, fin - debut);
          ligne[i][fin - debut + 1] = 0;
          debut = fin;
        }
    }
  return(ligne);
}
 
int     main()
{
  int   rd;
  char  *tab;
  char **av;
 
  tab = malloc(sizeof(*tab) *(200));
  rd = read(0, tab, 1000);
  if (rd == -1)
    my_putstr("Could not read properly the file\n");
  else
   av = function(tab);
  my_putstr(av[0]);
}