Discussion :

[Yoshio]

Bonjour,

J'ai fais une fonction qui manipule un tableau de pointeur de chaine de caractère.

Donc je passe en argument de la fonction ce fameux tableaux et je le modifie dans le fonction. Une partie du 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
29
30
31
32
33
int pregMatch(const char *pattern, const char *request, char **result)
{
        ...
	if ((result = malloc(sizeof(regmatch_t) * numMatch)) == NULL)
	{
		return EXIT_FAILURE;
	}
        ...
 
	if (...)
	{
	}
	else
	{
		for (i = 0; i < numMatch; i++)
	    {
			....
			if ((result[i] = malloc(length)) == NULL)
			{
				return EXIT_FAILURE;
			}
                        ...
		}
	}
 
        /* Ce code affiche bien les chaine de caractères que je veux */
	for (i = 0; i < numMatch; i++)
        {
		printf("%s\n", result[i]);
	}
 
	return numMatch;
}

L'appel de ma fonction :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
         char *result = NULL
 
         if((rc = pregMatch(pattern, string, &result)) == NOK)
	{
		...
	}
 
	for (i = 0; i < rc; i++)
	{
                /* Pas moyen de récupérer les chaine de caractère ici */
		printf("%d : %s\n", i, ????result??????);
	}

Je n'arrive pas a récupéré les chaine de caractère en dehors de la fonction alors que dedans ca marche nikel. Que mettre à la place des ????????? (j'ai tout essayé mais j'ai pas trouvé ). A chaque fois j'ai une erreur de segmentation quand j'affiche les chaine de caractère en dehors de ma fonction.



Yoshio

[nicolas.sitbon]

J'ai fais une fonction qui manipule un tableau de pointeur de chaine de caractère.

Tu voulais dire un tableau de chaine ou un pointeur de chaine je suppose?

[ram-0000]

Je pense que c'est plutôt cela qu'il faut faire lors du malloc

Code :

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

1
2
3
4
 
...
if ((*result = malloc(sizeof(regmatch_t) * numMatch)) == NULL)
...

Et même je le ferait en 2 lignes pour plus de clarté

Code :

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

1
2
3
4
5
 
...
*result = malloc(sizeof(regmatch_t) * numMatch));
if (*result == NULL)
...

[Thierry Chappuis]

Je vérifierais même au préalable que result ne vaut pas NULL avant de déréférencer:

Code :

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

1
2
3
4
5
6
7
8
9
10
/* -tc- validation des arguments */
if (result != NULL)
{
    /*...*/
    *result = malloc(sizeof(regmatch_t) * numMatch));
    if (*result == NULL)
    {
        /* ... */
    }
}

Thierry

[JeromeBcx]

C'est un tableau dynamique de chaine de caractères ou seulement une chaîne de caractère ?

[Yoshio]

C'est un tableau dynamique de chaine de caractères ou seulement une chaîne de caractère ?

J'ai un tableau de pointeur et les pointeur pointe sur une chaine de caractères. (Enfin du moins c'est ce que j'ai essayé de faire )

Merci pour ta solution ram_0000, ca marche un ptit peu mieux

Pour lire les chaine de caractère en dehors de la fonction vous feriez quoi ?
result[i] ca passe pas, result + i c'est pas bon non plus, ...

[JeromeBcx]

Pour lire les chaine de caractère en dehors de la fonction vous feriez quoi ?
result[i] ca passe pas, result + i c'est pas bon non plus, ...

Dans ton cas, tu ne devrais pas avoir de différences entre inter et extra fonction.

Ce n'est qu'une proposition mais je déclarerais

Code :

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

char** result = NULL; //Un tableau dynamique d'une chaine de caractères dynamique.

Pour l'appel à ta fonction

Code :

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

if((rc = pregMatch(pattern, string, result)) == NOK)

Pour l'allocation d'un tableau de chaine en C, je ne me souviens pas de la syntaxe mais de mémoire, il faut:
Allouer le tableau de pointeurs, puis pour chaque pointeurs, allouer la mémoire des chaînes de caractères.

Enfin pour l'affichage, la méthode sera la même en inter/extra du type

Code :

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

print("%s",result[i]);

Bon courage

[Médinoc]

Les notes d'Emmanuel peuvent utiles : http://emmanuel-delahaye.developpez....difie_variable
Même si sur ce chapitre, c'est quand même un poil trop succint...

[diogene]

Si tu veux fabriquer dynamiquement un tableau de chaînes de caractères, le type de la variable result doit être char** (du coté appel)

Code :

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

char** result = NULL;

La fonction pregMatch construit ce tableau, donc doit être apte à modifier cette variable result. Il faut donc passer son adresse :

Code :

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

1
2
3
int pregMatch(const char *pattern, const char *request, char *** result)
{
  if ((*result = malloc(sizeof(regmatch_t) * numMatch)) == NULL)

(Là, c'est curieux, car tu devrais avoir sizeof(char*) dans le malloc)

Pour créer les chaînes du tableau:

Code :

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

if (((*result)[i] = malloc(length)) == NULL)

(*result doit être mis entre parenthèses)

et appeler la fonction par

Code :

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

if((rc = pregMatch(pattern, string, &result)) == NOK)....

Pour sortir les chaînes (du coté appel) :

Code :

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

print("%s",result[i]);