在一些遗留代码中,我看到了以下扩展方法,以方便添加一个新的键-值项或更新值,如果键已经存在。

方法-1(遗留代码)。

public static void CreateNewOrUpdateExisting<TKey, TValue>(
    this IDictionary<TKey, TValue> map, TKey key, TValue value)
{            
    if (map.ContainsKey(key))
    {
        map[key] = value;
    }
    else
    {
        map.Add(key, value);
    }
}

不过,我已经检查了map[key]=value做完全相同的工作。也就是说,这个方法可以替换为下面的method -2。

方法2。

public static void CreateNewOrUpdateExisting<TKey, TValue>(
    this IDictionary<TKey, TValue> map, TKey key, TValue value)
{
    map[key] = value;
}

现在,我的问题是…如果我把Method-1换成Method-2会有问题吗?它会在任何可能的情况下断裂吗?

另外,我认为这曾经是哈希表和字典之间的区别。HashTable允许更新项目,或使用索引器添加新项目,而Dictionary不允许!!这个差异在c# > 3.0版本中被消除了吗?

该方法的目的是如果用户再次发送相同的键值,该方法只需用新值更新条目,如果新的键值对已经发送给该方法,则创建一个新的条目。


当前回答

如果我把Method-1换成Method-2会有问题吗?

不,只需使用map[key] = value。这两个选项是等价的。

关于Dictionary<> vs. Hashtable:当启动Reflector时,可以看到两个类的索引器设置器都调用this。Insert(key, value, add: false);当插入重复的键时,add参数负责抛出异常。所以这两个类的行为是一样的。

其他回答

在功能上,它们是等价的。

性能方面map[key] = value会更快,因为您只进行一次查找而不是两次。

风格方面,越短越好:)

在大多数情况下,代码在多线程上下文中工作得很好。然而,如果没有额外的同步,它就不是线程安全的。

如果我把Method-1换成Method-2会有问题吗?

不,只需使用map[key] = value。这两个选项是等价的。

关于Dictionary<> vs. Hashtable:当启动Reflector时,可以看到两个类的索引器设置器都调用this。Insert(key, value, add: false);当插入重复的键时,add参数负责抛出异常。所以这两个类的行为是一样的。

老问题了,但我觉得我应该添加以下内容,甚至更多,因为在写这个问题的时候。net 4.0已经发布了。

从。net 4.0开始,有命名空间System.Collections.Concurrent,其中包括线程安全的集合。

收集System.Collections.Concurrent。ConcurrentDictionary<>正是您想要的。它具有AddOrUpdate()方法,具有线程安全的额外优势。

如果你在一个高性能的场景中,不处理多线程,已经给出的map[key] = value的答案会更快。

在大多数情况下,这种性能优势是微不足道的。如果是这样,我建议使用ConcurrentDictionary,因为:

它在框架中——它经过了更多的测试,而且你不是那个必须维护代码的人 它是可扩展的:如果你切换到多线程,你的代码已经准备好了

没有问题。我甚至会从源代码中删除CreateNewOrUpdateExisting,并直接在代码中使用map[key] = value,因为这是惯用的c#;c#开发人员通常知道map[key] = value表示添加或更新。

我知道它不是字典<TKey, TValue>类,但是你可以避免KeyNotFoundException而增加一个值,如:

dictionary[key]++; // throws `KeyNotFoundException` if there is no such key  

通过使用ConcurrentDictionary<TKey, TValue>及其非常好的方法AddOrUpdate()..

让我举个例子:

var str = "Hellooo!!!";
var characters = new ConcurrentDictionary<char, int>();
foreach (var ch in str)
    characters.AddOrUpdate(ch, 1, (k, v) => v + 1);