主页 / 问题 / 572077
Accepted
XelaNimed
Asked:2020-09-30 16:35:06 +0000 UTC2020-09-30 16:35:06 +0000 UTC

通过文本值从枚举中提取常量

  • 772

有以下方法，它enum通过其字符串表示形式返回一个命名常量，或者在失败时返回一个默认值：

static TEnum GetEnum<TEnum>(string key, TEnum defaultValue = default(TEnum), bool ignoreCase = true) 
where TEnum : struct, IConvertible {
    TEnum result;
    return Enum.TryParse<TEnum>(key, ignoreCase, out result) ? result : defaultValue;
}

问题：为什么下面的代码会为一个不存在的常量返回错误的结果？如何解决？

enum Nums { none = 0, one, two, three, four, five }

Nums res;

res = GetEnum<Nums>("one");
Console.WriteLine($"{res} : {res == Nums.one}");   // one : True
res = GetEnum<Nums>("six");
Console.WriteLine($"{res} : {res == Nums.none}");  // none : True
res = GetEnum<Nums>("123");
Console.WriteLine($"{res} : {res == Nums.none}");  // 123 : False ???

我假设这是由于 string 的数字解释而发生的123，但仍然不清楚为什么会发生这种情况以及如何解决它？

c#

3 个回答

  1. Best Answer

    这种奇怪的行为被记录在案：-\

    如果value是一个不对应于 的命名常量的名称TEnum，则该方法返回false。如果value是不表示TEnum枚举的基础值的整数的字符串表示形式，则该方法返回一个枚举成员，其基础值是转换为整数类型的值。如果不需要此行为，请调用该IsDefined方法以确保整数的特定字符串表示实际上是 的成员TEnum。

    看起来您需要像这样重写函数：

    static TEnum GetEnum<TEnum>(
    string key,
    TEnum defaultValue = default(TEnum),
    bool ignoreCase = true)
        where TEnum : struct, IConvertible
{
    TEnum result;
    var b = Enum.TryParse<TEnum>(key, ignoreCase, out result) &&
              Enum.IsDefined(typeof(TEnum), result);
    return b ? result : defaultValue;
}

    更新：修改后的函数未通过字符串"1"和的测试"one,one"。所以看起来需要更多检查：

    static TEnum GetEnum<TEnum>(
    string key,
    TEnum defaultValue = default(TEnum),
    bool ignoreCase = true)
        where TEnum : struct, IConvertible
{
    TEnum result;
    var b =
        Enum.TryParse<TEnum>(key, ignoreCase, out result) &&
        Enum.IsDefined(typeof(TEnum), result) &&
        string.Equals(result.ToString(), key, StringComparison.OrdinalIgnoreCase);
    return b ? result : defaultValue;
}

    更新：上面的代码没有处理GetEnum<System.Windows.Forms.MessageBoxIcon>("Error")，因为里面的常量System.Windows.Forms.MessageBoxIcon是重复的！作为解决方案，我们可以查询所有名称的列表并进行比较。最终版本：

    static TEnum GetEnum<TEnum>(
    string key,
    TEnum defaultValue = default(TEnum),
    bool ignoreCase = true)
        where TEnum : struct, IConvertible
{
    TEnum result;
    if (Enum.GetNames(typeof(TEnum))
            .Contains(key, ignoreCase ? StringComparer.OrdinalIgnoreCase :
                                        StringComparer.Ordinal) &&
        Enum.TryParse<TEnum>(key, ignoreCase, out result))
        return result;
    else       
        return defaultValue;
}

    [你可以通过支票逃脱Enum.IsDefined(typeof(TEnum), key)（不是result！），但不可能要求忽略其中的大小写。]

    更新：作为评论中讨论的结果，您可以使用不那么漂亮但更高效的代码：

    static TEnum GetEnum<TEnum>(
    string key,
    TEnum defaultValue = default(TEnum),
    bool ignoreCase = true)
        where TEnum : struct, IConvertible
{
    var comparisonType = ignoreCase ? StringComparison.OrdinalIgnoreCase :
                                      StringComparison.Ordinal;
    var names = Enum.GetNames(typeof(TEnum));
    foreach (var name in names)
    {
        if (name.Equals(key, comparisonType))
            return (TEnum)Enum.Parse(typeof(TEnum), key, ignoreCase);
    }
    return defaultValue;
}

    它的本质是一样的，只是一些操作被“画”出来了。评论中的基准。

    如果您需要极端的微优化并了解缓存问题，这是 @Pavel Mayorov 提出的一个更快的缓存选项：

    private class Holder<TEnum>
{
    // относительный порядок элементов в массивах гарантирован документацией
    public static readonly TEnum[] values = (TEnum[])Enum.GetValues(typeof(TEnum));
    public static readonly string[] names = Enum.GetNames(typeof(TEnum));
}

static TEnum GetEnum<TEnum>(
    string key,
    TEnum defaultValue = default(TEnum),
    bool ignoreCase = true)
        where TEnum : struct, IConvertible
{
    var comparisonType = ignoreCase ? StringComparison.OrdinalIgnoreCase :
                                      StringComparison.Ordinal;
    for (int i = 0; i < Holder<TEnum>.names.Length; i++)
    {
        if (Holder<TEnum>.names[i].Equals(key, comparisonType))
            return Holder<TEnum>.values[i];
    }
    return defaultValue;
}
    • 9

  2. 这是因为里面 TryParse有两个分支：

    • 如果传递的字符串确实是一个字符串值（不是数字），那么该方法会诚实地将它与 enam 中定义的所有成员名称进行比较。
    • 如果传递的字符串结果是一个数字，则该方法返回 enam 值，该值与指定的数字相匹配。但是，如果 enam 中没有具有该值的成员，则返回数字本身。

    因此，如果您需要从字符串中解析 enam，而该字符串可以同时包含 enam 成员的名称及其值，那么您需要另外使用IsDefined().

    • 5

  3. 这是由于数字 123 不在集合中，并且由于 123 不等于 0，因此结果为 False。尝试指定 0 而不是 123，那么结果将为 True。

    • -1

相关问题