'this'参数的类型为const,但函数未标记为const

Question

好的,所以我在C++中有点像菜鸟,在我的第二次任务中,我需要使用公共和私有参数等来制作类.基本上,mutator函数不起作用,因为显然它们不是const类型的？

这是带有类的头文件:

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"

基本上,确切的错误消息是:

成员函数'set_Name'不可行:'this'参数的类型为'const Customer',但函数不是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;
}

另外，您还必须更改方法的接口：

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&作为函数参数比字符串更好的通过，当它作为参数传递给不可复制的字符串。由于它是一个const引用，因此您不必担心该函数会操纵输入。