DaniWeb Forum Index > Software Development > C

C RSS

error in copy constructor

Thread Solved

•

Join Date: Jun 2005

Posts: 23

Reputation: alone2005 is an unknown quantity at this point

Solved Threads: 0

alone2005

Offline

Newbie Poster

error in copy constructor

Jul 12th, 2005

Potion of my code:

 Help with Code Tags 
 C Syntax (Toggle Plain Text) 
class UserCommand{
	string cHandle;			//definition of command
	int numPara;
	vector<int> cPara;		//parameter of command
public:
	UserCommand(string ucData);
	//copy constructor
	UserCommand(const UserCommand& uc);
 
	inline const string getHandle(){return cHandle;};
	inline const int getPara(int i){return cPara.at(i);};
	inline const int getNumPara(){return numPara;};
 
	//overload operator = here
   	inline UserCommand operator=(UserCommand& uc); 
}; //
 
//...
 
UserCommand::UserCommand(const UserCommand& uc){ 
	cHandle=uc.getHandle();
	numPara=uc.getNumPara();
	for(int i=0;i<numPara;i++) cPara[i]=uc.getPara(i);
}
class UserCommand{
	string cHandle;			//definition of command
	int numPara;
	vector<int> cPara;		//parameter of command
public:
	UserCommand(string ucData);
	//copy constructor
	UserCommand(const UserCommand& uc);
	
	inline const string getHandle(){return cHandle;};
	inline const int getPara(int i){return cPara.at(i);};
	inline const int getNumPara(){return numPara;};
	
	//overload operator = here
   	inline UserCommand operator=(UserCommand& uc); 
}; //

//...

UserCommand::UserCommand(const UserCommand& uc){ 
	cHandle=uc.getHandle();
	numPara=uc.getNumPara();
	for(int i=0;i<numPara;i++) cPara[i]=uc.getPara(i);
}

Pretty straighforward. But I keep got error:
error: passing `const UserCommand' as `this' argument of `std::string UserCommand::getHandle()' discards qualifiers

I am using gcc under redhat, anyone can help me on this? thanks in advance.

•

Join Date: Apr 2004

Posts: 4,335

Reputation: Dave Sinkula has a brilliant future

Solved Threads: 236

Dave Sinkula Dave Sinkula is offline

Offline

long time no c

Re: error in copy constructor

Jul 12th, 2005

Help with Code Tags

(Toggle Plain Text)

	inline const string getHandle(){return cHandle;};
	inline const int getPara(int i){return cPara.at(i);};
	inline const int getNumPara(){return numPara;};

I believe it's telling you it would prefer that the this pointer be const. (And function definitions don't end with a semicolon.)

Help with Code Tags

(Toggle Plain Text)

   inline string getHandle() const { return cHandle; }
   inline int getPara(int i) const { return cPara.at(i); }
   inline int getNumPara()   const { return numPara; }

"One of the methods used by statists to destroy capitalism consists in establishing controls that tie a given industry hand and foot, making it unable to solve its problems, then declaring that freedom has failed and stronger controls are necessary." --Ayn Rand