Discussion :

[jamesandrew]

Bonjour,
Je souhaite créer une liste de chaînes de caractères de type vector<char*> en C++ qui puisse accepter de nouvelles chaînes issues d'une variable de type string grâce à une fonction. J'ai fait un code sans fonction qui marche très bien :

Code :

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

1
2
3
4
5
6
7
8
 
void main()
{
vector<char*> listest(0); 
string toto = "Hello";
listest.push_back(const_cast<char*>(toto.c_str())); // J'ai dû rajouter const_cast<char*> pour passer en type char* car la méthode push_back n'acceptait pas toto.c_str() qui est de type const char*.
cout << listest[0] << endl; 
}

En revanche, si je crée une fonction qui modifie directement listest (passage par référence), cela ne marche plus :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
void fct_test(vector<char*> *listref)
{
	 string toto = "Hello";
	(*listref).push_back(const_cast<char*>(toto.c_str()));
}
void main()
{
        vector<char*> listest(0);
	fct_test(&listest);
	cout << listest[0] << endl;
}

Quelqu'un saurait-il pourquoi ?
Je me suis demandé si cela pouvait venir du passage par référence dans la fonction, mais si je fais le test suivant, cela fonctionne très bien :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
void fct_test(vector<char*> *listref)
{
	(*listref).push_back("Hello");
}
void main()
{
        vector<char*> listest(0);
	fct_test(&listest);
	cout << listest[0] << endl;
}

Le problème vient donc de la ligne

Code :

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

(*listref).push_back(const_cast<char*>(toto.c_str()))

dans la fonction, mais cette ligne fonctionne très bien dans le main(), donc je ne comprends pas.

Merci de votre aide,
Andrew

[jo_link_noir]

Ces codes ont au minimum 3 problèmes:

- Un usage abusif de pointeur là le bon sens voudrait une référence.
- Vouloir manipuler des chaînes de caractères avec char*. Surtout quand l'origine est un std::string.
- Caster le retour de c_str().

Si tu veux comprendre ton erreur, relis la doc sur std::string::c_str(), c'est une valeur temporaire est contrôlé par std::string. Quand std::string est modifié, ce pointeur ne doit plus être utilisé.
La destruction d'un std::string est un changement d'état.

[jamesandrew]

Merci pour ta réponse.

- Un usage abusif de pointeur là le bon sens voudrait une référence

Je pensais que pointeur et référence voulaient dire la même chose. Dans le cas présent, mon bon sens me disait de passer des adresses en paramètres d'une fonction plutôt que la variable. Et de ce que j'ai lu, un pointeur est une adresse. Je vais revoir ce point.

- Vouloir manipuler des chaînes de caractères avec char*. Surtout quand l'origine est un std::string

En amont, je reçois un std::string, cela je ne peux rien y changer. En revanche, pour stocker les chaînes de caractères, je préférais utiliser un tableau de char* plutôt qu'un tableau de string car je pensais que c'était moins couteux en mémoire. Qu'en penses-tu ?

Concernant std::string::c_str(), j'ai lu qu'il s'agissait d'un pointeur vers la chaîne de caractères correspondant au string déclaré avec le caractère nul à la fin. Tu me dis qu'il ne faut plus utiliser ce pointeur quand std::string est modifié. Dans mon cas, puisqu'on est dans une fonction, je pense qu'à la fin de son exécution, le string toto est détruit. Mais j'accède au pointeur avant la destruction du string toto donc je ne comprends pas ?

[Pyramidev]

En revanche, pour stocker les chaînes de caractères, je préférais utiliser un tableau de char* plutôt qu'un tableau de string car je pensais que c'était moins couteux en mémoire.

Dans ce cas, au moins, remplace vector<char*> par vector<const char*> et supprime les const_cast<char*>.

char*, et donc aussi vector<char*>, permet de modifier le contenu d'une chaîne.
Mais si le programme a stocké la chaîne dans une zone mémoire accessible uniquement en lecture, alors tenter de la modifier fera probablement crasher le programme.

Hélas, en C++98, les constantes littérales comme "Hello" sont convertibles en char*, par rétrocompatibilité avec le C. Mais cette anomalie est corrigée en C++11. Ainsi, le code suivant ne compilera plus en C++11 :

Code :

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

1
2
3
4
void fct_test(vector<char*> *listref)
{
	(*listref).push_back("Hello");
}

Dans mon cas, puisqu'on est dans une fonction, je pense qu'à la fin de son exécution, le string toto est détruit. Mais j'accède au pointeur avant la destruction du string toto donc je ne comprends pas ?

Lors de la destruction de toto, la mémoire contenant la chaîne de caractères pointée par toto.c_str() est libérée.
Le fait d'avoir copié quelque part ailleurs l'adresse vers cette chaîne de caractères n'empêche pas cette libération.

Si tu veux persister à utiliser std::vector<const char*> :

• Tu peux faire push_back sur une constante littérale, par exemple myVector.push_back("Hello"). Ce n'est pas dangereux car ces chaînes sont allouées pour toute la durée de l'exécution du programme.
• Si tu fais myVector.push_back(objetStdString.c_str()), ça va être sportif. Je conseille alors que objetStdString soit de type const std::string. Comme ça, tu n'auras plus à te soucier que objetStdString ne soit pas modifié en cours de vie, mais "seulement" qu'il ne soit pas détruit trop tôt.

Cependant, à moins que la performance soit un critère crucial de ton programme, je conseille vivement de préférer std::vector<std::string>. La chaîne sera copiée à chaque insertion* mais, au moins, il n'y aura plus de risque de pointer vers une chaîne détruite.

* : Sauf si l'implémentation de std::string utilise le Copy On Write, mais c'est une autre histoire.

[jo_link_noir]

* : Sauf si l'implémentation de std::string utilise le Copy On Write, mais c'est une autre histoire.

Ou que le constructeur de déplacement soit utilisé. Surtout que si je comprends bien, la chaîne existe toujours et va être stockée dans le vecteur quoi qu'il arrive. Il y a tout intérêt à utiliser un vecteur de std::string et std::move dans cette situation.

[jamesandrew]

C'est bon j'ai compris mes erreurs sur les pointeurs, j’utilise maintenant un vecteur de string et ça marche très bien. Merci beaucoup