Question

我正在编写一个程序，它将文本文件中的一百个50位数字的前十位数相加。很容易，我得到了正确的答案，但我试图理解一些似乎正在发生的奇怪的事情。当包含第一个cout行时，它出现在程序输出下方：

8549904804355727945614717320-7-60-4-3-2-7-3-4-8-7-7-6-1-840-9-5-2-3-39622981389563771795892386478402037366780930326737553

如果将其更改为右侧的注释，则输出：

97414719961102146325906539660-4-7-2-4-4-3-4-8-2-8-3-8-2-870-8-5-2-3-39622981389563771795892386478402037366780930326737553

我无法理解 1）我输出的内容可能以某种方式影响它输出的未来值 2）破折号如何进入程序。

注意：输出是反向所有字符串的总和。

char file[] = "nums.txt";
ifstream ss(file);
string iString;
string sarr[100];
int sum[100];

while(getline(ss, iString, '\n')) {
  sarr[i] = iString;
  i++;
}
ss.close();

cout << sarr[0] << endl << endl; //or cout<<(int)sarr[0][3]-48<<endl<<endl;
for(int i = 99; i >= 0; i--) {
  sum[i] = 0;
  for(int j = 0; j < 100; j++) {
    sum[i] += (int)sarr[j][i];
  }
  sum[i] -= 4800;//since 100 strings and '0' in ascii is 48 subtract 48*100
}

for(int i = 99; i > 0; i--) {
  sum[i-1] += sum[i] / 10;
  sum[i] = sum[i] % 10;
  cout << sum[i];
}
cout << sum[0] << endl << endl;

Answer 1

你说你的文本文件的第一行是37107287533902102798797998220837590246510135740250，长度是50个字符，但是这里

for(int i = 99; i >= 0; i--) {
  sum[i] = 0;
  for(int j = 0; j < 100; j++) {
    sum[i] += (int)sarr[j][i];
  }
  sum[i] -= 4800;//since 100 strings and '0' in ascii is 48 subtract 48*100
}

您正在循环使用i从99到0并使用i索引该字符串。您的索引超出范围并导致未定义的行为。这是使用硬编码值而不是使用数据来确定循环边界的危险。

当像这样的随机事件发生并且您正在使用数组做很多工作时，您应该首先考虑的是在某处索引越界。这可能是UB最常见的原因。

cout的存在改变了程序的结果

1 个答案: