passage de tableau dynamique

salut,
j'ai écrit un bout de script simplifié au maximum pour comprendre pourquoi je n'arrive pas à passer un tableau de caractère dans une fonction.
voici mon 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

#include "libecoPCR/ecoPCR.h" #include <stdio.h> #include <string.h> #include <getopt.h>   void printpattern(char *pattern, int p){ int i; for(i=0;i<p;i++){ printf("%s\n",pattern[i]); // warning: format '%s' expects type 'char *', but argument 2 has type 'int' } }   int main(int argc, char **argv){   char **pattern = MALLOC(sizeof(char *),"error pattern initialisation"); int p =0;   for (p=0 ; argc > optind ; optind++, p++){ pattern[p] = MALLOC(strlen(argv[optind])+1, "Error on pattern allocation"); pattern[p] = argv[optind]; }   printpattern(*pattern,p);   return 0;   }

---

Mon compilateur me met un warning et j'ai un bus error à l'execution. J'ai essayé différentes solutions, mais je n'arrive pas à récupérer le contenu de mon tableau pattern.
Merci du coup de main

Salut !


Tu veux passer un pointeur de pointeur sur un argument qui n'est qu'un simple pointeur... normal, change l'argument de ta fonction au moins en pointeur de pointeur sur char !

L'appel n'est pas bon:

Code:

---

printpattern(*pattern,p);

---

Fait plutôt:

Code:

---

printpattern(pattern,p);

---

ok, ça marche. :D
merci beaucoup, je commençais à chauffer sérieusement

Citation:

---

Envoyé par julien.63

salut,
j'ai écrit un bout de script....

---

ATTENTION à l'usage des mots.. Tu es en informatique, chaque mot a un sens...

Ce n'est pas un script, mais un programme, que tu as écrit..

Citation:

---

Envoyé par julien.63

ok, ça marche. :D
merci beaucoup, je commençais à chauffer sérieusement

---

Bin faut juste rester logique c'est tout. Tu utilises un pointeur de pointeur alors il est normal que ton argument de la fonction soit au moins du même type !

Citation:

---

Envoyé par julien.63

salut,
j'ai écrit un bout de script simplifié au maximum pour comprendre pourquoi je n'arrive pas à passer un tableau de caractère dans une fonction.
voici mon code :
<...>Mon compilateur me met un warning et j'ai un bus error à l'execution. J'ai essayé différentes solutions, mais je n'arrive pas à récupérer le contenu de mon tableau pattern.

---

Bah, il faut être simple, logique et cohérent :

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

#include <stdio.h> #include <string.h> #include <stdlib.h>   void printpattern (char **pattern, int p) { int i; for (i = 0; i < p; i++) { printf ("%s\n", pattern[i]); } }   int main (int argc, char **argv) { char **pattern = malloc (sizeof *pattern * (argc - 1)); if (pattern != NULL) { int r = 1; int w = 0; while (r < argc) { pattern[w] = strdup (argv[r]); /* POSIX.1 */ r++; w++; } printpattern (pattern, w); free (pattern); } return 0; }

---

Pose des questions si tu ne comprends pas (et adapte ce code à getopt())

Avec :

Code:

---

1 2	* monappli -s10 -d -a12 -c15

---

ça donne :

Code:

---

1 2 3 4 5 6 7

-s10 -d -a12 -c15   Press ENTER to continue.

---

voici une version avec getopt

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

#include <stdio.h> #include <string.h> #include <getopt.h> #include <stdlib.h>   void printpattern(char **pattern, int p){ int i; for(i=0;i<p;i++){ printf("%s\n",pattern[i]); } }   int main(int argc, char **argv){   char **pattern = malloc(sizeof *pattern * (argc -1)); int p = 0;   while (getopt(argc, argv, "a:b:c:") != -1) { pattern[p] = strdup(optarg); p++; }   printpattern(pattern,p); free(pattern);   return 0;   }

---

donne

Code:

---

1 2 3 4

SANCY:~/Documents/c $ ./ecotest -asdf -boii -cppe # sdf # oii # ppe

