Discussion :

[aimad41]

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]

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]

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]

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]

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

[aimad41]

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

[diogene]

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

Code :

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

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

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

Code :

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

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

[benhoeil]

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

sinon encore une modif !

Code :

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

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

doit être

Code :

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

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

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

[granquet]

attention, mode goret

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

[zooro]

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

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

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

[granquet]

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