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 !

[Médinoc]

Tu n'as pas compris.
Ce ne sont pas les différents pointeurs du tableau qui ne sont pas initialisés.

C'est le fait que le tableau lui-même ne contienne PAS DE POINTEUR! IL A UNE DIMENSION ZÉRO! IL N'Y A PAS DE PLACE POUR DES POINTEURS DANS CE TABLEAU!

Tu as compris, maintenant ?

[beekeep]

Tu n'as pas compris.
Ce ne sont pas les différents pointeurs du tableau qui ne sont pas initialisés.

C'est le fait que le tableau lui-même ne contienne PAS DE POINTEUR! IL A UNE DIMENSION ZÉRO! IL N'Y A PAS DE PLACE POUR DES POINTEURS DANS CE TABLEAU!

Tu as compris, maintenant ?

oui, ça y est.

comment je peux faire pour lui donner la dimension voulue ?
comme le code de vicenzo ? (merci au passage)

[homeostasie]

comment je peux faire pour lui donner la dimension voulue ?

Par exemple:

Code :

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

1
2
3
 
#define TAB_SIZE 10
char * oTabMsg[TAB_SIZE];

[beekeep]

homeostasie,

je ne peux pas faire comme ça si je ne connais pas la dimension du tableau lors de sa déclaration.

je voudrai avoir des tableaux à taille variable.

Merci à tous pour votre aide

[homeostasie]

homeostasie,

je ne peux pas faire comme ça si je ne connais pas la dimension du tableau lors de sa déclaration.

je voudrai avoir des tableaux à taille variable.

Merci à tous pour votre aide

Alors, tu dois opter pour le code de vicenzo.

Au passage:
Comment allouer dynamiquement un tableau à 2 dimensions ?

[beekeep]

Au passage:
Comment allouer dynamiquement un tableau à 2 dimensions ?

le pire c'est que j'ai squatté la FAQ avant de poster ..

j'ai fait mon boulet mais il faut dire que le retour du c# vers le c n'est vraiment pas évident !

aller, encore Merci