3

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

#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++）

4

4 回答 4

2

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

主要问题在于以下行：

cumulativeScore += gradesOfPerson;

. 您已将累积分数声明为指针；它保存您感兴趣的数据的地址，而不是数据本身。您应该将其更改为

*cumulativeScore += gradeOfPerson

或将累积分数设为整数变量（并更改您将其用作指针的所有位置）。

另一个关键错误是在您的switch 语句中。而不是类似的东西：

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）

祝你好运！

于 2011-06-24T03:14:05.680 回答
1

在这条线上：

cumulativeScore += gradesOfPerson;

累积分数是一个指针，所以这条线移动到它指向的地方。你可能想写：

*cumulativeScore += gradesOfPerson;

它崩溃的原因是因为您正在移动累积分数指向的位置，后来当您尝试取消引用它时，它指向无效内存。

于 2011-06-24T03:24:08.183 回答
0

顺便说一句，你的delete陈述被严重破坏了。你不能一次删除多个变量，这个

delete numberOfCourses, credits, score, cumulativeScore;

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

于 2011-06-24T04:22:52.630 回答
0

如果没有看到错误，很难说出哪里出了问题，但乍一看，如果gpa( int gpa = (*cumulativeScore)/(*numberOfCourses);) 行现在没有抛出错误，那么它可能会在未来出现。gpa应该是一个double。

于 2011-06-24T02:56:05.580 回答