---

Citation:

---

Envoyé par julien.63

voici une version avec getopt

---

Attention à libérer ce qui a été alloué avec strdup() :

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

#include <stdio.h> #include <string.h> #include <getopt.h> #include <stdlib.h>   void printpattern (char **pattern, int p) { int i; for (i = 0; i < p; i++) { printf ("%s\n", pattern[i]); } }   int main (int argc, char **argv) { char **pattern = malloc (sizeof *pattern * (argc - 1)); int p = 0; while (getopt (argc, argv, "a:b:c:") != -1) { pattern[p] = strdup (optarg); p++; } printpattern (pattern, p);   { int i; for (i = 0; i < p; i++) { free (pattern[i]); } } free (pattern); return 0; }

---

Est qu'on pourrait imaginer une fonction pour libérer la mémoire?

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

#include <stdio.h> #include <string.h> #include <getopt.h> #include <stdlib.h>   void freememory(char **tab, int num){ int i; for (i=0;i<=num;i++) { free(tab[i]); } free(tab); }   void printpattern (char **pattern, int p) { int i; for (i = 0; i < p; i++) { printf ("%s\n", pattern[i]); } }   int main (int argc, char **argv) { char **pattern = malloc (sizeof *pattern * (argc - 1)); int p = 0; while (getopt (argc, argv, "a:b:c:") != -1) { pattern[p] = strdup (optarg); p++; } printpattern (pattern, p); freememory (pattern, p); return 0; }

---

Citation:

---

Envoyé par julien.63

Est qu'on pourrait imaginer une fonction pour libérer la mémoire?

---

Absolument. La prochaine étape, la structure :

Code:

---

1 2 3 4 5 6

struct tab_string { char **tab; int num; };

---

la suivante, l'ADT (TAD) :

http://emmanuel-delahaye.developpez.com/tad.htm

en utilisant une structure.
Je ne sais pas si les malloc sont bien faits? :roll:
D'autres part j'ai un souci sur le free(&tab->num); dans la fonction freememory. Ca compile bien mais j'ai une erreur à l'exécution :

Code:

---

malloc: *** Deallocation of a pointer not malloced: 0x3; This could be a double free(), or free() called with the middle of an allocated block; Try setting environment variable MallocHelp to see tools to help debug

---

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

#include <stdio.h> #include <string.h> #include <getopt.h> #include <stdlib.h>   typedef struct { char **tab; int num; } tab_string;   void freememory(tab_string *tab) { int i; for (i=0;i<=tab->num;i++) { free(tab->tab[i]); } free(&tab->num); free(tab); }   void printpattern (tab_string *tab) { int i; for (i = 0; i < tab->num; i++) { printf ("%s\n", tab->tab[i]); } }   int main (int argc, char **argv) { tab_string *tab = malloc( (sizeof(char) * (argc-1)) + 1); tab->num = 0; tab->tab = malloc(sizeof(char)*(argc-1));   while (getopt (argc, argv, "a:b:c:") != -1) { tab->tab[tab->num] = strdup (optarg); tab->num++; } printpattern (tab); freememory (tab); return 0; }

---

normal ..

tab.num est un ENTIER, pas un pointeur..

Quand tu libères tab il libère automatiquement la place de l'entier, pusiqu'il est DANS la structure.

Les free ne doivent correspondre qu'aux malloc/calloc correspondants..

Citation:

---

Quand tu libères tab il libère automatiquement la place de l'entier, pusiqu'il est DANS la structure.

---

ok peut être que je commence à comprendre.
Il libère num parce qu'il est DANS la structure donc, mais il faut libérer les tab->tab[i] car ils sont en dehors, la structure ne contenant que l'adresse de ces chaines qui sont elles mêmes libérées en même temps que num? :bug:

absolument..

C'est pour ça qu'il faut libérer les tab->tab[i] AVANT de libérer tab, sinon tu ne sauras plus où c'est et tu ne pourras plus atteindre cette zone..

un grand merci à tous et spécialement à Emmanuel
je vais imprimer ce post et le lirai tous les matins ;)