C++中NumberS类operator+多操作数重载出现未定义结果求助

我来帮你分析下问题根源，以及给出具体的修复方案～

问题核心原因 #

你代码里的operator+返回的是NumberS&（引用），但函数内部创建的是局部变量s——当函数执行完毕，这个局部对象会被销毁，返回的引用就变成了「悬垂引用」，指向一块已经被系统释放的内存。

当你执行number=number+"5"+"6"时，实际执行顺序是(number+"5")+"6"：

1. 第一个number+"5"返回的是悬垂引用，对应的对象已经销毁
2. 接着用这个无效引用去加"6"，访问的是不存在的内存，自然会产生未定义行为

修复方案 #

1. 修正operator+的返回类型与实现

把operator+的返回类型从引用改成值返回，让函数返回局部对象的副本，避免悬垂引用问题。同时优化实现逻辑，让代码更清晰：

修改NumberS.h中的运算符声明：

// 改成值返回，参数用const引用减少拷贝开销
NumberS operator+(const string& str);

修改NumberS.cpp中的实现：

NumberS NumberS::operator+(const string& str) {
    // 用stoll替代stoi，避免int溢出（因为成员变量是long long）
    long long add_value = stoll(str);
    // 创建新对象，直接计算当前值+待加值
    NumberS result(this->number + add_value);
    return result;
}

2. （可选但推荐）实现operator+=，符合C++惯例

通常我们会先实现operator+=（就地修改），再基于它实现operator+，这样代码更简洁高效，也符合C++的设计习惯：

在NumberS.h中添加operator+=声明：

NumberS& operator+=(const string& str);

在NumberS.cpp中实现：

// 就地修改当前对象，返回自身引用支持链式调用
NumberS& NumberS::operator+=(const string& str) {
    long long add_value = stoll(str);
    this->number += add_value;
    return *this;
}

// 基于operator+=实现operator+
NumberS NumberS::operator+(const string& str) {
    NumberS result(*this); // 复制当前对象
    result += str; // 调用+=完成加法
    return result;
}

3. 其他细节优化

• 把operator=的参数改成const string&，避免不必要的字符串拷贝：

  void operator=(const string& str);
  

• 友元operator<<的参数改成const NumberS&，减少对象拷贝：

  friend ostream& operator<<(ostream &os, const NumberS& numb) {
      os << to_string(numb.number);
      return os;
  }
  

现在再运行number=number+"5"+"6"，执行流程就变成了：

1. number+"5"返回一个值为22+5=27的新NumberS对象
2. 该对象再与"6"相加，返回值为27+6=33的新对象
3. 最终把33赋值给number，就能得到你期望的结果了

内容的提问来源于stack exchange，提问作者StephenKhoa