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

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

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

[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