Как я могу облегчить боль при инициализации словарей списков в C #? - PullRequest
Новая
       12

Как я могу облегчить боль при инициализации словарей списков в C #?

5 голосов
/ 15 мая 2009

Я часто использую такую ​​структуру: 

Dictionary<string, List<string>> Foo = new Dictionary<string, List<string>>();

Что приводит к такому виду кода: 

foreach (DataRow dr in ds.Tables[0].Rows)
{
    List<string> bar;
    if (!Foo.TryGetValue(dr["Key"].ToString(), out desks))
    {
        bar= new List<string>();
        Foo.Add(dr["Key"].ToString(), bar);
    }
    bar.Add(dr["Value"].ToString());
}

Как вы думаете, стоит ли писать собственный класс DictionaryOfList, который бы автоматически обрабатывал подобные вещи?

Есть ли другой способ ленивой инициализации этих списков?

Ответы [ 6 ]

7 голосов
/ 15 мая 2009

Вы можете написать метод расширения - GetValueOrCreateDefault () или что-то в этом роде: 

foreach (DataRow dr in ds.Tables[0].Rows)
{
    Foo.GetValueOrCreateDefault( dr["Key"] ).Add( dr["Value"].ToString() )
}

Может быть, вы даже можете написать метод расширения для всей инициализации?

4 голосов
/ 15 мая 2009

Словарь списка ... в .NET 3.5, который будет ILookup<TKey,TValue>. Реализация по умолчанию (Lookup<TKey,TValue>) неизменна, но я написал EditableLookup<TKey,TValue> для MiscUtil . Это будет намного проще в использовании - т.е. 

var data = new EditableLookup<string, int>();
data.Add("abc",123);
data.Add("def",456);
data.Add("abc",789);

foreach(int i in data["abc"]) {
    Console.WriteLine(i); // 123 & 789
}

Кроме этого, метод расширения: 

public static void Add<TKey, TList, TValue>(
    this IDictionary<TKey, TList> lookup,
    TKey key, TValue value)
    where TList : class, ICollection<TValue>, new()
{
    TList list;
    if (!lookup.TryGetValue(key, out list))
    {
        lookup.Add(key, list = new TList());
    }
    list.Add(value);
}

static void Main() {
    var data = new Dictionary<string, List<string>>();
    data.Add("abc", "def");
}
2 голосов
/ 15 мая 2009

Я думаю, что следующее должно сделать: 

class DictionaryOfList : Dictionary<string, List<string>> {}
  • Правка Я должен читать более правильно. Это не отвечает на вопрос. Tanascius предоставил изящный способ решить эту проблему.
1 голос
/ 15 мая 2009

Добавьте ссылку на System.Data.DataSetExtensions и вы можете использовать расширения Linq: 

var dictOfLst = ds.Tables[0].Rows.
    //group by the key field
    GroupBy( dr => dr.Field<string>("key") ).
    ToDictionary(
        grp => grp.Key,
        //convert the collection of rows into values
        grp => grp.Select( dr => dr.Field<string>("value") ).ToList() );

Я не уверен, что буду беспокоиться о другом классе, но утилита или метод расширения могут сделать это проще: 

public static Dictionary<TKey, List<TValue>> ToGroupedDictionary<TKey, List<TValue>>(
    this DataTable input, 
    Func<TKey, DataRow> keyConverter, 
    Func<TValue, DataRow> valueConverter )
{
    return input.Rows.
        //group by the key field
        GroupBy( keyConverter ).
        ToDictionary(
            grp => grp.Key,
            //convert the collection of rows into values
            grp => grp.Select( valueConverter ).ToList() );
}

//now you have a simpler syntax
var dictOfLst = ds.Tables[0].ToGroupedDictionary(
    dr => dr.Field<string>("key"),
    dr => dr.Field<string>("value") );
0 голосов
/ 17 мая 2009

Почему бы просто не упростить немного: 

foreach (DataRow dr in ds.Tables[0].Rows)
{
   string key = dr["Key"].ToString();
   if (!Foo.ContainsKey(key)) Foo.Add(key, new List<string>());
   Foo[key].Add(dr["Value"].ToString());
}
0 голосов
/ 16 мая 2009

Не забудьте директива об использовании .

Это не напрямую реагирует, но может быть полезно в любом случае. «Использование псевдонима» для универсального типа коллекции может сделать ваш код проще для глаз. 

using StoreBox = System.Collections.Generic.Dictionary<string, System.Collections.Generic.List<string>>; 
using ListOfStrings = System.Collections.Generic.List<string>; 
class Program
{
    static void Main(string[] args)
    {
      var b = new StoreBox ();
      b.Add("Red", new ListOfStrings {"Rosso", "red" });
      b.Add("Green", new ListOfStrings {"Verde", "green" });
    }
}

Кредит SO за эту подсказку.

...