Oсвен за съхраняване на някакво състояние, класовете може да имат и методи. Например класа Player може да има метод, който да добавя или ъпдейтва position + skill, или друг метод, който да връща общия skill на играча. Това ще разтовари main метода от специфичната за Player логика. 

Добра практика при писането на класове е да скрием от ползвателите им всичко, което не е необходимо за работата с тях. Ако още не е засегната в курса тази тема, скоро ще бъде, но и сега може да потърсите и прочетете повече за енкапсулацията. 

Ето примерен вариант на Player класа:

private class Player
{
    public string Name { get; }

    private readonly Dictionary<string, int> _dictPositionSkill;

    public Player(string name)
    {
        Name = name;
        _dictPositionSkill = new Dictionary<string, int>();
    }

    public void AddOrUpdatePosition(string position, int skill)
    {
        if (!_dictPositionSkill.ContainsKey(position))
        {
            _dictPositionSkill.Add(position, skill);
        }
        else if (_dictPositionSkill[position] < skill)
        {
            _dictPositionSkill[position] = skill;
        }
    }

    public IEnumerable<string> GetPositions()
    {
        return _dictPositionSkill.Keys;
    }

    public ImmutableDictionary<string, int> GetPositionsAndSkillsDictionary()
    {
        return _dictPositionSkill.ToImmutableDictionary();
    }

    public int GetTotalSkillPoints()
    {
        return _dictPositionSkill.Values.Sum();
    }
}

След горните промени по Player класа, редове #21 - #41 може да бъдат заменени с:

if (!dictNamePlayer.ContainsKey(playerName))
{
    var newPlayer = new Player(playerName);
    dictNamePlayer.Add(playerName, newPlayer);
}

dictNamePlayer[playerName].AddOrUpdatePosition(position, skill);

А това е логиката за дуелите:

if (!dictNamePlayer.ContainsKey(firstPlayer) || !dictNamePlayer.ContainsKey(secondPlayer))
{
    continue;
}

var playerOne = dictNamePlayer[firstPlayer];
var playerTwo = dictNamePlayer[secondPlayer];

if (playerOne.GetTotalSkillPoints() == playerTwo.GetTotalSkillPoints())
{
    continue;
}

var duelsCount = playerOne.GetPositions()
    .Intersect(playerTwo.GetPositions())
    .Count();

if (duelsCount <= 0)
{
    continue;
}

dictNamePlayer.Remove(playerOne.GetTotalSkillPoints() > playerTwo.GetTotalSkillPoints()
    ? playerTwo.Name
    : playerOne.Name);

Това е цялото решение.

Можем да разроварим main метода от логиката за ъпдейтване на играчите и за дуелите, ако я изнесем в отделен клас - например PlayersPool, с два метода - решение.

Разбира се, може да се стигне и още по-далеч с абстракцията и дори да се прекали с нея, поне за просто задание като това.