Help with value returning function.

[bbunn77]

Afternoon guys, so I am having an issue with my calcGPA function. It is not returning any value other then 0, and im not really sure why. It is probably something I am overlooking, I could use a hint. Thanks

  #include<iostream>
#include<string>

using namespace std;


double calcGPA(int classNum, int credit[], char grade[]);



int main()
{
	int classNum;
	cout <<"How many courses did you take this semester?" << endl;
    cin >> classNum;
    string courseName[10];
    char grade[10];
    int credit[10];

    for(int i =0; i< classNum; i++)
	 {
	       cout <<"Please enter the course name." << endl;
	       cin >> courseName[i];
	       cout <<"Please enter the courses grade and credit amount.";
	       cin >> grade[i] >> credit[i];
	       
	   }
   calcGPA(classNum, credit, grade);
   cout <<"Your GPA is :" << calcGPA(classNum,credit,grade);
   system("pause");
   return 0;
}
double calcGPA(int classNum, int credit[], char grade[])
{
	double qualityPoint = 0;
	double creditHours = 0;
	double count = 0;
	for(int i=0; i< classNum; i++)
	{
		creditHours = credit[i] + creditHours;
	}
	
	for(int i=0; i>=classNum; i++)
	{
		switch(grade[i])
		{
			case 'A': count = credit[i] * 4.00 + count;
			break;
			
			case 'B': count = credit[i] *3.00 + count;
			break;
			
			case 'C': count = credit[i] * 2.00 + count;
			break;
			
			case 'D': count = credit[i] *1.00 + count;
			break;
			
			case 'F': count = credit[i] *0.00 + count;
			break;
			
			default : cout << "Invalid grade" << endl;
                     
			}
			
		}
		qualityPoint = count;
		
		return qualityPoint * creditHours;
	}

[TarikNeaj]

It's because your switch statement is never entered, therefore count is always equal to 0, therefore qualityPoint will be zero, therefore qualityPoint * creditHours will be 0.

The reason is your for-loop - for(int i=0; i>=classNum; i++)

Specifically the i >= classNum. Im sure you can see your mistake here.
Change it to - for(int i=0; i < classNum; i++)

[bbunn77]

Thanks, its always something simple.

[TarikNeaj]

You should learn how to debug so you can catch them in a minute instead of an hour. It's pretty important^^