copier sortie standard dans un pointeur de pointeur de char

Version imprimable

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:

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

Code:

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:

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.

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

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

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:

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.

03/11/2006, 11h46
aimad41

ok

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

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:

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]);
}

03/11/2006, 17h19
diogene

Citation:

seule probleme une erreur qui type ""cpy_tab.c:35: warning: comparison between pointer and integer"

Code:

if ((fin =='\n') || (fin == 0)) ...

fin est un pointeur sur char et '\n' un int. Probablement

Code:

if (*fin =='\n' || *fin == 0) ...
03/11/2006, 18h06
benhoeil

oui diogene, erreur de ma part (que je fais tout le temps :()

sinon encore une modif !

Code:

ligne[i][fin-debut+1] = 0;

doit être

Code:

ligne[i][fin-debut] = 0;

sinon tu tapes en dehors du tableau. Moi ça m'embête toujours ces indices.

attention, mode goret :aie:

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
32
33
34
35
36
37
38
39
40
41
42
 
void
getfilecontent ()
{
 
  long lenfic;
  int j, l;
  char *tab, *pstr, *istr;
  FILE *fic;
 
  fic = fopen ("fic.txt", "r");
  if (fic != NULL)
    {
      fseek (fic, 0L, SEEK_END);
      lenfic = ftell (fic);
      tab = malloc (lenfic);
 
      if (tab == NULL)
	{
	  perror ("malloc");
	  return;
	}
      fread (tab, 1, lenfic, fic);
      if (fclose (fic) != 0)
	{
	  perror ("fclose");
	}
      istr = tab;
      while ((pstr = strchr (istr, '\n')) != NULL)
	{
	  nbrlines++;
	  *pstr = 0;
	  istr = pstr + 1;
	}
 
      for (j = 0, l = 0; j < nbrlines; j++, l += strlen (tab[l]) + 1)
	{
	  printf ("%s\n", tab[l]);
	}
    }
  free (tab);
}

bon alors? je score combien sur l'echelle du goret? :mrgreen:

03/11/2006, 19h51
zooro

Citation:

Envoyé par Dark_Ebola

bon alors? je score combien sur l'echelle du goret? :mrgreen:

Sur celle-là, je dirais Force 4, à cause des TAB au lieu des espaces...

Non, je n'ai pas que ça à faire, passer mon vendredi soir à compter les espaces dans le code posté. J'ai vraiment une vie, moi. Si, si, je vous jure :mouarf:
03/11/2006, 20h36
Médinoc

Pour cette utilisation-là des tabs, pour une fois je suis d'accord: C'est ignoble!

Edit: Et j'ajoute un niveau de goret pour utilisation de l'indentation GNU!
03/11/2006, 23h53
granquet

z'avez aucun gout, l'indentation GNU c'est la meilleure :mrgreen: (on est unpeu hors sujet y me semble :P)