Something i cannot get...

[Nati]

Hi,
Can someone explain to me the following error:
"no matching function for call to 'Student::Student(Student)'"
i got the message only in the return statement. (line 4)

Student StudentNode::getStudent(){
	Student s(*(this->student));
//	return s;
	return Student (*(this->student));
}

Here is Student:

class Student {
public:
	Student(std::string ,int ,int);
	Student(Student&); //
	std::string getName() const; //
	int getID() const; //
	int getAverage() const; //
	void setName(std::string); //
	void setID(int); //
	void setAverage(int); //
	std::string format();
	virtual ~Student();
private:
    std::string name;
    int id, average;
};

[shacktar]

Try changing

Student(Student&);

to

Student(const Student&);

[Nati]

Well, thank you very much. its work.

Can you tell me please what the differences?

[shacktar]

What you had before, i.e. Student(Student&); is a copy constructor that takes a non-const reference. This is possible to have but there is a problem when it comes to temporary objects, namely that non-const references cannot bind to them. This means that the call to return Student (*(this->student)); was looking for a function that took in a Student because the copy-constructor you had could not be used in that situation. Making the copy constructor take in a const reference allowed it to handle the *(this->student) argument correctly.

[Nati]

Thank you very much!