remove duplicate code & pass counter

[krisl100]

In my last excercise I have an IF/Else statement & a second pass counter, here is the code:

How can I remove the second pass counter (++pass) & the duplicate line of code? I just need a very simple solution. Thanks.

//Students are allowed to fail one exam, not both, this is how we achieve that goal.
		if ((midterm < EXAMSPASS && final < EXAMSPASS) || course < PASSMARK)
		{
			fail++;
			cout << endl << "RESULT: " << name << " " << "Failed " << endl;
			cout << "-------------------------------------------" << endl;
		}
		else if (course >= HONOURS)
		{
			pass++;
			cout << endl << "RESULT: " << name << " " << "Passed with Honours " 
				 << endl;
			cout << "-------------------------------------------" << endl;

		}
		
		else
		{
			//Since pass+1 is set above, need to reset the counter this way
			//so it counts the passed results correctly
			++pass;
			cout << endl << "RESULT: " << name << " " << "Passed " << endl;
			cout << "-------------------------------------------" << endl;
		}

[redhead]

hmm.. from what I can read out of your code part here, shouldn't this be enough ?

if ((midterm < EXAMSPASS && final < EXAMSPASS) || course < PASSMARK)
		{
			fail++;
			cout << endl << "RESULT: " << name << " " << "Failed " << endl;
			cout << "-------------------------------------------" << endl;
		}
		else
		{  /** if they don't fit the fail part, they pass **/ 
			pass++;
			cout << endl << "RESULT: " << name << " " << "Passed";

			if(course >= HONOURS)
			    cout << " with Honours ";
			cout << endl;
			cout << "-------------------------------------------" << endl;

		}

[krisl100]

Wow, that did it, sometimes it just takes a second pair of eyes, I knew it was something simple. Thanks.

[redhead]

You could even simplify it more ie:

/* allways show name */
cout <<  endl << "RESULT: " << name << " ";
if ((midterm < EXAMSPASS && final < EXAMSPASS) || course < PASSMARK)
  {
    fail++;
    cout << "Failed ";
  }
else
  {  /** if they don't fit the fail part, they pass **/ 
    pass++;
    cout << "Passed";
    if(course >= HONOURS)
      cout << " with Honours ";
  }
cout << endl << "-------------------------------------------" << endl;

[krisl100]

It works well, except that when the program is first run, it does not ask for the user's name ex: "Enter name or type QUIT to exit"

It just starts with Enter midterm mark, once in the loop it works fine, ex: when it prints out the first set of results, then it asks for the user name.

[krisl100]

Please disregard my last entry, I figured it out. Thanks.