基本平均程序中的意外数组行为

Question

似乎我总是来这里问愚蠢的问题，但它就是这样。截至目前，我正在参加我的第一门 compsci 课程，我们正在学习 c++。我之前对 c 有过非常基本的介绍，所以我想我会超越我目前的任务。现在我这样做只是为了炫耀，我觉得如果我不练习我以前的概念，它们最终会褪色。无论如何，解决问题！我应该编写一些代码，允许用户输入他们的姓名首字母，并进行一系列考试。现在这应该完成三件事：平均考试，打印输入的考试，并打印出他们的姓名首字母。嗯，本来很简单的任务，被你搞得一团糟。

#include <iostream>
#include <string>
#include <iomanip>

using namespace std;

int main()
{
  string uInitials;
  float avgExam = 0, tExam = 0;
  int aExams[10] = {'0'};
  int i, nExam = 0, cExam;


  cout << "Enter your three initials!";
  cin >> uInitials;


  do
      {

        cout << "Enter your exam(s) to be averaged. Enter 0 when          complete!\n";
        cin >> cExam;
        aExams[nExam] = cExam;  //I used this before nExam was incremented, in order to get nExam while it was '0' That way the first exam score would be properly saved in the first space
        nExam++;
        tExam += cExam;     //This is just to add all the exams up to later calculate the average
    }
  while(cExam != 0);

avgExam = tExam/(nExam - 1); //subtracted '1' from nExams to remove the  sentinel value from calculations.

cout << "The average for initials: " << uInitials << " is: " << avgExam <<   endl;
cout << "This average was obtained using the following scores that were entered: \n";

for(i = 0; i < (nExam+1); i++)
{
    cout << aExams[i] << endl; //Used a for loop to prevent redundancy
}

return 0;
}

前面是我的代码，问题是当我打印出输入的考试列表时，我得到了输出错误，它添加了两个“0”。此外，我觉得我让整个 do{}while() 循环变得非常笨拙，所以我也想改进它。如果有人能帮助这个可怜的、无知的初学者，我将不胜感激。感谢您的宝贵时间！

Answer 1

我可以给出的一些建议是例如在第 5 行没有必要 将 0 放在 ' ' 之间，甚至不需要使用 assign = 运算符。 您可以像这样初始化数组： int aExams[10]{0}; 它将所有元素初始化为0，但不能用于其他值。 例如，如果您编写，您将不会拥有所有值为 1 的元素 int aExams[10]{1}; 如果您打算初始化数组中的所有元素的值不是 0，则可以使用 fill_n();功能。

fill_n(aExams, 10, 1); 第一个参数是数组的名称，第二个是您希望使用第三个参数初始化的元素，第三个是您希望所有元素具有的值。

不要将未初始化的变量（如第 6 行中的 cExam 和 i 变量）留下。像 cExam=0; 一样初始化它（复制分配初始化）或 cExam(0)； （直接初始化）。后者调用 int 内置类型的构造函数。

我在你的 do-while 循环中看到的一个负面因素是你不能确保用户将输入 10 个考试，如果用户尝试在一个只能容纳 10 个的数组中输入 15 个考试，就会发生不好的事情。

只需将 while 更改为类似这样的内容： while( cExam != 0 && (nExam

您也可以在循环之外编写 do-while 循环的前两行。 只需要一次告诉用户要停止循环，他/她需要输入 0。无需在每次迭代时都告诉他们这一点，而且如果将这两行放在循环。

看这里我将如何编写代码并询问您是否有任何问题。

http://pastebin.com/3BFzrk5C

Answer 2

它在代码末尾打印出两个 0 的问题是您编写 for 循环的方式造成的。

代替：

for(i = 0; i < (nExam+1); i++)
{
    cout << aExams[i] << endl; //Used a for loop to prevent redundancy
}

用途：

for (i = 1; i < (nExam); i++)
{
    cout << aExams[i - 1] << endl; //Used a for loop to prevent redundancy
}