Displaying lists from arrays.

[mgjohns61585]

I am really having trouble with the last for loop. I am supposed to display the question numbers that were answered incorrectly, not the actual answers. Thanks

 #include <iostream>
using namespace std;

int main() {
char answers[20], usersAnswers[20];
int correct = 0, incorrect = 0;
answers[0] = 'A';
answers[1] = 'D';
answers[2] = 'B';
answers[3] = 'B';
answers[4] = 'C';
answers[5] = 'B';
answers[6] = 'A';
answers[7] = 'B';
answers[8] = 'C';
answers[9] = 'D';
answers[10] = 'A';
answers[11] = 'C';
answers[12] = 'D';
answers[13] = 'B';
answers[14] = 'D';
answers[15] = 'C';
answers[16] = 'C';
answers[17] = 'A';
answers[18] = 'D';
answers[19] = 'B';


	cout <<"Driver's License Exam"<< endl;


//Loop allowing the user to enter their answers //
for (int i = 0; i < 20; i++)
{
cout << "Enter your answer: ";
	cin >> usersAnswers[i];
if(usersAnswers[i] == 'A')
	cin >> usersAnswers[i];
else if(usersAnswers[i] == 'B')
	cin >> usersAnswers[i];
else if(usersAnswers[i] =='C')
	cin >> usersAnswers[i];
else if(usersAnswers[i] == 'D')
	cin >> usersAnswers[i];
else
	cout << "You must enter A, B, C, D "<< endl;

}





for(int i = 0; i < 20; ++i) {
if(usersAnswers[i] == answers[i])
++correct;
else
++incorrect;
}

if(correct >= 15)
	cout <<"You Passed!!!"<< endl;
else
	cout <<"You Failed!!!"<< endl;



cout << endl;
cout << "You got " << correct << " correct" << endl;
cout << "You got " << incorrect << " incorrect" << endl;

for (int i = 0; i < 20; ++i)
    {
        if (usersAnswers[i] != answers[i])
           
			cout<< '#' << i+1 << " was answered incorrectly" << '\n';    
    }
return 0;
}

[cire]

If you want to output the question number instead of the answer, then... do that.

    for (std::size_t i=0; i<20; ++i)
    {
        if (usersAnswers[i] != answers[i])
            std::cout << i+1 << '\n';    // assuming you want numbers in the range 1 to 20.
    }

[shadder]

You need to save the incorrect answer in another loop
For that you have tho change your line 42-43 somewhat like this

else
{
incorrectNo[k]=i;
k++;
++incorrect;
}

Don't forget to add the incorrectNo variable :D

Then you can change your line 57-60 somewhat like this

for(int i=0;i<k;i++)
cout<<incorrectNo[i]<<endl;

[mgjohns61585]

Thank you so much.

[shadder]

Welcome :D

[mgjohns61585]

I am having one last problem with my program. I cannot get the input validation to work correctly. It is making me input a char twice if it is enter correctly. Somewhere in there something if off. I have updated my code up top.

[shadder]

You are checking the answer and again asking the user to answer it... You have to take the input and then check for its validity...

this

for (int i = 0; i < 20; i++)
{
cout << "Enter your answer: ";
	cin >> usersAnswers[i];
if(usersAnswers[i] == 'A')
	cin >> usersAnswers[i];
else if(usersAnswers[i] == 'B')
	cin >> usersAnswers[i];
else if(usersAnswers[i] =='C')
	cin >> usersAnswers[i];
else if(usersAnswers[i] == 'D')
	cin >> usersAnswers[i];
else
	cout << "You must enter A, B, C, D "<< endl;

}

should be

reattempt:
cout << "Enter your answer: ";
	cin >> usersAnswers[i];
switch (usersAnswers[i])
{
case 'A': case 'B': case 'C': case 'D':
break;
default:
cout<<"cout << "You must enter A, B, C, D only"<< endl;";
goto reattempt;
break;//I dont think this break would be necessary would it be?

}

here I mention two things

1.switch-case was deliberately preferred over if-else as it is shorter... You could also have used ||(OR) operator but that would also be very long...

2.I have used a goto it is a bad habit.... You should sincerely try to avoid it on big codes... Alternatively you could have used a while loop

hope it helps

[mgjohns61585]

I tried to do a while loop. I don't understand what the heck is going on.

for (int i = 0; i < 20; i++)
{
cout << "Enter your answer: ";
	cin >> usersAnswers[i];
while(usersAnswers[i] != 'A', 'B', 'C', 'D')
	cout <<"You must enter A, B, C, D "<< endl;
cin >> usersAnswers[i];
}

[shadder]

check your line 5 it should be like this

while(usersAnswers[i] != 'A' || usersAnswers[i] != 'B' || usersAnswers[i] != 'C' || usersAnswers[i] != 'D')

you have to use || (OR) operator instead of comma (,) operator