Discussion :

[firemax]

Bonjour

Je travaille sur un projet et je ne heurte a un seg fault qui me resiste.

Je cherche a recopier une map (carre de nb_line sur len_line) mais je n'arrive pas a mallocer un tableau a plus d'une dimension

Voila le bout de code qui fait des siennes :

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
char	**declare(int nb_line, int len_line, char *path)
{
  char	**mat;
  int	fd;
  int	x;
  int	y;
 
  y = 0;
  /*Dans cette partie je bloque*/
  while (y <= nb_line)
    {
      if((mat[y] = malloc((len_line + 1) * sizeof(*mat))) == 0)
	{
	  my_putstr("Error: Could not malloc\n");
	  return (0);
	};
      y++;
    }
  /*J'ouvre le fichier*/
  if ((fd = open(path, O_RDONLY)) <= 0)
    my_putstr("Error: File can't be open in function nb_line\n");
  /*Je passe la premiere ligne car je ne dois pas la recopier et je la traite ailleurs*/
  mat[0][0] = 2;
  while (mat[0][0] != '\n')
    if (read(fd, &mat[0][0], BUF_SIZE) <= 0)
      return (0);
  /*Je recopie ma map*/
  y = 0;
  while (y <= nb_line)
    {
      x = 0;
      while (x <= len_line)
	{
	  read(fd, &mat[y][x], BUF_SIZE);
	  x++;
	}
      y++;
    }
  close(fd);
  return (mat);
}

