Discussion :

[mahboub]

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

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

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
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 :

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
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;
 
	}
 
 
}

[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 :

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

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 :

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

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

[mahboub]

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

Code :

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

 const char*

pour

Code :

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

strcat(..)

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

caster

Code :

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

char* vers const char*

dans

Code :

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

 const char* getChaine(){return machaine;}

et ne peut le faire pour directement pour

Code :

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

strcat(..)

Merci

[3DArchi]

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

Code :

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

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 :

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

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 :

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

char* getChaine()const{return machaine;}

[Médinoc]

Ta fonction operator+ devrait s'appeler operator+=.

[mahboub]

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

Code :

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

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 :

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

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 :

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

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

[Trunks]

Code :

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

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.

[3DArchi]

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 :

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

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 :

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
 
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 :

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

char * getChaine()

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

Code :

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

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 :

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

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 :

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

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 :

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

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 :

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

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).