passer un tableau en argument dans une fonction

salut j'ai visiblement un probleme avec une fonction

je veux passer un tableau dans la fonction pour lister tous les elements, comparer les strlen et en extraire la taille de la cellule la plus grande

exemple de declaration de tableau

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

char label[][255] = { "accueil", "direction", "medical/psy", "cuisine", "standard", "vide", "portable/wired", "portable/wifi", "accueil", "direction", "medical/psy", "cuisine", "standard", "vide", "portable/wired", "portable/wifi", "accueil", "direction", "medical/psy", "cuisine", "standard", "vide", "portable/wired", "portable/wifi", "file server", "file server", "file server", "live box", "cups file server" };

---

l'ebauche de la fonction

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

/***************************************************************************************************** *                                               max cell size * * takes an array and return the size of the max strlen cell ******************************************************************************************************/ int max_cell_size(char ** array1, int sum){ // int i;   printf("%d\n", sum);   for(i = 0; i < sum; i++) { printf("%s - \n", array1[i]); } return 0; }

---

le passage du tableau dans la fonction

Code:

---

max_cell_size((char **)label, sum_tunnels);

---

la j'ai carrément un memleak

si quelqu'un pouvait m'aider, ce serait cool

rogerio

Tu entends quoi par "j'ai carrément un memleak" ? Tu ne peux pas avoir de fuite mémoire puisque tu n'alloue rien, en tout cas dans ce que tu nous a montré.

y a un probleme avec

Code:

---

printf("%s - \n", array1[i]);

---

le memleak est ici

Ton problème c'est que label est un char[][255], par un char **. Ca n'a rien à voir. En particulier label[1] se trouve 255 octets après label[0], alors que array1[1] se trouve sizeof (char *) après array1[0]. Ton cast dans l'appel de fonction ne sert qu'à masquer l'erreur.

Il faut déclarer max_cell_size comme ça :

Code:

---

int max_cell_size(char array1[][255], int sum)

---

Ou à la limite comme ça :

Code:

---

int max_cell_size(char (*array1)[255], int sum)

---

Edit : est ça n'a rien à voir avec un "memory leak"

je te remercie, ça fonctionne

le probleme c'est que les tableaux à passer dans la fonction n'ont pas tous des cellules de 255 mais je vais les redimentionner à la declaration

merci

rogerio

Citation:

---

Envoyé par rogerio

je te remercie, ça fonctionne

le probleme c'est que les tableaux à passer dans la fonction n'ont pas tous des cellules de 255 mais je vais les redimentionner à la declaration

merci

rogerio

---

Si les chaines sont fixes (ou allouées dynamiquement), tu peux utiliser un tableau de pointeurs :

Code:

---

1 2 3 4 5 6 7

char const *label[] = { "accueil", "direction", "medical/psy", <...> };

---

dans ce cas, cet interface est correcte :

Code:

---

1 2 3 4

int max_cell_size(char const * array1[], int sum) { <...>

---

Vérification :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

int max_cell_size (char const *array1[], int sum) { return 0; }   int main (void) { char const *label[] = { "accueil", "direction", "medical/psy", };   max_cell_size (label, 0);   return 0; }

---

je te remercie des precisions

mais cette declaration const char me permet-elle de modifier le contenu d'une cellule

je crois que je me suis tellement enervé sur cette fonction que j'ai mal conçue cette partie de mon programme

mon programme lance plink avec une serie de redirection de ports
ensuite il log un netstat et parse le log pour verifier que les redirections sont effectuées

cette derniere fonction fait partie d'une macro qui va afficher le resultat du parsing

je prend la taille du plus grand element tu tableau
ensuite à l'affichage je verifie la taille de l'element à afficher avec le resultat precedent et rempli la cellule avec des espaces

donc j'arrive à un retour console pseudo tabulé

je suis arrivé à mes fins grace au forum mais je pense que ça fait beaucoup de code pour pas grand chose

les deux fonctions :

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

/***************************************************************************************************** *                                               cell resize * * resize a cell filling it with spaces ******************************************************************************************************/ char * cell_resize(char * string, int max_size){ // int str_len = strlen(string); // while(str_len < max_size) { strcat(string, "_"); str_len ++; } return string; } /***************************************************************************************************** *                                               max cell size * * takes an array and return the size of the max strlen cell ******************************************************************************************************/ int max_cell_size(char array1[][255], int sum){ // int i, str_len, buffer = 0; // for(i = 0; i < sum; i++) { str_len = strlen(array1[i]);   if(str_len > buffer) { buffer = str_len; } } return buffer; }

---

l'output :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

if(strstr(array_net_log[j], match1[i]) != NULL && strstr(array_net_log[j], match2[i]) != NULL && strstr(array_net_log[j], "LISTENING") != NULL) { printf("%s %s %s %s:%s vers %s:%s\n", cell_resize(ssh_status[0], max_ssh_status), cell_resize(connection_type[i], max_connection_type), cell_resize(label[i], max_label), cell_resize(ssh_host_listen[i], max_ssh_host_listen), cell_resize(ssh_port_listen[i], max_ssh_port_listen), cell_resize(ssh_host_redirect[i], max_ssh_host_redirect), cell_resize(ssh_port_redirect[i], max_ssh_port_redirect)); res ++; break; }

---