Question

我正在尝试创建一个简单的GPA计算器，提示用户输入课程数量（使用新的）。接下来是依赖于课程数量的for循环，要求用户输入课程的成绩和学分数。该程序完成循环和错误。请帮忙。这是代码（我第一次使用这个论坛网站btw）：

#include <iostream>
#include <conio.h>
using namespace std;

 int main(){
   cout<<"Welcome to the GPA calculator";
   cout <<endl;
   cout<<"Please enter the number of courses you wish to calculate : ";
  int*numberOfCourses = new int;
  cin>>*numberOfCourses; //must dereference as it is a pointer and I AM SETTING variable.
  char grade, *credits= new char; 
  int gradesOfPerson = 0;
  int*score = new int; 
  int j = 0;
  int i = 0; 
  int*cumulativeScore= new int; 

  while( i< *numberOfCourses){
  cout <<"Please enter the credits of your " <<(i+1) <<" course. " ;
  cin >>*credits;
  cin.get();
  cout << "Please enter your grade :";
  cin>>(grade); 
  cout <<endl;
  switch (grade){
   case 1: if (grade=='A'){
        *score = 4;
        break; }
   case 2: if (grade=='B'){
        *score = 3;
        break; }
   case 3: if (grade=='C'){
        *score = 2;
        break; }
   case 4: if (grade=='D'){
        *score = 1;
        break; }

   case 5: if (grade=='D'){
        *score = 1;
        break; }
   case 6: if (grade =='E'){
        *score = 0;
        break;
        }
        }
   gradesOfPerson = ((*score)*(*credits));
   cumulativeScore += gradesOfPerson;
   i++;
   }
 int gpa = (*cumulativeScore)/(*numberOfCourses); 
 cout <<"Your GPA is : " <<gpa;
 delete numberOfCourses, credits, score, cumulativeScore;
 }

抱歉糟糕的缩进（使用Dev C ++）

Answer 1

您的代码存在许多问题，但不要认为我通过指出它们而令人沮丧。假设你是一个相对初学者，这是非常好的。

主要问题在于以下几行：

cumulativeScore += gradesOfPerson;

。您已将cumulativeScore声明为指针;它包含您感兴趣的数据的地址，而不是数据本身。您应该将其更改为

*cumulativeScore += gradeOfPerson

或使cumulativeScore成为整数变量（并更改将其用作指针的所有位置）。

另一个关键错误在于切换语句。而不是像：

case 4: if (grade == 'D') {
 // logic to execute if grade is 'D'
}
break;

这样做：

case 'D':
 // logic to execute if grade is 'D'
break;

接下来，您需要将*cumulativeScore初始化为0，因为在执行开始时它可以包含任何内容。

最后，*score应该是数字类型，而不是字符。被解释为数字的字符'4'的值实际上不是4，这会导致错误，因为您将其视为此类错误。有关参考，请参阅此处的ASCII字符代码列表：http://www.asciitable.com/

至于其他问题（实际上并不会导致程序失败，但不是最佳实践）：

你对指针的使用很奇怪 - 为什么不简单地分配普通变量？您使用grade，gradesOfPerson，i和j执行了此操作，因此您清楚地知道如何操作。你为什么选择制作其余的变量指针？
不要包含conio.h。首先，你没有使用那里声明的任何函数。其次，它是非标准的，在大多数平台上都不可用。
你应该做更多的错误处理：用户输入合理的成绩字母？你确定他们输入的课程数量是肯定的吗？等等。

最后，作为一个注释，您可以免费获得Microsoft Visual C ++的Express版本。它得到了积极的维护，并且比Dev-C ++领先一年（例如，它可以帮助您正确地缩进代码！：D）

祝你好运！

Answer 2

在这一行：

cumulativeScore += gradesOfPerson;

cumulativeScore是一个指针，因此该行移动到指向的位置。你可能想写：

*cumulativeScore += gradesOfPerson;

它崩溃的原因是因为你正在移动accumScore指向的地方，后来当你试图取消引用它时，它指向无效的内存。

Answer 3

如果没有看到错误，很难说出错是什么，但乍一看，如果gpa行（int gpa = (*cumulativeScore)/(*numberOfCourses);）现在没有抛出错误，那么将来可能会出错。 gpa应为double。

Answer 4

顺便说一下，你的delete陈述严重受损。你不能一次删除多个变量，这个

delete numberOfCourses, credits, score, cumulativeScore;

实际上是使用丢弃其左侧的C ++逗号运算符，因此只有cumulativeScore被释放。

协助使用基于C ++的基本GPA计算器和cin用法

4 个答案: