erreur de compilation

Version imprimable

bonjour a vous,
je suis encore avec ma class chaine
je ne comprends pas pourquoi quand je met dans l'operator+
const chaine &
j'ai une erreur de compilation sans le const tout fonctionne

Citation:

1>chaine.cpp(30) : error C2662: 'chaine::getChaine' : cannot convert 'this' pointer from 'const chaine' to 'chaine &'

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
26
27
28
 
#ifndef _chaine_
#define _chaine_
#include <iostream>
#include <cstring>
 
using namespace std;
class chaine
{
public:
 
	chaine();
	chaine(char c);
	chaine (char * );
	chaine& operator+(const chaine&);
	char * getChaine(){return machaine ;}
	int getSize(){return size;}
private :
 
	char * machaine;
	int size;
 
};
 
 
 
 
#endif

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
26
27
28
29
30
31
32
33
34
35
36
37
#include "chaine.h"
 
chaine::chaine()
{
	machaine = new char [1]; 
	strcpy(machaine ,"\0");
	size =0;
}
 
chaine::chaine(char * str)
{
	if(str)
	{
		size= strlen(str);
		machaine = new char[size+1];
		machaine = strcpy(machaine,str);
	}
	else
		chaine();
 
}
chaine &chaine::operator +( const chaine & str)//const
{
	if (machaine)
	{
		char * temp = new char[size+= str.size+1];
 
		temp = strcpy(temp,machaine );
		delete [] machaine;
		machaine = strcat(temp,str.getChaine());
 
		return *this;
 
	}
 
 
}

11/03/2009, 22h38
Arzar
Bonsoir,
Il faut avouer que le message d'erreur n'est pas très explicite.
Par contre, le prototype de strcat est sans appel:

Code:

char* strcat(char* _Dest, const char* _Source);

getChaine() doit donc être une fonction constante. Ceci dit, autant implémenter les deux versions, constante et non-constante, comme ça le compilateur choisira automatiquement la bonne.
Code:

1 2 char * getChaine(){return machaine ;} const char* getChaine()const{return machaine;}
11/03/2009, 23h43
mahboub

Merci pour ta reponse,
je vois ce que tu veux dire par getChaine doit renvoyer

Code:

const char*

pour

Code:

strcat(..)

cependant il y a un truc que je ne comprends pas truc pourquoi le compilateur peut

caster

Code:

char* vers const char*

dans

Code:

const char* getChaine(){return machaine;}

et ne peut le faire pour directement pour

Code:

strcat(..)

Merci
12/03/2009, 10h23
3DArchi

Ton problème n'est pas qu'il n'arrive pas à convertir de char* vers const char*.
Dans ta méthode :

Code:

chaine &chaine::operator +( const chaine & str)

str est qualifié de const. Donc tu ne peux le modifier ni appeler d'autre méthode de str qui ne soit pas elle même const. D'où le besoin de faire un accesseur en const (comme montré par Arzar) :

Code:

const char* getChaine()const{return machaine;}

L'important ici étant le second const, qui s'applique à la méthode et pas celui de retour. Tu aurais bien pu écrire (quoi que conceptuellement ce serait une erreur) :

Code:

char* getChaine()const{return machaine;}
12/03/2009, 11h03
Médinoc

Ta fonction operator+ devrait s'appeler operator+=.
12/03/2009, 16h26
mahboub

Citation:

Envoyé par 3DArchi

Ton problème n'est pas qu'il n'arrive pas à convertir de char* vers const char*.
Dans ta méthode :

Code:

chaine &chaine::operator +( const chaine & str)

str est qualifié de const. Donc tu ne peux le modifier ni appeler d'autre méthode de str qui ne soit pas elle même const. D'où le besoin de faire un accesseur en const (comme montré par Arzar) :

Code:

const char* getChaine()const{return machaine;}

L'important ici étant le second const, qui s'applique à la méthode et pas celui de retour. Tu aurais bien pu écrire (quoi que conceptuellement ce serait une erreur) :

Code:

char* getChaine()const{return machaine;}

merci pour ta reponse.
j'ai encore des questions :) je croyais que lorsqu'on declare dans une fonction le parametre comme constant la fonction c'est pour proteger les données contenues dans ce parametre et il ne peut etre modifier par cette fonction.
d'apres ce que tu viens de me dire lorsque c'est un objet il ne doit appeler que des methodes membre constantes.
si je comprends bien le probleme est plus general et ne concerne pas strcat donc
merci de m'eclairecire est ce que ce que vous dite a une relation qu'on est dans la declaration d'une classe
12/03/2009, 16h58
Trunks

Code:

void MyClass::foo(const Bar& bar) const;

Le 1er const, en tant que paramètre, permet d'éviter de modifier l'objet en paramètre (ici l'objet bar) tandis que le const de fin fin s'applique à la fonction. Il évite de modifier un membre de la class MyClass.

Citation:

Envoyé par mahboub

merci pour ta reponse.
j'ai encore des questions :) je croyais que lorsqu'on declare dans une fonction le parametre comme constant la fonction c'est pour proteger les données contenues dans ce parametre et il ne peut etre modifier par cette fonction.
d'apres ce que tu viens de me dire lorsque c'est un objet il ne doit appeler que des methodes membre constantes.
si je comprends bien le probleme est plus general et ne concerne pas strcat donc
merci de m'eclairecire est ce que ce que vous dite a une relation qu'on est dans la declaration d'une classe

Comme l'a souligné Trunks, ce n'est pas tout à fait ce que j'ai voulu dire.
Si on reprend ta fonction :

Code:

chaine &chaine::operator +( const chaine & str)

Le const s'applique au paramètre str et pas à this. Donc, pour str, tu ne peux appeler que des fonctions const et pour this tu peux appeler des fonctions const ou non const.
D'ailleurs, tu pourrais faire :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 chaine &chaine::operator +( const chaine & str) { if (machaine) { char * temp = new char[size+= str.size+1]; temp = strcpy(temp,machaine ); delete [] machaine; machaine = temp; machaine = strcat( getChaine(), //(1) str.getChaine()); //(2) return *this; } return *this; }
Dans le cas (1), c'est la version non const qui est appelée sur l'objet this:

Code:

char * getChaine()

Dans le cas (2), c'est la version const qui es appelée sur l'objet str :

Code:

const char * getChaine()const

Pendant que j'y suis, quelque remarques sur ton code :
1/Ne pas utiliser using namespace dans le .h. Tu peux le mettre dans le cpp, mais il quand même préférable de garder l'utilisation complète du namespace (std:: n'est pas trop long à écrire).
2/ Tu n'as pas de destructeur pour libérer machaine
3/ Tes allocations peuvent échouer. Et tu ne gère pas cet échec.
4/ Dans ce constructeur :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 chaine::chaine(char * str) //(1) { if(str) { size= strlen(str); machaine = new char[size+1]; machaine = strcpy(machaine,str); } else chaine();//(2) }
(1) : tu pourrais passer un const char * au lieu d'un char * :
Code:

1 2 chaine::chaine(const char * str)
(2) : ceci n'appelle pas le constructeur par défaut sur this !!!!!! Cela crée une variable non nommée de type chaine. C'est équivalent à :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 chaine::chaine(char * str) { if(str) { size= strlen(str); machaine = new char[size+1]; machaine = strcpy(machaine,str); } else { chaine une_chaine_qui_ne_sert_a_rien;//(2) } }
5/ Dans ta surcharge de l'opérateur + :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 chaine &chaine::operator +( const chaine & str)//const { if (machaine) { char * temp = new char[size+= str.size+1]; temp = strcpy(temp,machaine ); delete [] machaine; machaine = strcat(temp,str.getChaine()); return *this; } // Et ici, on ne retourne rien ????? }
6/ Puisque tu as un pointeur (machaine), tu dois soit utiliser un pointeur intelligent, soit redéfinir correctement les constructeurs par copie et l'opérateur d'affectation.
Comme je suppose que tout ceci n'a comme but que d'apprendre, peut-être peux-tu essayer les deux solutions : 1) redéfinir les constructeurs par copie et l'opérateur par défaut pour comprendre les problèmes qui se cachent derrière, puis utiliser un pointeur intelligent (car on ne va pas sans cesse ré-inventer la roue).

12/03/2009, 17h58
mahboub

Citation:

Envoyé par 3DArchi

Comme l'a souligné Trunks, ce n'est pas tout à fait ce que j'ai voulu dire.
Si on reprend ta fonction :

Code:

chaine &chaine::operator +( const chaine & str)

Le const s'applique au paramètre str et pas à this. Donc, pour str, tu ne peux appeler que des fonctions const et pour this tu peux appeler des fonctions const ou non const.

en resumé si je comprends bien ceci s'appllicque meme en dohrs des definition de classe. meme dans le main.
une fois que j'ai mis

Code:

mafonc(const & paramtre)

à l'interieur de ma fonction je ne peux modifier le parametre et l'objet ne peux appeler que des fonctions constantes (qui par definition ne doivent pas changer l'etat de l'objet) si j'appelle une methode non constant le compilateur n'a pas la garantie que l'objet n'a pas changé, même si la methode ne change pas l'objet.comme dans mon cas

Code:

getchaine() ou getsize()

.
Si par exemple je dois avoir la taille je suis obliger de declarer une version const.
merci pour vos reponse je crois que j'ai compris :):king: je sais que c'etais dure mais bon:):yaisse2:
merci pour tes commentaires. Pour le std c'est plus facile puisque je ne connais pas ce qu'il y a dans ce namespace et comme ça j'inclus le tout.
12/03/2009, 18h02
3DArchi

Citation:

Envoyé par mahboub

pour le std c'est plus facile puisque je ne connais pas ce qu'il y a dans ce namespace et comme ca j'inclue le tout

C'est une mauvaise habitude :nono:
Lorsque tu utilises quelque chose, et bien regarde dans la doc dans quel namespace il est définit.
12/03/2009, 18h07
mahboub

bonjour,
merci pour tes commentaire et exactement je veux (exercice educatif, je veux comprends la manipulation des pointeur et des chaine).
merci encore
je vais faire tes commentaire et revise mon code
12/03/2009, 18h11
3DArchi

Penses à :resolu:
Tu rouvriras une nouvelle discussion quand tu auras de nouveaux problèmes :mrgreen: