char*, String et strcpy

Version imprimable

Bonjour,

En c++ je veux réaliser un constructeur qui pred une chaine de caractère en entrée pour un des attributs. J'ai essayé plusieurs mais j'ai toujours plusieurs messages d'erreur quelque soit la solutions que j'essayer :

voici comment j'initialise mon objet :

Code:

tab[1]=new Sandwich(4.0,5.0,"1993 12 09");

Dans ma classe et mon constructeur :
Code:

1 2 3 4 5 6 7 Sandwich::Sandwich(float hauteur_prod,float prix_prod,char* chaine): Produit(hauteur_prod,prix_prod) { cout << "Construction d'un objet de type Sandwich \n"; // solution 1 : strcpy(date_lim_conso,chaine); //solution 2: date_lim_conso=chaine; }
J'ajoute également que j'ai essayé de mettre des const sans succès ...

question 1) Quand je mets strcpy j'ai un core dumped, mais avec la solution 2 j'ai un warning conversion from string constant to char*, mon programme marche mais j'aimerai enlever ce message d'erreur, comment faire ?
question 2) Vais je pouvoir utiliser les autres fonctions de la bibliothèque String?

Merci d'avance

Lucile

22/12/2014, 02h19
JolyLoic

Citation:

Envoyé par foetus

[*]Toujours terminer ces chaînes par '\0': "1993 12 09\0". Cela peut aussi venir de là ton problème.

Un littéral de chaîne est déjà fini par 0, pas la peine d'en mettre un second...

Sinon, même si je suis d'accord qu'une chaîne de caractères n'est probablement pas idéale ici, si on voulait en faire une, il faudrait oublier ces char*, et utiliser std::string à la place !

http://cpp.developpez.com/faq/cpp/?p...s-en-Cplusplus

Citation:

Envoyé par JolyLoic

Un littéral de chaîne est déjà fini par 0, pas la peine d'en mettre un second...

Sinon, même si je suis d'accord qu'une chaîne de caractères n'est probablement pas idéale ici, si on voulait en faire une, il faudrait oublier ces char*, et utiliser std::string à la place !

http://cpp.developpez.com/faq/cpp/?p...s-en-Cplusplus

Question 1) j'ai tout fait en char* : tout a compilé mais valgrind affiche une erreur :
j'ai idetnifié ou cela posait problème c'est dans une de mes fonctions :

Code:

cout << "date limite de consommation : " << date_lim_conso << "\n" ;

avec mon attribut date_im_conso qui est déclaré : date_lim_conso[10]
Pourquoi cela pose t il problèem au niveau de valgrind :
ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)

ce n'est pas une fuite mémoire alors qu'est ce que c'est ?
Question 2) J'ai essayé de passer tout en string
mon attribut :

Code:

string date_lim_conso;

mon constructeur :
Code:

1 2 3 4 Sandwich::Sandwich(float hauteur_prod,float prix_prod,const char* chaine): Produit(hauteur_prod,prix_prod) { cout << "Construction d'un objet de type Sandwich \n"; string date_lim_conso(chaine); }
Mon soucis se pose au niveau de la construction de mon date_lim_conso, pour la recopie, comment faire, je suis completement perdue avec les string.
De plus comment j'utilise strncpy sur des string, sachant qu'elle prend des chr*

23/12/2014, 16h38
r0d

Bonjour,

Citation:

Envoyé par superlucile

J'ai essayé de passer tout en string

C'est la bonne solution.

Citation:

Envoyé par superlucile

mon attribut :

Code:

string date_lim_conso;

En C++, on ne parle pas d'attributs, mais de variables membres.
Idem pour les méthodes: on ne parle pas de méthode, mais de fonction membre.
Je sais ça peut paraître stupide d'avoir une terminologie différente d'un langage à l'autre, mais en fait cela cache de réelles et importantes différences.

Citation:

Envoyé par superlucile

mon constructeur : Sandwich::Sandwich(float hauteur_prod,float prix_prod,const char* chaine)

Pourquoi ne pas aller jusqu'au bout, et faire plutôt: Sandwich::Sandwich(float hauteur_prod,float prix_prod,const string & chaine)?

Citation:

Envoyé par superlucile

De plus comment j'utilise strncpy sur des string, sachant qu'elle prend des chr*

Il ne faut pas utiliser les fonctions "C legacy" comme strcpy, strcomp, etc,. La classe string propose tout un tas de constructeurs, de fonctions membres et d'opérateurs (sans parler de <algorithm>) qui te permettent de faire tout ça.
Par exemple pour copier une chaine, plus besoin de passer par strcpy, il suffit de faire chaine1 = chaine2;

Quand tu écris string date_lim_conso(chaine); dans ton constructeur, tu définis une nouvelle variable, et donc, tu ne modifies plus ta donnée membre. Fais plutôt :
Code:

1 2 3 4 5 6 Sandwich::Sandwich(float hauteur_prod,float prix_prod,const char* chaine): Produit(hauteur_prod,prix_prod), date_lim_conso(chaine) { cout << "Construction d'un objet de type Sandwich \n"; }
Sinon, je suis d'accord avec rOd, passe en string jusqu'au bout !

Citation:

Envoyé par r0d

