= 重载不起作用

= overloading doesn't work

本文关键字:不起作用 重载      更新时间:2023-10-16
#include"stdafx.h"
#include<iostream>
class Item
{
 int exponent;
 double coefficient;     //two private members 
 public:
 //constructor
 Item(int exp = 0, int coef = 0) :exponent(exp), coefficient(coef) {}
 Item(Item& item) {
    exponent = item.exponent;
    coefficient = item.coefficient;
 }                                      //copy constructor
 //interface to change private member exponent
 void change_exp(const int n) {           
    Item::exponent = n;
 }
 //interface to change private member coefficient
 void change_coef(const double n) {
    Item::coefficient = n;
 }
 int get_exp() { return exponent; }// interface to get exponent
 double get_coef() { return coefficient; }//  interface to get coefficient
 ~Item() {}
 Item operator=(Item item2) {
    change_exp(item2.get_exp());
    change_coef(item2.get_coef());
    return (*this);
 }
 Item operator*(Item & item2) {
    Item result;
    result.change_coef(coefficient * item2.get_coef());
    result.change_exp(exponent + item2.get_exp());
    return result;
 }
 };
 int main() {
 Item test(2, 7);
 Item test2(4, 5);
 Item result;
 result = test * test2;    //BUG!!! 
 return 0;
 }

错误出现在注释为 BUG 的行上错误 C2679 二进制"=":未找到采用类型为"Item"的右侧操作数的运算符(或者没有可接受的转换(
当我只分配一个对象时,例如 结果=测试;没事。我不知道我的=重载函数哪里出了问题。请帮忙...

您需要

将复制构造函数声明为 Item(const Item& item)

test * test2的结果是一个没有地址的临时右值,因此您无法创建对它的(左值(引用。但是,允许创建常量引用。

此问题和答案可能会产生与此主题相关的更多信息:C++:对右值的常量引用

你的代码有很多问题。

  • 首先,您应该尊重访问器和突变器方法等命名约定。您应该将访问器方法命名为 setExp 而不是 change_exp

  • 其次,你应该知道什么时候重载赋值运算符之类的东西(请在 C++ 中谷歌大三(。基本上,如果您只有基元类型的字段,则不必重载赋值运算符。

  • 最后,您必须尽可能使用 const 限定符。例如,在访问器方法的末尾,在复制构造器的参数前面等。

复制构造函数应如下所示

Item(const Item& item) {
    exponent = item.exponent;
    coefficient = item.coefficient;
 }   

您的分配运算符应如下所示

Item& operator=(const Item& item2) {
    change_exp(item2.get_exp());
    change_coef(item2.get_coef());
    return (*this);
 }

字段系数的访问器方法如下所示

double getCoefficient() const { return coefficient; }//  interface to get coefficient