Copy constructor error, 'this' pointer conversion

Making a copy constructor, I am getting a problem.

Code:

---

CPost::CPost(const CPost& rhs)
{
        rhs.GetMessageLength(); //error on this line
}

---

Quote:

---

error C2662: 'GetMessageLength' : cannot convert 'this' pointer from 'const class CPost' to 'class CPost &'

---

Some discussions I found about this recommended taking away the const, but when I do that, MS VC++ 6.0 wont recognize it as a copy constructor. Is anyone aware of a solution to this problem?

Try to make the function const aswell, like

Code:

---

class c
{
public:
    c(const c& rhs);
    int GetSomething() const { return something; }
private:
    int something;
};

c::c(const c& rhs)
{
    rhs.GetSomething();
}

---

Not sure if that is it but according to msdn this should solve your problem.

That worked, thanks. But what if the function I want to call CANNOT be const, ie, it modifies the object?

Then you cant send the object as a const. At least this is how I think it is.

>But what if the function I want to call CANNOT be const, ie, it modifies the object?
Look up the mutable keyword. I can't guarantee that this is what you want since I haven't seen your code, but it's likely.

If the function that cannot be const is logically const - meaning it might change the value of some private member variables but the logical state of the object is unchanged - then like Prelude suggested you can use the mutable keyword on the variables that might change and still mark the function as const.

However, if the function that cannot be const actually changes the state of the object, then you shouldn't be using it inside the copy constructor. It doesn't make sense to modify the argument passed into the constructor. That is why the convention is to use a const reference parameter.

Quote:

---

then you shouldn't be using it inside the copy constructor. It doesn't make sense to modify the argument passed into the constructor.

---

:p I never thought about it that way. Cheers:D .