Discussion :

[anat1212]

Bonjour,

Avant je fesait comme ca pour allouer un tableau et le remplir :

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
 
                                char **param
char **test;
 
test = malloc(sizeof(test)*Taille tableau + 1);
+ 1 pour le dernier élément ou je mettrais l'adr 0
 
while (param[i] != 0)
{
test[i] = malloc(sizeof(char)*(strlen(param[i])+1));
+ 1 pour le \0
test[i] = param[i];
i++;
}
test[i] = 0;

Dites moi ce que je fais mal dans le malloc mais ca marchait avant, sauf que maintenant a partir du 2eme malloc je dois faire sizeof(char*) pour que j'ai ce que je voulais

pouvez vous m'expliquer pourquoi ?

[diogene]

-

Code :

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

test = malloc(sizeof(test)*Taille tableau + 1);

est incorrect :

Code :

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

test = malloc(sizeof(*test)*Taille tableau + 1);

Pour la suite, je suppose que param est correctement initialisé et que c'est un tableau de pointeurs sur des chaines de caractères.
-

Code :

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

1
2
test[i] = malloc(sizeof(char)*(strlen(param[i])+1));
test[i] = param[i];

La deuxième ligne écrase la valeur retournée par le malloc. On ne copie pas un tableau simplement avec le signe =. Accessoirement, sizeof(char) vaut toujours 1

Code :

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

1
2
test[i] = malloc(strlen(param[i])+1);
if(test[i] != NULL) strcpy(test[i] , param[i]);

-

Code :

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

test[i] = 0;

Vu que test[i] est une adresse, il est préférable d'écrire

Code :

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

test[i] = NULL;