Voila si vous avez une idee ou que vous voyez une erreur (pas besoin de dire que mon code est moche ca je l'avais vu XD) n'hesitez pas a m'expliquez

Merci

[salseropom]

Bonjour

Je travaille sur un projet et je ne heurte a un seg fault qui me resiste.

Je cherche a recopier une map (carre de nb_line sur len_line) mais je n'arrive pas a mallocer un tableau a plus d'une dimension

Voila le bout de code qui fait des siennes :

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
char	**declare(int nb_line, int len_line, char *path)
{
  char	**mat;
  int	fd;
  int	x;
  int	y;
 
  y = 0;
  /*Dans cette partie je bloque*/
  while (y <= nb_line)
    {
      if((mat[y] = malloc((len_line + 1) * sizeof(*mat))) == 0)
	{
	  my_putstr("Error: Could not malloc\n");
	  return (0);
	};
      y++;
    }
  /*J'ouvre le fichier*/
  if ((fd = open(path, O_RDONLY)) <= 0)
    my_putstr("Error: File can't be open in function nb_line\n");
  /*Je passe la premiere ligne car je ne dois pas la recopier et je la traite ailleurs*/
  mat[0][0] = 2;
  while (mat[0][0] != '\n')
    if (read(fd, &mat[0][0], BUF_SIZE) <= 0)
      return (0);
  /*Je recopie ma map*/
  y = 0;
  while (y <= nb_line)
    {
      x = 0;
      while (x <= len_line)
	{
	  read(fd, &mat[y][x], BUF_SIZE);
	  x++;
	}
      y++;
    }
  close(fd);
  return (mat);
}

Voila si vous avez une idee ou que vous voyez une erreur (pas besoin de dire que mon code est moche ca je l'avais vu XD) n'hesitez pas a m'expliquez

Merci

en dehors du seg fault, si le malloc échoue tu renvoie 0 qui est un int et non un char **

Code :

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

1
2
if((mat[y] = malloc((len_line + 1) * sizeof(*mat[y]))) == 0)

Code :

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

1
2
 
mat[0][0] = 2;

2 est un int et non un char !

[homeostasie]

Je cherche a recopier une map (carre de nb_line sur len_line) mais je n'arrive pas a mallocer un tableau a plus d'une dimension

Comment allouer dynamiquement un tableau à 2 dimensions ?

[salseropom]

sinon tu peux faire qqch du genre

Code :

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

1
2
3
 
char ** mat=malloc(N*sizeof(*mat));
if(mat==NULL) /* erreur */

puis

Code :

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

1
2
 
mat[i]=strdup("ma chaine");

avec un joli

Code :

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

1
2
3
 
free(mat[i]), mat[i]=NULL;
free(mat), mat=NULL;

je pense que c'est encore plus simple...

[diogene]

Code :

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

1
2
3
4
5
6
7
char	**declare(int nb_line, int len_line, char *path)
{
  char	**mat;
...
      if((mat[y] = malloc((len_line + 1) * sizeof(*mat))) == 0)
...
}

Ton erreur est que mat n'est pas un tableau, mais un pointeur. mat[y] va donc planter si y !=0.
- tu dois d'abord allouer un tableau de pointeur sur char et mettre l'adresse de ce tableau dans mat.
- tu peux ensuite mettre dans chaque élément de ce tableau l'adresse des tableaux de char que tu alloues
- dans ton malloc, le sizeof * mat est incorrect : c'est la taille d'un pointeur sur char (couramment 4) et non celle d'un char (qui vaut toujours 1)
- Compare à NULL, pas à 0
-En cas d'erreur d'allocation, tu devrais libérer toutes les mémoires précédemment allouées avant de retourner NULL

[archer]

salut tu peux essayer celà

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
 
#include <stdio.h>
#include<malloc.h>
 
int main ()
{
  int i;
  char	**mat;
  mat=(char**)malloc(10*(sizeof(char*)));//allouer un pointeur de pointeurs qui représentera les lignes
  for(i=0;i<10;i++)
	mat[i]=(char*)malloc(10*sizeof(char));//allouer chaque ligne séparément
//comme ça tu obtiens une matrice 10*10
  for(i=0;i<10;i++)
	  free(mat[i]);//liberer chaque ligne
 
  free(mat); /liberer le pointeur des pointeurs
}

[firemax]

en dehors du seg fault, si le malloc échoue tu renvoie 0 qui est un int et non un char **

Code :

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

1
2
if((mat[y] = malloc((len_line + 1) * sizeof(*mat[y]))) == 0)

Code :

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

1
2
 
mat[0][0] = 2;

2 est un int et non un char !

Bah pas grave pour le renvoi de 0 c'est quand meme un char qui correspond a NULL et pour le 2 ca correspond au troisieme caractere de la table ascii , c'est une valeur comme une autre pour etre sur que ma boucle puisse "demarrer"

Mais le *mat[y] dans malloc j.avais pas vu merci

Comment allouer dynamiquement un tableau à 2 dimensions ?

Thanks

Mais ca segfault toujours a cette ligne

Code :

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

1
2
3
4
Program received signal SIGSEGV, Segmentation fault.
0x08048868 in declare (nb_line=9, len_line=28, path=0xbfbfe8fe "expl_map")
    at main.c:108
108           if((mat[y] = malloc((len_line + 1) * sizeof(**mat))) == 0)

[salseropom]

Mais ca segfault toujours a cette ligne

Code :

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

1
2
3
4
Program received signal SIGSEGV, Segmentation fault.
0x08048868 in declare (nb_line=9, len_line=28, path=0xbfbfe8fe "expl_map")
    at main.c:108
108           if((mat[y] = malloc((len_line + 1) * sizeof(**mat))) == 0)

je metterai

Code :

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

1
2
3
 
mat[y]=malloc((len_line + 1) * sizeof(*mat[y]));
if(mat[y] == NULL) /* erreur */

[firemax]

Code :

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

1
2
3
4
5
6
7
char	**declare(int nb_line, int len_line, char *path)
{
  char	**mat;
...
      if((mat[y] = malloc((len_line + 1) * sizeof(*mat))) == 0)
...
}

Ton erreur est que mat n'est pas un tableau, mais un pointeur. mat[y] va donc planter si y !=0.
- tu dois d'abord allouer un tableau de pointeur sur char et mettre l'adresse de ce tableau dans mat.
- tu peux ensuite mettre dans chaque élément de ce tableau l'adresse des tableaux de char que tu alloues
- dans ton malloc, le sizeof * mat est incorrect : c'est la taille d'un pointeur sur char (couramment 4) et non celle d'un char (qui vaut toujours 1)
- Compare à NULL, pas à 0
-En cas d'erreur d'allocation, tu devrais libérer toutes les mémoires précédemment allouées avant de retourner NULL

Je suis un gros boulay

Merci j'oubliais d'allouer ma 1er dimnsion a mon tableau

Et j'avais mal fait mon deuxieme malloc aussi XD

Now ca fonctionne thanks a tous ceux qui ont repondu

[salseropom]

pense à la balise Résolu

[nicolas.sitbon]

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
#include <stdio.h>
#include <malloc.h>
#define RAWS 10
#define COLS 10
 
int main (void)
{
   int i, err = 0;
   char **mat = malloc (RAWS * sizeof * mat);
   if (mat != NULL)
   {
      for (i = 0; i < COLS; i++)
      {
         mat[i] = malloc (COLS * sizeof * mat[i]);
         if(mat[i] == NULL)
         {
            err = 1;
         }
      }
      if(err == 0)
      {
         /* ce que tu veux... */
         for(i = 0; i < COLS; i++)
            free(mat[i]), mat[i] = NULL;
         free(mat), mat = NULL;
      }  
   }
   else
   {
      err = 1;
   }
 
   return err;
}

[archer]

salut
ne vous chamaillez pas ça marche pour les deux cas 0 ou NULL
essayer ce 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
 
#include <stdio.h>
#include<malloc.h>
 
int main ()
{
  int i;
  char	**mat;
  if((mat=(char**)malloc(10*(sizeof(char*))))!=NULL)//allouer un pointeur de pointurs qui représentera les lignes ici on peut mettre 0 à la place de NULL
  { 
	for(i=0;i<10;i++)
		mat[i]=(char*)malloc(10*sizeof(char));//allouer chaque ligne séparemment
	mat[5][5]='c';
	printf("%c",mat[5][5]);
	for(i=0;i<10;i++)
	  free(mat[i]);//liberer chaque ligne
	free(mat); //liberer le pointeur des pointeurs
  }
 
}

[nicolas.sitbon]

salut
ne vous chamaillez pas ça marche pour les deux cas 0 ou NULL
essayer ce 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
 
#include <stdio.h>
#include<malloc.h>
 
int main ()
{
  int i;
  char	**mat;
  if((mat=(char**)malloc(10*(sizeof(char*))))!=NULL)//allouer un pointeur de pointurs qui représentera les lignes ici on peut mettre 0 à la place de NULL
  { 
	for(i=0;i<10;i++)
		mat[i]=(char*)malloc(10*sizeof(char));//allouer chaque ligne séparemment
	mat[5][5]='c';
	printf("%c",mat[5][5]);
	for(i=0;i<10;i++)
	  free(mat[i]);//liberer chaque ligne
	free(mat); //liberer le pointeur des pointeurs
  }
 
}

Il y a des cours si tu veux ici :
http://emmanuel-delahaye.developpez....tes.htm#malloc
http://emmanuel-delahaye.developpez.....htm#tabdyn_2d

[salseropom]

certes, ça marche pour 0 et NULL mais pour ma part, je trouve que NULL est plus "parlant" que 0...

[archer]

je suis d’accords avec toi sur NULL puisqu’on traite ici des pointeurs
mon but était seulement de mettre fin à cette dispute