Есть ли более элегантный способ безопасного добавления элемента в словарь <>?

Question

Мне нужно добавить пары ключ / объект в словарь, но мне, конечно, нужно сначала проверить, существует ли ключ, в противном случае я получаю « ключ, который уже существует в словаре ». Приведенный ниже код решает эту проблему, но неуклюже.

Какой более элегантный способ сделать это без использования метода вспомогательного ввода строк, подобного этому?

using System;
using System.Collections.Generic;

namespace TestDictStringObject
{
    class Program
    {
        static void Main(string[] args)
        {
            Dictionary<string, object> currentViews = new Dictionary<string, object>();

            StringHelpers.SafeDictionaryAdd(currentViews, "Customers", "view1");
            StringHelpers.SafeDictionaryAdd(currentViews, "Customers", "view2");
            StringHelpers.SafeDictionaryAdd(currentViews, "Employees", "view1");
            StringHelpers.SafeDictionaryAdd(currentViews, "Reports", "view1");

            foreach (KeyValuePair<string, object> pair in currentViews)
            {
                Console.WriteLine("{0} {1}", pair.Key, pair.Value);
            }
            Console.ReadLine();
        }
    }

    public static class StringHelpers
    {
        public static void SafeDictionaryAdd(Dictionary<string, object> dict, string key, object view)
        {
            if (!dict.ContainsKey(key))
            {
                dict.Add(key, view);
            }
            else
            {
                dict[key] = view;
            }
        }
    }
}

Answer 1

Просто используйте индексатор - он перезапишет, если он уже там, но у него нет , чтобы быть там первым:

Dictionary<string, object> currentViews = new Dictionary<string, object>();
currentViews["Customers"] = "view1";
currentViews["Customers"] = "view2";
currentViews["Employees"] = "view1";
currentViews["Reports"] = "view1";

В основном используйте Add, если существуетключа указывает на ошибку (так что вы хотите, чтобы он выкинул) и индексатор в противном случае. (Это немного похоже на разницу между приведением и использованием as для преобразования ссылок.)

Если вы используете C # 3 , и у вас есть отдельный набор ключей , вы можете сделатьэто даже аккуратнее:

var currentViews = new Dictionary<string, object>()
{
    { "Customers", "view2" },
    { "Employees", "view1" },
    { "Reports", "view1" },
};

Это не сработает в вашем случае, хотя, поскольку инициализаторы коллекции всегда используют Add, который выбрасывает вторую запись Customers.

Answer 2

Что не так с ...

dict[key] = view;

Он автоматически добавит ключ, если он не существует.

Answer 3

просто

dict[key] = view;

Из документации MSDN для Dictionary.Item

Значение, связанное с указанным ключом. Если указанный ключ не найден, операция get создает исключение KeyNotFoundException, а операция set создает новый элемент с указанным ключом .

Мое выделение

Answer 4

Как обычно, Джон Скит входит туда со скоростью освещения с правильным ответом, но, что интересно, вы могли бы также написать свой SafeAdd как метод расширения для IDictionary.

public static void SafeAdd(this IDictionary<K, T>. dict, K key, T value)...

Answer 5

Хотя использование индексатора, безусловно, является правильным ответом для вашей конкретной проблемы, другим более общим решением проблемы добавления дополнительных функций к существующему типу было бы определение метода расширения.

Очевидно, что это не так 'Это особенно полезный пример, но есть кое-что, о чем следует помнить в следующий раз, когда вы обнаружите реальную потребность:

public static class DictionaryExtensions
{
    public static void SafeAdd<TKey, TValue>(this Dictionary<TKey, TValue> dict, 
                                             TKey key, TValue value)
    {
        dict[key] = value;
    }
}