Loading...

Във форума е въведено ограничение, което позволява на потребителите единствено да разглеждат публикуваните въпроси.

NikolayNeykov92 avatar NikolayNeykov92 617 Точки
Best Answer

Нямаш проверка за следния случай

(!sidesAndUsers.ContainsKey(side)&&sidesAndUsers.Values.Any(x => x.Contains(user)))

Ето поправка:

using System;
using System.Collections.Generic;
using System.Linq;
namespace Pb4ForceBook
{
    class Program
    {
        static void Main(string[] args)
        {
            var sidesAndUsers = new Dictionary<string, List<string>>();
            while (true)
            {
                string input = Console.ReadLine();
                if (input == "Lumpawaroo")
                {
                    break;
                }
                else if (input.Contains("|"))
                {
                    string[] inputLine = input.Split('|');
                    string side = inputLine[0].Trim();
                    string user = inputLine[1].Trim();
                    if (!sidesAndUsers.ContainsKey(side))
                    {
                        sidesAndUsers[side] = new List<string>();
                        if (!sidesAndUsers.Values.Any(x => x.Contains(user)))
                        {
                            sidesAndUsers[side].Add(user);
                        }
                    }
                    else if (!sidesAndUsers.Values.Any(x => x.Contains(user)))
                    {
                        sidesAndUsers[side].Add(user);
                    }
                }
                else if (input.Contains("->"))
                {
                    string[] inputLine = input.Split(new[] { "->" },StringSplitOptions.RemoveEmptyEntries);
                    string side = inputLine[1].Trim();
                    string user = inputLine[0].Trim();
                    if (!sidesAndUsers.ContainsKey(side))
                    {
                        sidesAndUsers[side] = new List<string>();
                        if (!sidesAndUsers.Values.Any(x => x.Contains(user)))
                        {
                            sidesAndUsers[side].Add(user);
                            Console.WriteLine($"{user} joins the {side} side!");
                        }
                        else
                        {
                            foreach (var userSide in sidesAndUsers)
                            {
                                if (userSide.Value.Contains(user))
                                {
                                    userSide.Value.Remove(user);
                                    sidesAndUsers[side].Add(user);
                                    Console.WriteLine($"{user} joins the {side} side!");
                                    break;
                                }
                            }
                        }
                    }
                    else if (!sidesAndUsers.Values.Any(x => x.Contains(user)))
                    {
                        sidesAndUsers[side].Add(user);
                        Console.WriteLine($"{user} joins the {side} side!");
                    }
                    else
                    {
                        foreach (var userSide in sidesAndUsers)
                        {
                            if (userSide.Value.Contains(user))
                            {
                                userSide.Value.Remove(user);
                                sidesAndUsers[side].Add(user);
                                Console.WriteLine($"{user} joins the {side} side!");
                                break;
                            }
                        }
                    }
                }
            }
            foreach (var forceSide in sidesAndUsers.OrderByDescending(count => count.Value.Count).ThenBy(nameSide => nameSide.Key))
            {
                if (forceSide.Value.Count > 0)
                {
                    Console.WriteLine($"Side: {forceSide.Key}, Members: {forceSide.Value.Count}");
                    foreach (var user in forceSide.Value.OrderBy(name => name))
                    {
                        Console.WriteLine("! " + user);
                    }
                }
            }
        }
    }
}

 

BTW имаш само 2 случая 

!sidesAndUsers.Values.Any(x => x.Contains(user) и sidesAndUsers.Values.Any(x => x.Contains(user) (т.е. if/else)

и няма нужда от толкова проверки и повтарящ се код.

0
27/06/2018 10:31:29
BigNikO avatar BigNikO 3 Точки

Благодаря, доста разсеяно съм го пропуснал :)

0
Можем ли да използваме бисквитки?
Ние използваме бисквитки и подобни технологии, за да предоставим нашите услуги. Можете да се съгласите с всички или част от тях.
Назад
Функционални
Използваме бисквитки и подобни технологии, за да предоставим нашите услуги. Използваме „сесийни“ бисквитки, за да Ви идентифицираме временно. Те се пазят само по време на активната употреба на услугите ни. След излизане от приложението, затваряне на браузъра или мобилното устройство, данните се трият. Използваме бисквитки, за да предоставим опцията „Запомни Ме“, която Ви позволява да използвате нашите услуги без да предоставяте потребителско име и парола. Допълнително е възможно да използваме бисквитки за да съхраняваме различни малки настройки, като избор на езика, позиции на менюта и персонализирано съдържание. Използваме бисквитки и за измерване на маркетинговите ни усилия.
Рекламни
Използваме бисквитки, за да измерваме маркетинг ефективността ни, броене на посещения, както и за проследяването дали дадено електронно писмо е било отворено.