好的,所以我在 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 也会发生这种情况。任何帮助将不胜感激!谢谢!
更新:我搞定了。感谢大家帮助我!
最佳答案
如果要设置值,请使用 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 引用,因此您不必担心该函数会操纵输入。
关于c++ - 'this' 参数的类型为 const 但函数未标记为 const,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/32520642/