[Programming Fundamentals] Lists more exercises / Distinct list - only getting 20/100

DadiEinarsson 21 Точки

[Programming Fundamentals] Lists more exercises / Distinct list - only getting 20/100

I am having a problem with the first task in "Lists more exercises". In it the program is to delete all repeating values while keeping the first instance of the number in the list. All of my tests come back with everything working fine and the zero tests are fine in judge but the score I get is just 20/100. My code is in the link below. Could someone take a look and see where my mistake is or at least point me in the right direction?

Thanks in advance.

https://pastebin.com/ZuMUa7du

Тагове:

27/06/2017 16:43:33 Programming Fundamentals 29/06/2017 20:41:47

Виж всички отговори

melik 285 Точки

Best Answer

Hello, DadiEinarsson.

While I was checking your code I've come to the conclusion that you're using unnecessary for loops. Basically, the solution of this problem is 2 lines of code, you need one list in order to read the input and second list in order to add the final result in it. Then, we can create foreach loop and distinct the numbers. For example: foreach (var distinctedNumbers in numbers.Distinct() and then we should add the result in our empty list which we have created earlier.

The solution should look like this - https://github.com/melikpehlivanov/Programming-Fundamentals-CSharp/blob/master/More%20Exercises%20-%20Lists/1.%20Distinct%20List/Program.cs.

I hope this will help you to understand where you're mistaking in the code and if it didn't help you please let me know and I will explain it once again!

Kind regards,

Melik.

27/06/2017 17:20:21 27/06/2017 20:10:17

DadiEinarsson 21 Точки

Thanks melik, I had totally overlooked the Distinct method.

28/06/2017 08:32:54

kolioi 641 Точки

You don't even need the foreach() loop, just write

var numbers = Console.ReadLine().Split().Select(int.Parse).Distinct();
Console.WriteLine(string.Join(" ", numbers));

and it works like a charm

28/06/2017 11:15:23 28/06/2017 11:33:45

Виж всички отговори