Pourquoi ne pas aller jusqu'au bout, et faire plutôt: Sandwich::Sandwich(float hauteur_prod,float prix_prod,const string & chaine)?

Merci de votre aide à tous !! J'ai reussi en string, mais j'ai deux autres questions :

voilà ce que m'affiche valgrind :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 ==3310== HEAP SUMMARY: ==3310== in use at exit: 46 bytes in 2 blocks ==3310== total heap usage: 7 allocs, 5 frees, 126 bytes allocated ==3310== ==3310== 46 bytes in 2 blocks are definitely lost in loss record 1 of 1 ==3310== at 0x402B9B4: operator new(unsigned int) (in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so) ==3310== by 0x40D57D3: std::string::_Rep::_S_create(unsigned int, unsigned int, std::allocator<char> const&) (in /usr/lib/i386-linux-gnu/libstdc++.so.6.0.16) ==3310== by 0x40D7A47: char* std::string::_S_construct<char const*>(char const*, char const*, std::allocator<char> const&, std::forward_iterator_tag) (in /usr/lib/i386-linux-gnu/libstdc++.so.6.0.16) ==3310== by 0x40D7BB5: std::basic_string<char, std::char_traits<char>, std::allocator<char> >::basic_string(char const*, std::allocator<char> const&) (in /usr/lib/i386-linux-gnu/libstdc++.so.6.0.16) ==3310== by 0x41614D2: (below main) (libc-start.c:226) ==3310== ==3310== LEAK SUMMARY: ==3310== definitely lost: 46 bytes in 2 blocks ==3310== indirectly lost: 0 bytes in 0 blocks ==3310== possibly lost: 0 bytes in 0 blocks ==3310== still reachable: 0 bytes in 0 blocks ==3310== suppressed: 0 bytes in 0 blocks ==3310== ==3310== For counts of detected and suppressed errors, rerun with: -v ==3310== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)
1) Les deux pertes sont du à mes deux objets qui utilisent des variables membres string. Et donc, pour la classe string pour éviter les fuites mémoires je dois appeler le destructeur de la classe string ? Mais à priori je n'ai pas de pointeur ici ... donc je peux pas faire de delete
2) La 2ème erreur (ERROR SUMMARY) vient de cette ligne :

Code:

cout << "date limite de consommation : " << date_lim_conso << "\n" ;

, avez vous une idée ?

24/12/2014, 14h25
Médinoc

Si je comprends bien la dernière ligne de la stack trace, il s'agit d'une variable globale.
24/12/2014, 18h08
superlucile

si c'est cela comment résoudre mon problème, car dans ma fonction c'est ma variable membre ?
24/12/2014, 21h31
LinuxUser

On peut voir tout le code, du moins la classe et le main?

voici ma classe, c'est une classe dérivée de Produit :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
Sandwich::Sandwich(float hauteur_prod,float prix_prod,const string & chaine): Produit(hauteur_prod,prix_prod) {
	cout << "Construction d'un objet de type Sandwich \n";	
	date_lim_conso=chaine;
}
 
Sandwich::~Sandwich(){
	cout << "destruction d'un objet de type Sandwich \n";	
}
 
string Sandwich::ObtenirDateLimConso()const{
	return date_lim_conso;
}
 
void Sandwich::AfficherDatelim()const{
	cout << "date limite de consommation : " << date_lim_conso << "\n" ;
}
 
void Sandwich::AfficherProd()const{
	Produit::AfficherProd();
	this->AfficherDatelim();
}
 
char Sandwich::ObtenirType()const{
	return 'S';
}

des bouts de mon main :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
int main(void){
	// Initialisation d'un tableau dynamique
	int taille=4;
	Produit** tab;
	tab=new Produit*[taille];
        tab[1]=new Sandwich(4.0,5.0,"1993 12 09");
	cout << "\n<SANDWICH> \n \n" ;
	tab[1]->AfficherProd();
 
	for(unsigned int i=0; i<taille; i++)
	{
		//boucle pour eliminer les tab[i]
		if(tab[i]!=NULL)
		{delete tab[i];
		tab[i]=NULL;}
	}
 
	//boucle pour eliminer tab
	if(tab!=NULL)
	{delete [] tab;
	tab=NULL;}

Mon souci, avec valgrind sont les fuites memoires qui s'eliminent dès que je mets en commentaire cette ligne :

Code:

date_lim_conso=chaine;

et les ERRORS qui proviennent de cette ligne

Code:

cout << "date limite de consommation : " << date_lim_conso << "\n" ;

merci d'avance

27/12/2014, 16h27
LinuxUser

Salut,

La variable date_lim_conso est membre de la classe Produit, peut-on voir sa déclaration dans Produit.h?

non date_lim_conso se trouve dans sandwich, voilà le .h :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
using namespace std;
 
class Sandwich : public Produit{
 
	private :
		// à voir comment il faut la déclarer
		string date_lim_conso;
 
	public :
		// CONSTRUCTEUR
		Sandwich(float hauteur_prod,float prix_prod,const std::string & chaine);
 
		// DESTRCUTEUR
		~Sandwich();		
 
		// ACCESSEURS
		virtual string ObtenirDateLimConso()const;
 
		// AFFICHAGE
		virtual void AfficherDatelim()const;
		virtual void AfficherProd()const;
 
		virtual char ObtenirType()const;
};