Discussion :

[beekeep]

Salut à tous,


je rencontre des difficulté pour allouer la mémoire de tableaux de chaines de caractères.

les tableaux sont des variables globales déclarées comme cela :

Code :

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

char * oTabMsg[];

ensuite j'alloue la mémoire comme ceci :

Code :

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

*oTabMsg = calloc(iNbMsg, 512*sizeof(char));

mais je pense que l'allocation ne doit se faire que sur le premier élément du tableau,
car quand j'utilise mon tableau (initialisation avec strcpy), ça plante lors de la deuxième itération. (tab[1])

avec un joli Segmentation Fault ..

Merci d'avance pour votre aide

[Médinoc]

Code :

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

char * oTabMsg[];

Ça compile, ça ?


Ensuite, je pense que tu mettre le résultat de ton malloc() dans une case d'un "tableau" qui est en réalité un pointeur non-initialisé...

[beekeep]

Code :

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

char * oTabMsg[];

Ça compile, ça ?

petit warning c'est vrai, mais c'est pas grave.

Ensuite, je pense que tu mettre le résultat de ton malloc() dans une case d'un "tableau" qui est en réalité un pointeur non-initialisé...

(j'utilise la fonction calloc)

j'ai pas très bien compris,
il faudrait que je m'y prenne autrement ?

[beekeep]

c'est bien ce que je pensai,

lorsque je fait :

Code :

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

1
2
*oTabMsg = malloc(512*sizeof(char));
*(oTabMsg+1) = malloc(512*sizeof(char));

j'ai bien une itération de plus qui fonctionne.

donc ma question est : comment utiliser calloc pour allouer mon tableau entier ?
(en spécifiant le nombre d'éléments bien sur)

[Médinoc]

Commence par faire oTabMsg = malloc( /*une taille*/ );, car j'ai bien l'impression que oTabMsg n'est pas un vrai tableau (ou bien, est un tableau de taille nulle, ce qu'on verra si le oTabMsg= ne compile pas.)

PS: Quel est le warning affiché ?

Au passage: Quand on me dit qu'un warning est "pas grave", c'est faux dans 90% des cas.

[beekeep]

en effet, le code suivant ne compile pas
(c'est ce que j'avais voulu faire au tout début)

Code :

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

oTabMsg = malloc(512*sizeof(char));

mais en faisant une boucle de malloc, ça fonctionne très bien :

Code :

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

1
2
3
4
  for(i=0; i<= iNbMsg; i++)
  {
    *(oTabMsg +i) = malloc( 512*sizeof(char));
  }

ça serait pas plus propre avec un calloc ?

Au passage: Quand on me dit qu'un warning est "pas grave", c'est faux dans 90% des cas.

je veux bien te croire !!

warning: null dimension: oTabMsg

[Vincent Rogier]

Plutôt quelque chose du genre

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
 
char **tab_str;
int i, tab_size;
 
/* alloue le tableau */
 
tab_size = 100;
 
tab_str = malloc (tab_size * sizeof(char *) )
 
/* test allocation */
 
/* ... */
 
/* alloue chaque entrée du tableau */
 
for (i = 0; i <  tab_size; i++)
{
    tab_str[i] = malloc (512 * sizeof(char) );
}

ça serait pas plus propre avec un calloc ?

Ca dépend ce que tu comptes en faire, de la taille des allocations, car calloc() a un coût ... et parfois n'est pas nécessaire..

C'est pas plus propre mais plus sur..

Dans le cas de chaines : un malloc() suivi d'un chaine[0] = 0 suffit !