如何使这个Dictionary TryGetValue代码更具可读性？

Question

我想测试一个id是否尚未知道,或者,如果已知,则关联值是否已更改.我目前正在使用与此类似的代码,但对于那些不熟悉该模式的人来说很难理解.您是否可以想到一种方法,使其在LOC中保持简短时更具可读性？

string id;
string actual;
string stored;

if (!someDictionary.TryGetValue (id, out stored) || stored != actual) {
    // id not known yet or associated value changed.
}

Answer 1

您可以编写一个具有良好名称的扩展方法:

public static class Utility
{
    public static bool ValueChangedOrUnknown(this Dictionary<string, string> dictionary, string id, string actual)
    {
        string stored = null;
        return (!dictionary.TryGetValue(id, out actual) || stored != actual);
    }
}

以后你可以使用

string id;
string actual;

if (someDictionary.ValueChangedOrUnknown(id, actual) {
    // id not known yet or associated value changed.
}

Answer 2

它对我来说看起来不错...读起来和任何其他 2 个条件 if 语句一样简单。我唯一可能改变的就是翻转提前退出的否定：

if (someDictionary.TryGetValue(id, out stored) && stored == actual) {
    return;
}
// store new value

我根本没有看到它有任何混乱，从来没有认为它是一个特别麻烦的习惯用法，并谦虚地建议那些被它困惑的 C# 开发人员习惯它。它很常见、简洁，并且为问题提供了尽可能多的 LOC。把它变成10行代码就可以了太重要了。

如果我经常使用它，一个名为类似名称的扩展方法ContainsEqualValue将是合适的 - 但我会在扩展方法中使用与您完全相同的代码。