Discussion :

[rogerio]

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 :

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

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
/*****************************************************************************************************
*                                               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 :

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

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

la j'ai carrément un memleak

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

rogerio

[matafan]

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é.

[rogerio]

y a un probleme avec

Code :

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

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

le memleak est ici

[matafan]

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 :

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

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

Ou à la limite comme ça :

Code :

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

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

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

[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

[Emmanuel Delahaye]

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 :

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

1
2
3
4
5
6
7
 
char const *label[] = {
"accueil",
"direction",
"medical/psy",
<...>
};

dans ce cas, cet interface est correcte :

Code :

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

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

Vérification :

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

[rogerio]

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 :

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
/*****************************************************************************************************
*                                               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 :

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

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