Discussion :

[ChrisCondit]

Bonjour tout le monde.

En bref :
Je crée une classe MyString dont le constructeur prend une chaîne de caractères.

Code :

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

1
2
3
4
5
 
int main(){
	MyString test("lalala");
	test.print();
}

Question :
Combien de temps est diponible la chaîne de caractères "lalala"?
Le temps de l'appel au constructeur?
Ou tant qu'on est dans la fonction main?

Merci d'avance pour votre aide.

PS : le code:
test.cpp

Code :

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

1
2
3
4
5
6
7
 
#include "MyString.h"
 
int main(){
	MyString test("lalala");
	test.print();
}

Mystring.h

Code :

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

1
2
3
4
5
6
7
8
 
class MyString{
public :
MyString(const char* str);
void print();
private :
const char* mystr;
};

MyString.cpp

Code :

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

1
2
3
4
5
6
7
8
9
10
 
#include "MyString.h"
#include <iostream>
using namespace std;
 
MyString::MyString(const char* str) : mystr(str){}
 
void MyString::print(){
	cout<<mystr<<endl;
}

[Flow_75]

salut,

En faite ta chaine de caractère "lalala" n'est disponible que le temps de l'appel du constructeur.
Donc, là, tu assignes l'adresse de la chaine "lalala" qui sera detruite lorsque tu quitteras ton constructeur.
Il faudrait plutot faire :

Code :

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

1
2
3
4
5
6
 
MyString::MyString(const char * str)
{
     mystr = new char[strlen(str)+1];
     strcpy(mystr, str);
}

[ChrisCondit]

Ok j'ai compris.
Merci pour la réponse rapide et efficace Flob91 !!!

[gbdivers]

Bonjour

Sans être fausse, la solution de Flob91 me semble pas être ce qu'il y a de mieux en général (c'est du code style C et non C++, ce qui n'est pas forcement une bonne habitude à prendre)

Il est préférable d'utiliser des chaines C++ : std::string (voir la FAQ)

Code :

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

1
2
3
MyString::MyString(const std::string & str) : mystr(str)
{
}