在 C++ 中输出非常大的数字答案

【问题标题】：Outputting weirdly large number in C++在 C++ 中输出非常大的数字
【发布时间】：2018-06-19 00:51:40
【问题描述】：

嘿，C++ 新手，正在研究一个简单的问题，该问题采用 Int 序列并输出数字之和，没有最小和最大数字。

如果向量有一个或没有元素，则返回 0。这是我的代码：

#include <iostream>
#include <algorithm>
#include <vector>
using namespace std;

int test(vector<int> numbers) {

  typedef vector<int>::size_type vec_sz;
  vec_sz size = numbers.size();


  if (size <= 1) {
    cout << "Vector less than 2" << endl;
    return 0;
  } else {
    sort(numbers.begin(), numbers.end());

    int answer;
    for (int i=1; i < size-1; i++) {
        answer += numbers[i];
    }
    return answer;

  }

}

int main() {
    vector<int> vec = {2,1,3,4,6,5,7,9,8,10};
    cout << test(vec) << endl;

    vector<int> vec_2 = {1};
    cout << test(vec_2) << endl;

}

当我运行它时，我会得到以下内容：

3829804
Vector less than 2
0

当向量 > 1 时，为什么我得到一个大得离谱的数字，而它只是应该返回 2-8 的总和？

当我在不检查包含 1 个或更少项目的向量的情况下制作程序时，我没有问题。感谢您的帮助！

【问题讨论】：

局部变量（比如你的函数中的answer）默认是未初始化的。它们的值是不确定的（并且看似随机）。在没有初始化的情况下使用这些变量会导致undefined behavior。
..... 所以将int answer 更改为int answer = 0。或者，更好的是，用answer 替换整个逻辑并在循环中添加值，而不是return std::accumulate(numbers.begin(), numbers.end(), 0)
您应该打开编译器的警告，这会指出问题所在。
@Peter - 你是说std::accumulate(numbers.begin()+1, numbers.end()-1, 0)吗？由于必须排除第一个和最后一个。（最好还是跳过所有的存储和排序，只需在输入结束时从滚动和中减去最小值和最大值）。

标签： c++ function vector

【解决方案1】：

answer 在您的函数中未初始化，将具有 indeterminate 值。
任何使用都会导致undefined behavior。

int answer; // uninitialized
            // answer will have indeterminate value
for (int i=1; i < size-1; i++) {
    answer += numbers[i]; // undefined behavior
}
return answer;

根据dcl.init/12：

如果为对象指定了 no 初始化程序，则该对象是默认初始化的。

存储具有自动或动态存储持续时间的对象时得到，对象有一个indeterminate值，

所以只需初始化answer:

int answer = 0;

【讨论】：

【解决方案2】：

尝试在 for 循环之前将答案设置为 0。这应该可以解决问题，因为 APU 正在尝试将一个数字添加到一个未定义的数字。

【讨论】：