c# - 启用Enum(带Flags属性)而不声明每个可能的组合?




enums switch-statement (6)

你将填写的Dictionary<CheckType,Action>怎么样?

dict.Add(CheckType.Form, DoSomething);
dict.Add(CheckType.TempDate, DoSomethingElse);
...

分解你的价值

flags = Enum.GetValues(typeof(CheckType)).Where(e => (value & (CheckType)e) == (CheckType)e).Cast<CheckType>();

然后

foreach (var flag in flags)
{
   if (dict.ContainsKey(flag)) dict[flag]();
}

(代码未经测试)

我如何打开一个设置了flags属性的枚举(或更精确地用于位操作)?

我希望能够在与所声明的值匹配的开关中击中所有情况。

问题是,如果我有以下枚举

[Flags()]public enum CheckType
{
    Form = 1,   
    QueryString = 2,
    TempData = 4,
}

我想用这样的开关

switch(theCheckType)
{
   case CheckType.Form:
       DoSomething(/*Some type of collection is passed */);
       break;

   case CheckType.QueryString:
       DoSomethingElse(/*Some other type of collection is passed */);
       break;

   case CheckType.TempData
       DoWhatever(/*Some different type of collection is passed */);
       break;
}

如果“theCheckType”设置为CheckType.Form | CheckType.TempData我希望它能同时击中两个案例。 显然,由于中断,它不会在我的示例中同时出现,但除此之外它也会失败,因为CheckType.Form不等于CheckType.Form | CheckType.TempData

那么我能看到的唯一解决方案就是为每个可能的枚举值组合做一个案例?

就像是

    case CheckType.Form | CheckType.TempData:
        DoSomething(/*Some type of collection is passed */);
        DoWhatever(/*Some different type of collection is passed */);
        break;

    case CheckType.Form | CheckType.TempData | CheckType.QueryString:
        DoSomething(/*Some type of collection is passed */);
        DoSomethingElse(/*Some other type of collection is passed */);
        break;

... and so on...

但这真的不是很理想(因为它会很快变大)

现在我有另外3个If条件

就像是

if ((_CheckType & CheckType.Form) != 0)
{
    DoSomething(/*Some type of collection is passed */);
}

if ((_CheckType & CheckType.TempData) != 0)
{
    DoWhatever(/*Some type of collection is passed */);
}

....

但这也意味着,如果我有一个包含20个值的枚举,它必须每次都经过20个If条件,而不是像使用开关那样“跳转”到所需的“case”/。

是否有一些神奇的解决方案来解决这个问题?

我已经考虑过循环声明值然后使用开关的可能性,然后它只会触发声明的每个值的开关,但我不知道它将如何工作以及它是否是一个好主意(相比很多if)?

是否有一种简单的方法来循环声明所有枚举值?

我只能使用ToString()并按“,”拆分,然后循环遍历数组并解析每个字符串。

更新:

我看到我没有做足够好的解释工作。 我的例子很简单(试图简化我的场景)。

我将它用于Asp.net MVC中的ActionMethodSelectorAttribute,以确定在解析url / route时是否应该有一个方法。

我通过在方法上声明类似的东西来做到这一点

[ActionSelectorKeyCondition(CheckType.Form | CheckType.TempData, "SomeKey")]
public ActionResult Index()
{
    return View();
} 

这意味着它应检查Form或TempData是否具有为可用方法指定的密钥。

它将调用的方法(doSomething(),doSomethingElse()和doWhatever()在我之前的例子中)实际上将bool作为返回值并将使用参数调用(不同的集合不共享可以是的接口)使用 - 请参阅下面链接中的示例代码等)。

为了更好地了解我在做什么,我已经粘贴了一个关于我在pastebin上实际做的简单示例 - 可以在这里找到http://pastebin.com/m478cc2b8


使用C#7,您现在可以编写如下内容:

