Array is printing out weird symbols

[MinusTheBear]

Homework Statement

Compare two strings which represent test scores for a 20 question exam. If the student scores 15 or higher, they pass. Less than that they fail. Indicate whether a student passes or fails. Display how many answers they got right and wrong. Display which number they got wrong on the exam.

I simplified the problem at hand (it's actually a lot longer) and removed a lot of my code except the portion that is relevant. I made sure this portion of the code runs. The problem is the output for the answers they got wrong are weird symbols. I made bold the lines that are probably causing an issue. The code runs as expected with every test trial the professor gave EXCEPT for printing out the numbers they missed.

Homework Equations

n/a

The Attempt at a Solution

1. #include <iostream>
2. #include <string>
3. #include <cctype>
4. using namespace std;
5. 
   
6. const int NUM_QUESTIONS = 20;
7. int main()
8. {
9. int pass = 15;
10. int correct = 0;
11. int wrong = 0;
12. string missed;
13. string key = "BDAACABACDBCDADCCBDA";
14. string student = "BDAACABACDBCDADCCBDD"; // Only 1 answer is wrong, answer 20
15. for (int answer = 0; answer < NUM_QUESTIONS; answer++){
16. if (key[answer] == student[answer]){
17. correct++;
18. }
19. else{
20. missed[wrong] = static_cast<char>((answer+1));
21. wrong++;
22. }
23. }
24. if (correct >= pass){
25. count << "Congratulations. You passed the exam.\n";
26. }
27. else{
28. count << "Sorry. You failed the exam.\n";
29. }
30. count << "You got " << correct << " questions correct.\n";
31. count << "You missed the following" << wrong << " question(s): ";
32. for (int element = 0; element < wrong; element++){
33. count << missed[element] << " ";
34. }
37. return 0;
38. }

When it runs I get this:
Congratulations. You passed the exam.
You got 19 questions correct.
You missed the following 1 question(s):

 

[phinds]

Just as a side note, you should learn to use the code tags so that your code is readable.

 

[Khashishi]

Line 20 is problematic. You shouldn't use array subscripting on the string, because you don't know how long the elements are. (Some questions have multiple digits). Just concatenate the result onto the string. And you shouldn't use static_cast here. I think you can use to_string() or itoa() to convert to a string.

 

[MinusTheBear]

Just as a side note, you should learn to use the code tags so that your code is readable.

How do I do this?

Line 20 is problematic. You shouldn't use array subscripting on the string, because you don't know how long the elements are. (Some questions have multiple digits). Just concatenate the result onto the string. And you shouldn't use static_cast here. I think you can use to_string() or itoa() to convert to a string.

This solved it! Thank you! I was using to_string() originally, but the subscripting must've messed it up. I'm not sure why, though. In the main part of the real program I have input validation within a class so I know for a fact each element is 1 character and each string is 20 characters. I was also using static_cast on another example which worked fine -- and it was similar to this. Whatever it was it fixed it. Thanks!

The only issue now is getting it to print out double digits as 10, 11, etc. and not 1 0 1 1, but this is a start. Thanks.

 

[phinds]

How do I do this?

https://www.physicsforums.com/help/bb-codes