Discussion :

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

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

[Franck.H]

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 :

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

printpattern(*pattern,p);

Fait plutôt:

Code :

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

printpattern(pattern,p);

[julien.63]

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

[souviron34]

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

[Franck.H]

ok, ça marche.
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 !

[Emmanuel Delahaye]

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 :

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

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

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

ça donne :

Code :

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

1
2
3
4
5
6
7
 
-s10
-d
-a12
-c15
 
Press ENTER to continue.

[julien.63]

voici une version avec getopt

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

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

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

[Emmanuel Delahaye]

voici une version avec getopt

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

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

[julien.63]

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

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

[Emmanuel Delahaye]

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

Absolument. La prochaine étape, la structure :

Code :

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

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

[julien.63]

en utilisant une structure.
Je ne sais pas si les malloc sont bien faits?
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 :

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

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 :

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

[souviron34]

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

[julien.63]

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?

[souviron34]

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

[julien.63]

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