public void Run(CheckType checkType)
{
    switch (checkType)
    {
        case var type when CheckType.Form == (type & CheckType.Form):
            DoSomething(/*Some type of collection is passed */);
            break;

        case var type when CheckType.QueryString == (type & CheckType.QueryString):
            DoSomethingElse(/*Some other type of collection is passed */);
            break;

        case var type when CheckType.TempData == (type & CheckType.TempData):
            DoWhatever(/*Some different type of collection is passed */);
            break;
    }
}

应该可以在C#7中使用

switch (t1)
    {
        case var t when t.HasFlag(TST.M1):
            {
                break;
            }
        case var t when t.HasFlag(TST.M2):
            {
                break;
            }

最简单的方法是只执行ORed枚举,在您的情况下,您可以执行以下操作:

[Flags()]public enum CheckType
{
    Form = 1,   
    QueryString = 2,
    TempData = 4,
    FormQueryString = Form | QueryString,
    QueryStringTempData = QueryString | TempData,
    All = FormQueryString | TempData
}

一旦你有了enum设置,它现在很容易执行你的switch语句。

例如,如果我设置了以下内容:

var chkType = CheckType.Form | CheckType.QueryString;

我可以使用以下switch语句如下:

switch(chkType){
 case CheckType.Form:
   // Have Form
 break;
 case CheckType.QueryString:
   // Have QueryString
 break;
 case CheckType.TempData:
  // Have TempData
 break;
 case CheckType.FormQueryString:
  // Have both Form and QueryString
 break;
 case CheckType.QueryStringTempData:
  // Have both QueryString and TempData
 break;
 case CheckType.All:
  // All bit options are set
 break;
}

更清洁,你不需要使用HasFlagif语句。 您可以进行任何所需的组合,然后使switch语句易于阅读。

我建议拆开你的enums ,尝试看看你是不是将不同的东西混合到同一个enum 。 您可以设置多个enums以减少案例数量。


根据您的编辑和现实代码,我可能会更新IsValidForRequest方法,看起来像这样:

public sealed override bool IsValidForRequest
    (ControllerContext cc, MethodInfo mi)
{
    _ControllerContext = cc;

    var map = new Dictionary<CheckType, Func<bool>>
        {
            { CheckType.Form, () => CheckForm(cc.HttpContext.Request.Form) },
            { CheckType.Parameter,
                () => CheckParameter(cc.HttpContext.Request.Params) },
            { CheckType.TempData, () => CheckTempData(cc.Controller.TempData) },
            { CheckType.RouteData, () => CheckRouteData(cc.RouteData.Values) }
        };

    foreach (var item in map)
    {
        if ((item.Key & _CheckType) == item.Key)
        {
            if (item.Value())
            {
                return true;
            }
        }
    }
    return false;
}

这个怎么样。 当然,DoSomething等的参数和返回类型可以是您喜欢的任何内容。

class Program
{
    [Flags]
    public enum CheckType
    {
        Form = 1,
        QueryString = 2,
        TempData = 4,
    }

    private static bool DoSomething(IEnumerable cln)
    {
        Console.WriteLine("DoSomething");
        return true;
    }

    private static bool DoSomethingElse(IEnumerable cln)
    {
        Console.WriteLine("DoSomethingElse");
        return true;
    }

    private static bool DoWhatever(IEnumerable cln)
    {
        Console.WriteLine("DoWhatever");
        return true;
    }

    static void Main(string[] args)
    {
        var theCheckType = CheckType.QueryString | CheckType.TempData;
        var checkTypeValues = Enum.GetValues(typeof(CheckType));
        foreach (CheckType value in checkTypeValues)
        {
            if ((theCheckType & value) == value)
            {
                switch (value)
                {
                    case CheckType.Form:
                        DoSomething(null);
                        break;
                    case CheckType.QueryString:
                        DoSomethingElse(null);
                        break;
                    case CheckType.TempData:
                        DoWhatever(null);
                        break;
                }
            }
        }
    }
}






bit