Malloc tableau de chaine de caractere

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 :oops:

Voila le bout de code qui fait des siennes :

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

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

Citation:

---

Envoyé par 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 :oops:

Voila le bout de code qui fait des siennes :

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

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:

---

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

---

Code:

---

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

---

2 est un int et non un char !

Citation:

---

Envoyé par firemax

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

---

:faq:
Comment allouer dynamiquement un tableau à 2 dimensions ?

;)

sinon tu peux faire qqch du genre

Code:

---

1 2 3

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

---

puis

Code:

---

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

---

avec un joli

Code:

---

1 2 3

free(mat[i]), mat[i]=NULL; free(mat), mat=NULL;

---

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

Code:

---

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

salut tu peux essayer celà

Code:

---

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 }

---

Citation:

---

Envoyé par salseropom

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

Code:

---

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

---

Code:

---

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

Citation:

---

Envoyé par homeostasie

:faq:
Comment allouer dynamiquement un tableau à 2 dimensions ?

;)

---

Thanks

Mais ca segfault toujours a cette ligne

Code:

---

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)

---

Citation:

---

Envoyé par firemax

Mais ca segfault toujours a cette ligne

Code:

---

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:

---

1 2 3

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

---

Citation:

---

Envoyé par diogene

Code:

---

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 :oops:

Et j'avais mal fait mon deuxieme malloc aussi XD

Now ca fonctionne thanks a tous ceux qui ont repondu

pense à la balise Résolu

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

#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; }

---

salut
ne vous chamaillez pas ça marche pour les deux cas 0 ou NULL
essayer ce code

Code:

---

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

---

Citation:

---

Envoyé par archer

salut
ne vous chamaillez pas ça marche pour les deux cas 0 ou NULL
essayer ce code

Code:

---

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

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

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