c++ - 'this' 参数的类型为 const 但函数未标记为 const

Question

好的，所以我在 C++ 方面有点菜鸟，在我的第二个作业中，我需要使用公共(public)和私有(private)参数等创建类。基本上，mutator 函数不起作用，因为它们显然不是输入常量？

这是带有类的头文件:

class Customer {

private:
    string PhoneNumber_;
    string Name_;
    string Address_;

public:
    string get_PhoneNumber() const {return PhoneNumber_;} // Accessor
    const void set_PhoneNumber(unsigned x) {PhoneNumber_ = x;} // Mutator

    string get_Name() const {return Name_;}
    const void set_Name(unsigned x) {Name_ = x;}

    string get_Address() const {return Address_;}
    const void set_Address(unsigned x)  {Address_ = x;}
};

// declare the CreateCustomer function prototype with default values
Customer* CreateCustomer(const string& id = BLANK, const string& name = BLANK, const string& address = BLANK);

Customer* CreateCustomer(const string& id, const string& name, const string& address) {
    Customer* temp = new Customer();

    temp->get_PhoneNumber() = id; // Due to the Accessors and Mutators PhoneNumber, Name and Address are now functions
    temp->get_Name() = name;
    temp->get_Address() = address;

    return temp;
}

这是我在 main.cpp 文件中遇到的错误:

cout << "\n\nDear ";
    cout << Charge[0].Holder.set_Name() << " (" << Charge[0].Holder.set_PhoneNumber() << ")";  //  DisplayCustomer(customer) ;

    cout << ",\n" << Charge[0].Holder.set_Address() << "\n\n"

基本上，确切的错误信息是:

Member function 'set_Name' not viable: 'this' argument has type 'const Customer', but function is not type const

set_PhoneNumber 和 set_Address 也会发生这种情况。任何帮助将不胜感激!谢谢!

更新:我搞定了。感谢大家帮助我!

Answer 1

如果要设置值，请使用 set 方法。 get 方法只是获取变量，而不是设置类的内部变量(如果它们是按照您的方式定义的)。

正确的用法是:

Customer* CreateCustomer(const string& id, const string& name, const string& address) {
    Customer* temp = new Customer();

    temp->set_PhoneNumber( id );
    temp->set_Name( name );
    temp->set_Address( address );

    return temp;
}

此外，您必须更改方法的接口(interface):

class Customer {

private:
    string PhoneNumber_;
    string Name_;
    string Address_;

public:
    string get_PhoneNumber() const {return PhoneNumber_;} // Accessor
    void set_PhoneNumber(const string& x) {PhoneNumber_ = x;} // Mutator

    string get_Name() const {return Name_;}
    void set_Name(const string& x) {Name_ = x;}

    string get_Address() const {return Address_;}
    void set_Address(const string& x)  {Address_ = x;}
};

因为你想设置字符串而不是数字。

使用 const string& 作为函数参数比 string 在作为参数传递时不复制字符串要好。由于它是一个 const 引用，因此您不必担心该函数会操纵输入。