C# how to add comma in output

Question

I have this code all working good - it only accepts 20 input from user - compare them with the correct answers and finally show the result that which questions are not correct- code is working - shows up the result BUT not separated. its like 156791012 are incorrect. see the attachment.

enter image description here

    static void Main(string[] args)
    {
        char[] studentAnswers = new char[20];            
        char[] answers = new char[] { 'E', 'D', 'D', 'B', 'A', 'C', 'E', 'B', 'D', 'C', 'D', 'A', 'A', 'D', 'E', 'E', 'A', 'E', 'A', 'D' };
        int[] wrongAnswers = new int[20];            
        int correctlyAnswered = 0;
        int falselyAnswered = 0;
        string list = "";


        for (int i = 0; i < studentAnswers.Length; i++)
        {
            studentAnswers[i] = InputOutput_v1.GetValidChar("Question ",i);

            if (studentAnswers[i] == answers[i])
            {
                correctlyAnswered = correctlyAnswered + 1;
            }
            else
            {
                falselyAnswered = falselyAnswered + 1;                    
                wrongAnswers[i] = i + 1;
                list += i + 1;

            }

        }
        if (correctlyAnswered >= 15)
        {
            Console.WriteLine("Passed with {0} correct answers",correctlyAnswered);
        }
        else
        {
            Console.WriteLine("Failed with {0} incorrect answers. Incorrect answers are: {1} ", falselyAnswered, list);
        }
        Console.ReadLine();
    }

if (list == "") then list = i + 1 else list += ',' + i + 1; . — Shan
– Shan, Commented Mar 28, 2017 at 18:56

Dmitrii Bychenko · Accepted Answer · 2017-03-28 18:58:41Z

1

Your are looking for string.Join (and Linq):

using System.Linq;

... 

string list = string.Join(", ", studentAnswers
  .Where((answer, i) => answer != InputOutput_v1.GetValidChar("Question ", i))
  .Select((answer, i) => i + 1));

answered Mar 28, 2017 at 18:58

Dmitrii Bychenko

188k20 gold badges178 silver badges231 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Samvel Petrosov · Accepted Answer · 2017-03-28 19:00:23Z

0

Replace

list+= i+1;
Console.WriteLine("Failed with {0} incorrect answers. Incorrect answers are: {1} ", falselyAnswered, list);

to this

list+= i+1 + ",";
Console.WriteLine("Failed with {0} incorrect answers. Incorrect answers are: {1} ", falselyAnswered, list.Remove(list.Length-2));

answered Mar 28, 2017 at 19:00

Samvel Petrosov

7,7222 gold badges24 silver badges49 bronze badges

2 Comments

user4298687 Over a year ago

How about the last comma at the end? how can i get rid of that?

Samvel Petrosov Over a year ago

@IvyKnm list.Remove(list.Length-2) removes last come at the end

Collectives™ on Stack Overflow

C# how to add comma in output

2 Answers 2

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related