C# store the question number and outut the question number

Question

0.00/5 (No votes)

See more:

Hello, so i am having a logical error i cant seem to store and incrament the questions variable

Program Specifications The human resources department has asked you to write a program that grades the candidate’s technical skills. The exam has 20 multiple choice questions. Here are the correct answers: 1. E 2. D 3. D 4. B 5. A 6. C 7. E 8. B 9. D 10. C 11. D 12. A 13. A 14. D 15. E 16. E 17. A 18. E 19. A 20. D A candidate must correctly answer 15 out of 20 questions to pass the exam and advance to the next level of interviews. The exam questions are printed on paper and the candidate enters their answers to the questions using the program you write. After the candidate enter all the answers to the questions, the program will either display a message indicating that they passed the exam and the total number of correct answers or a message indicating that they failed the exam and the total number of incorrect answers and the list of question numbers that the candidate answered incorrectly.

Sample data Expected result
Passed with 20 correct answers.
Passed with 15 correct answers.
Failed with 10 incorrect answers. Incorrect questions: 6, 7, 8, 9, 10, 16, 17, 18, 19, 20 if i need to make alterations to my code/ questions please LET ME KNOW, i will try and make it readable as possible

Thank you

What I have tried:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace CPSC1012_CorePortfolioAssignment3_EmilyJohnson
{
    class Program
    {
        static void Main(string[] args)
        {
            //Declare variables
            char[] anwserKey = { 'E', 'D', 'D', 'B', 'A', 'C', 'E', 'B', 'D', 'C', 'D', 'A', 'A', 'D', 'E', 'E', 'A', 'E', 'A', 'D' };
            char anwser;

            int[] questions = { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20 };
            int correct = 0;
            int incorrect = 0;
            
            int question = questions.Length;
            int wrong = 0;
           
            for (int i = 0; i < questions.Length; i++)
            {
                question = i + 1;
                Console.Write("{0}: ", question);
                anwser = char.Parse(Console.ReadLine().ToUpper());

                if (anwser == anwserKey[i])
                {
                    correct++;
                }
                else
                {
                    incorrect++;
                    
                    if(anwser != anwserKey[i])
                    {                        
                        questions[i] = i + 1;
                        question = questions[i];
                    }
                }
            }
            if (correct >= 15)
            {
                Console.WriteLine("Passes! you got {0} questions correct", correct);
            }

            else
            {
                Console.WriteLine("Failed! you got {0} questions wrong", incorrect);
                Console.WriteLine("Incorrect Question: ");
                for (int n = 0; n < questions.Length; n++)
                {

                    Console.Write("{0} ", questions[incorrect]);
                }


            } Console.ReadLine();

        }

    }
}

Posted 24-Nov-16 19:11pm

SpazticUnicorn

Updated 24-Nov-16 20:23pm

Add a Solution

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Peter Leow · Answer 1 · 2016-11-24T19:43:00

1. Look at this part:

C#

if(anwser != anwserKey[i])
{
    questions[i] = i + 1;
    question = questions[i];
}

do you mean:

C#

if(anwser != anwserKey[i])
{
    i = i + 1;
    question = questions[i];
}

? Even that, it is wrong. The increment of question have nothing to do with the correctness of answer.
2. The answerKey is an array, each of its answer element is identified by an index number, why not use this index number as the question number rather than creating another array called questions?

3. The two variables correct and incorrect always add up to total number of questions, do you need both?
+++++ [Round 2 ] +++++
4. If you want to be able to output the question numbers that have the wrong answers, then you will need to store these question numbers, I suggest using a List, e.g.

C#

List<int> questionWithWrongAnswer = new questionWithWrongAnswer<int>();
// other code omitted
 
if (anwser == anwserKey[i])
{
    correct++;
}
else
{
    incorrect++;
                    
    questionWithWrongAnswer.Add(i);
}

Learn C# List[^]
5. Last but not least, what do you think of this:

incorrect = answerKey.Length - correct;

Patrice T · Answer 2 · 2016-11-24T20:23:00

First I would simplify

C#

for (int i = 0; i < questions.Length; i++)
{
    question = i + 1;
    Console.Write("{0}: ", question);
    anwser = char.Parse(Console.ReadLine().ToUpper());

    if (anwser == anwserKey[i])
    {
        correct++;
    }
    else
    {
        incorrect++;

        if(anwser != anwserKey[i])
        {
            questions[i] = i + 1;
            question = questions[i];
        }
    }
}

by removing weird useless code

C#

for (int i = 0; i < questions.Length; i++)
{
    question = i + 1;
    Console.Write("{0}: ", question);
    anwser = char.Parse(Console.ReadLine().ToUpper());

    if (anwser == anwserKey[i])
    {
        correct++;
    }
    else
    {
        incorrect++;
    }
}

Then, I would kick the debugger to see what the code is doing.
Then you have to think how to halter questions, which already contain questions numbers, to be able to tell which ones was failed.

You should learn to use the debugger as soon as possible. Rather than guessing what your code is doing, It is time to see your code executing and ensuring that it does what you expect.

The debugger allow you to follow the execution line by line, inspect variables and you will see that there is a point where it stop doing what you expect.
Debugger - Wikipedia, the free encyclopedia[^]
Mastering Debugging in Visual Studio 2010 - A Beginner's Guide[^]

The debugger is here to show you what your code is doing and your task is to compare with what it should do.
There is no magic in the debugger, it don't find bugs, it just help you to. When the code don't do what is expected, you are close to a bug.

[Update]
The problem is in your mind.
Storing the question number at question position in questions would ne a good idea if questions was not already containing the question number at same position.
You have to define what you store in questions and when, in order to know which question was failed.

C# store the question number and outut the question number

2 solutions

Solution 1

Solution 2

Add your solution here

Preview 0