Discussion :

[inh40]

Bonjour,

Je manipule des tableau de chaînes de caractères. Voici mon code :

Code :

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

1
2
3
4
5
6
7
8
char * Nomatt[SHE_TAILLE_NOMATT];
   char * Valatt[SHE_TAILLE_VALLAT];
 
   for (int i=0; i< NB_MAX_EXTRAS; i++)
   {
      Nomatt[i] = (char *)fde_evt.get_nomatt(i).c_str();
      Valatt[i] = (char *)fde_evt.get_valatt(i).c_str();
   }

Pourtant, à la sortie de la boucle for, les chaînes de caractères dans les tableaux Nomatt et Valatt ne sont pas du tout celles retournées par les get_nomatt et get_valatt, mais ce sont des chaînes incompréhensibles.

Qu'est ce que je fais de pas bien ?

[louf99]

tu finis bien tes chaines avec des \0 ?

[inh40]

Oui, puisque c_str() rajoute tout seul le caractère \0.

[loufoque]

fde_evt.get_nomatt(i) retourne peut-être un objet temporaire, qui est détruit à la prochaine itération...

[inh40]

loufoque, c'était exactement ca. Ce qui est étrange, c'est que ce code fonctionnait sous Visual 6, mais plus sous Visual 8. J'ai corrigé ca en faisant des allocations dynamiques :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
char * Nomatt[SHE_TAILLE_NOMATT];
char * Valatt[SHE_TAILLE_VALLAT];
 
for (int i=0; i< NB_MAX_EXTRAS; i++)
{
   Nomatt[i] = new char[TAILLE_MAX];
   Valatt[i] = new char[TAILLE_MAX];
   strcpy(Nomatt[i], fde_evt.get_nomatt(i).c_str());
   strcpy(Valatt[i], fde_evt.get_valatt(i).c_str());
}

Sans bien sûr oublier les delete[].

[louf99]

Attend j'avais pas vu le cast.
Tu veux que le compilo cast un CString dans un char * ???
Ca n'ettonerais que ca se fasse comme ca ... si ?

[inh40]

c_str() renvoie un (const char *). Je le cast en (char *) pour le mettre dans mon tableau. Mais de toute facon, je ne fais plus ce cast avec la solution que j'ai trouvé.

[loufoque]

Ton code est non exception-safe et comporte fort probablement des fuites.
Fais un tableau de std::string, tout simplement....

[inh40]

Si ca ne tenait qu'a moi, j'aurai bien évidemment utilisé un tableau de string. Mais malheureusement, après je fais appel à une fonction qui attend un tableau de type (char **), et ca je ne peux pas le modifier... :/

[inh40]

Aussi, si à la fin de ma méthode, je fais un :

Code :

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

1
2
3
4
5
6
 
for(int j = 0; j < NB_MAX_EXTRAS; j++)
   {
      delete[] Nomatt[j];
      delete[] Valatt[j];
   }

pourquoi penses tu que j'aurai probablement des fuites mémoire ?

[loufoque]

Exemple simplifié :

Code :

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

1
2
3
4
T* a = new T;
T* b = new T;
delete b;
delete a;

Si le deuxième "new T" lève une exception, la mémoire ne sera jamais libérée.

[inh40]

Ok je vois, il faudrait donc que j'entoure les allocations de try/catch, avec des delete[] dans le catch, c'est ca ?

[koala01]

Salut,

Le mieux est, toujours, de laisser la gestion dynamique de la mémoire à des éléments dont on sait qu'ils ont été créés, justement, pour la prendre en charge de manière cohérente et sécurisée.

Ainsi, il n'y a rien à faire, la classe vector (ou toute autre classe de conteneur de ton choix parmi ceux de la STL) est à préférer de manière systématique aux différents tableaux "C style", surtout si tu envisage d'en gérer la taille de manière dynamique.

De la même manière la classe string est à préférer aux chaines de caractères "C style", qui ne vont pas sans poser certains problèmes.

Au final, ton code pourrait très bien ressembler à

Code :

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

1
2
3
4
5
6
7
8
9
10
 
std::vector<std::string> Nomatt;
std::vector<std::string> Valatt;
for(int i=0; i< NB_MAX_EXTRAS; i++)
{
    std::string nom=fde_evt.get_nomatt(i).c_str();
    std::string val=fde_evt.get_valatt(i).c_str();
    Nomatt.push_back(nom);
    Valatt.push_back(val);
}

et tous tes problèmes seraient résolus (nécessite l'inclusion des fichiers d'en-têtes <string> et <vector> )

[inh40]

Bonjour koala,

Comme je disais dans un post plus haut :

Si ca ne tenait qu'a moi, j'aurai bien évidemment utilisé un tableau de string. Mais malheureusement, après je fais appel à une fonction qui attend un tableau de type (char **), et ca je ne peux pas le modifier... :/

Je n'ai malheureusement pas trouvé une méthode simple pour transformer un std::vector<std::string> en (char **).

[loufoque]

Les blocs try/catch pour faire du clean-up ça pue.

Voici une solution simple et portable, bien qu'un peu sous-optimale

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
boost::scoped_array<char> Nomatt[SHE_TAILLE_NOMATT];
boost::scoped_array<char> Valatt[SHE_TAILLE_NOMATT];
 
for (int i=0; i< NB_MAX_EXTRAS; i++)
{
   const std::string& nom = fde_evt.get_nomatt(i).
   const std::string& val = fde_evt.get_valatt(i);
 
   Nomatt[i] = boost::scoped_array<char>(new char[nom.size()+1]);
   Valatt[i] = boost::scoped_array<char>(new char[val.size()+1]);
 
   memcpy(Nomatt[i].get(), nom.c_str(), nom.size()+1);
   memcpy(Valatt[i].get(), val.c_str(), val.size()+1);
}
 
char* Nomatt2[SHE_TAILLE_NOMATT];
char* Valatt2[SHE_TAILLE_VALATT];
 
for (int i=0; i< NB_MAX_EXTRAS; i++)
{
    Nomatt2[i] = Nomatt[i].get();
    Valatt2[i] = Valatt[i].get();
}

J'ai aussi corrigé cette histoire de TAILLE_MAX.

Pour que le code soit valide, il faut bien évidemment que NB_MAX_EXTRAS soit inférieur ou égal à SHE_TAILLE_*. (du coup, pourquoi ne pas utiliser directement NB_MAX_EXTRAS ?)

Pour aller plus vite tu peux
- Ne pas utiliser new[] qui initialise la mémoire à 0 mais un allocateur mémoire
- Supposer que scoped_array<char> et char* sont compatibles binairement.

[inh40]

Merci pour toutes ces précisions !