Объединение однотипных Dictionary

[OmegaBerkut]

Здравствуйте. Есть два словарика, у обоих типы данных - Dictionary<UnitType,int>.
UnitType - это enum.
Важное условие: в одном словаре может присутствовать ключ, который в другом словаре может отсутствовать. Так же - нулевое значение для какого либо ключа недопустимо.
Нужно организовать полное слияние этих словарей по закону:
к значению каждого ключа первого словаря добавить количество этого же ключа во втором словаре; если в первом или втором словаре отсутствует какой либо ключ, но присутствует в одном из этих словарей - ключ с количеством должен быть в результате слияния.

Это нужно реализовать только руками ? Иль уже есть готовый метод ?

[OmegaBerkut]

Что то вроде соорудил, но нет возможности смоделировать ситуацию, в которой эта функция понадобится. Долго объяснять.
Помогите пожалуйста проверить правильность решения поставленной задачи.

static Dictionary<UnitType,int> FusionMilitary(Dictionary<UnitType,int> first, Dictionary<UnitType,int> second)
{
	UnitType[] types=new UnitType[] { UnitType.Cavalry, UnitType.Infantry, UnitType.Militia, UnitType.Ranged };
	int i=0,currentSum,tmp;
	Dictionary<UnitType,int> result=new Dictionary<UnitType,int>();
	for (;i<types.Length;i++)
	{
		currentSum=0;
		if (first.TryGetValue(types[i],out tmp))
			currentSum+=tmp;
		else {}
		if (second.TryGetValue(types[i],out tmp))
			currentSum+=tmp;
		else {}
		if (currentSum>0)
			result.Add(types[i],currentSum);
		else {}
	}
	types=null;
	return result;
}

[hoolygan]

1. Зачем пустые блоки else {}?
2. Обнулять types необязательно - при выходе из метода, когда система посчитает это допустимым - сборщик сам удалит объект.
А в остальном - почему бы и нет.

[OmegaBerkut]

Сообщение от hoolygan

1. Зачем пустые блоки else {}?

Это мой code-style; с практической точки зрения - при большом количестве вложений могу запутаться в разветвлениях. Выход - ввести в привычку писать все части конструкций.

Сообщение от hoolygan

2. Обнулять types необязательно

Я знаю про сборщик; опять же - вопрос привычки ... На плюсах сборкой мусора никто не занимается, не хочу отвыкать.

[hoolygan]

Ну как бы Dispose() и null это совершенно разные вещи, а разве на плюсах обнуление ссылки высвобождает память?

[OmegaBerkut]

hoolygan, высвобождение ссылки не освобождает память ...
Яж говорю, тут чисто вопрос привычки, что бы я не забывал подчищать за собой.
Я делаю Dispose() там, где он есть. Конкретно здесь его нет.
И, конкретно в этой ситуации для высвобождения памяти массива на плюсах нужно было бы написать delete[] types.

[Пепел Феникса]

Сообщение от OmegaBerkut

практической точки зрения - при большом количестве вложений могу запутаться в разветвлениях.

такое может произойти если только забивать на офф гайдлайны или писать в блокноте(и то сложно).
хотя насколько я помню, с вами на этот счет разговаривать бесполезно.

а чего бы не так?
работает на любом количестве словарей.

static Dictionary<UnitType,int> FusionMilitary(params Dictionary<UnitType,int>[] dicts)
{
   var result=new Dictionary<UnitType,int>();
   foreach(var dict in dicts)
   {
      int curCount;
      if(result.TryGetValue(dict.Key, out curCount))
          result[dict.Key]=curCount+dict.Value;
      else
          result[dict.Key]=dict.Value;
   }
}

var combined=FusionMilitary(first,second,third)

Сообщение от OmegaBerkut

опять же - вопрос привычки ...

советую избавляться от привычки писать ненужный код.
С++ это С++, C# это C#.
я пишу не на одном языке, и мне ж как-то это не мешает.

[pu4koff]

var a = new Dictionary<string, int>();
            var b = new Dictionary<string, int>();

            a.Add("a", 1);
            a.Add("b", 2);
            a.Add("d", 3);
            a.Add("e", 5);

            b.Add("a", 4);
            b.Add("b", 3);
            b.Add("c", 5);
            b.Add("d", 2);

            var c = a.Concat(b).GroupBy(item => item.Key).Select(sumitem => new KeyValuePair<string, int>(sumitem.Key, sumitem.Sum(item => item.Value))).ToDictionary(x => x.Key, x => x.Value);

[OmegaBerkut]

Сообщение от Пепел Феникса

забивать на офф гайдлайны

Это про меня.

Сообщение от Пепел Феникса

я пишу не на одном языке, и мне ж как-то это не мешает

В этом и проблема. Вы - это вы. Я - это я.
Со мной разговаривать об этом бесполезно.