3

为什么编译器会在指示的行抱怨？

class C
{
    std::string s;
public:
    C() { s = "<not set>";}
    ~C() {}
    void Set(const std::string ss) { s=ss; }
    const std::string Get() { return s; }

    C &operator=(const C &c) { Set(c.Get()); return *this; }
    //error: passing ‘const C’ as ‘this’ argument of ‘const string C::Get()’
    // discards qualifiers [-fpermissive]


    //C &operator=(C &c) { Set(c.Get()); return *this; }   <-- works fine

};
4

2 回答 2

5

您需要将函数声明Get()为const：

const std::string Get() const { return s; }

即使Get()不更改任何成员值，编译器也会被指示只允许您调用显式标记的函数const。

gcc 指示您可以使用参数覆盖它的投诉-fpermissive；但是，通常最好不要这样做（否则为什么要声明任何内容const？）。通常，最好确保在const参数上调用的每个成员函数都是const成员函数。

这篇关于Const Correctness的文章非常有趣。

于 2012-07-02T23:14:19.630 回答
3

在你的operator =对象里面c是一个常量对象：它有const C类型。在 C++ 语言中，不允许调用常量对象的非常量成员函数。即调用c.Get()是非法的，因为你Get是一个非常量成员函数。这就是编译器报告错误的原因。

要么使您的c非常量（如代码的注释版本），要么使Get常量。由您决定哪种方法是正确的，但看起来您应该采用后者。

Get()附带说明一下，声明为返回没有多大意义const std::string。如果您通过引用（as const std::string &）返回它，那么这const将是合适的。但是由于您是按值返回，因此将返回类型声明const为没有那么有用。不过，这是你个人风格的问题。

于 2012-07-02T23:16:40.020 回答