If/else if/else problem

Ok so I am not seeing why my code is not working. The only way I can get the A resposne is if I input 1000 points or higher. If anything less that 1000 total points are put in the program jumps to the else statment unless the exampoints are less than 240, at least that part of the program works. Also I would love to be able to debug this myself but I can not figure out how to do it with jgrasp, i think the issue is because I am using borland 5.5 compiler. Anyways here is the program. Any help would be appriciated.

Code:

---

#include <iostream>


using namespace std;


const int POSSIBLE = 1000;
const int A = 90;
const int B = 80;
const int C = 70;
const int D = 60;
const int EXAM = 240;
int main()
{
  int exam;
  int hw;
  int assignment;
  int earned;
  int grade;
 
  cout << "How many points did you earn on exams?" << endl;
  cin >> exam;
  cout << "How many points did you earn on the MPL homework?" << endl;
  cin >> hw;
  cout << "How many points did you earn on the program assignments?" << endl;
  cin >> assignment;
 
  earned = exam + hw + assignment;
 
  grade = (earned / POSSIBLE) * 100;
 
  if (grade >= A && exam >= EXAM)
      {
            cout << "You earned a total of" << " " << earned << " " << "points" << endl;
            cout << "Congratulations this gives you a letter grade of A!" << endl;
      }
  else if (grade >= B && exam >= EXAM)
        {
              cout << "You earned a total of" << " " << earned << " " << "points" << endl;
              cout << "Congratulations this gives you a letter grade of B!" << endl;
        }
  else if (grade >= C && exam >= EXAM)
      {
            cout << "You earned a total of" << " " << earned << " " << "points" << endl;
            cout << "Good job this gives you a letter grade of C!" << endl;       
      }
  else if (grade >= D && exam >= EXAM)
      {
            cout << "You earned a total of" << " " << earned << "points" << endl;
            cout << "Well at least you passed with a letter grade of D!" << endl;     
      }
 
  else if (exam < EXAM)
      { 
        cout << "You earned a total of" << " " << earned << "points" << endl;
        cout << "You got a grade of F and have failed this class" << endl;
        cout << "If you did not get at least 240 points on the exams" << endl;
        cout << "you have failed because to pass the class you must" << endl;
        cout << "have at least 240 points on exams in order to pass" << endl;
      }
  else
      {
        cout << "Error you have entered a negative vaule for points earned." << endl;
        cout << "in this class" << endl;
        cout << "Please run program again and enter positive value" << endl;
        cout << "for points earned in the course" << endl;
      }   
  return 0;
}

---

Look-up Integer division.

Code:

---

grade = (earned / POSSIBLE) * 100;

---

Example 0 = 3/4

I would suggest this

Code:

---

grade = ((double)earned / POSSIBLE) * 100;

---

or this

Code:

---

grade = (100.0 * earned) / POSSIBLE;

---

Tim S.

Every integer division produces an integral result with rounding toward zero. So earned/POSSIBLE will always produce a result of zero if earned is less than 1000.

Instead of "grade = (learned/POSSIBLE)*100" try doing "(100*learned)/POSSIBLE". The division by POSSIBLE (i.e. 1000) still rounds toward zero, but 100*learned is a bigger value, so only values of learned less than 10 will produce a result of zero.

The compiler is not permitted to reorder operations on integral values/types to give a result you might expect (as you are in this case) - because the standard describes rules of precedence and associativity of operators that are fixed. That means YOU have to reorder operations to give the result you expect.

I wouldn't suggest floating point (like Tim describes) unless there is a particular need for floating point in subsequent code - which is not the case here.

thank you so much I have been trying to figure this out for like 5 hours across two days and could not figure out why it wasn't working.

Now that I think of it, that makes sense and I know that with int it takes anything less than the whole number and rounds down but I kept doing the math and thinking it works here why doesnt it work in the code not thinking of how integers work in c++.

By the way, at the moment you're declaring all variables used in your main function at the top of the function. Don't do that. That isn't the C++ way, in fact it's not even the modern C way anymore. You want to declare your local variables as close to their first use as possible, thereby limiting them to the smallest scope possible. That way you can also assign them a proper value right away ... instead of leaving them uninitialized until you (hopefully) remember to assign them a value later in the program. Now with int exam;, int hw; and int assignment you don't have much of a choice, but earned and grade could have been declared and initialized in one step (and made const to boot!).