Включить Enum (с атрибутом Flags), не объявляя все возможные комбинации? - PullRequest
Новая
       29

Включить Enum (с атрибутом Flags), не объявляя все возможные комбинации?

45 голосов
/ 25 июня 2009

как включить перечисление с установленным атрибутом 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 ((_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 Если условия каждый раз вместо "прыжка" только в нужный "регистр" / ', как при использовании переключателя.

Есть ли какое-то волшебное решение для решения этой проблемы?

Я подумал о возможности перебрать объявленные значения и затем использовать переключатель, тогда он будет нажимать на переключатель только для каждого объявленного значения, но я не знаю, как это будет работать, и если это снижение производительности является хорошая идея (по сравнению с большим количеством "если")?

Есть ли простой способ перебрать все объявленные значения перечисления?

Я могу только придумать использование ToString () и разделение на «,», а затем перебрать массив и проанализировать каждую строку.

UPDATE:

Я вижу, что я недостаточно хорошо объяснил. Мой пример прост (попытался упростить мой сценарий).

Я использую его для ActionMethodSelectorAttribute в Asp.net MVC, чтобы определить, должен ли метод быть доступен при разрешении URL / маршрута.

Я делаю это, объявляя что-то вроде этого в методе 

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

Это будет означать, что он должен проверить, есть ли у формы или TempData ключ, указанный для того, чтобы метод был доступен.

Методы, которые он будет вызывать (doSomething (), doSomethingElse () и doWhwhat () в моем предыдущем примере), на самом деле будут иметь значение bool в качестве возвращаемого значения и будут вызываться с параметром (разные коллекции, не имеющие общего интерфейса это можно использовать - см. мой пример кода в ссылке ниже и т. д.).

Надеюсь, чтобы дать лучшее представление о том, что я делаю, я привел простой пример того, что я на самом деле делаю на pastebin - его можно найти здесь http://pastebin.com/m478cc2b8

Ответы [ 8 ]

45 голосов
/ 07 июля 2009

Как насчет этого. Конечно, аргументы и возвращаемые типы 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;
                }
            }
        }
    }
}
13 голосов
/ 25 июня 2009

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

РЕДАКТИРОВАТЬ: Конечно, мы могли бы сделать этот код более компактным за счет использования LINQ и некоторых вспомогательных функций, но я думаю, что это легче понять в менее сложной форме. Это может быть случай, когда ремонтопригодность превосходит элегантность.

Вот пример: 

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

void PerformActions( CheckType c )
{
  // array of bits set in the parameter {c}
  bool[] actionMask = { false, false, false };
  // array of delegates to the corresponding actions we can invoke...
  Action availableActions = { DoSomething, DoSomethingElse, DoAnotherThing };

  // disassemble the flags into a array of booleans
  for( int i = 0; i < actionMask.Length; i++ )
    actionMask[i] = (c & (1 << i)) != 0;

  // for each set flag, dispatch the corresponding action method
  for( int actionIndex = 0; actionIndex < actionMask.Length; actionIndex++ )
  {
      if( actionMask[actionIndex])
          availableActions[actionIndex](); // invoke the corresponding action
  }
}

В качестве альтернативы, если порядок, в котором вы оцениваете, не имеет значения, здесь есть более простое и ясное решение, которое работает так же хорошо. Если порядок имеет значение, замените операции сдвига битов массивом, содержащим флаги в том порядке, в котором вы хотите их оценить: 

int flagMask = 1 << 31; // start with high-order bit...
while( flagMask != 0 )   // loop terminates once all flags have been compared
{
  // switch on only a single bit...
  switch( theCheckType & flagMask )
  {
   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;
  }

  flagMask >>= 1;  // bit-shift the flag value one bit to the right
}
7 голосов
/ 12 декабря 2016

Просто используйте HasFlag 

if(theCheckType.HasFlag(CheckType.Form)) DoSomething(...);
if(theCheckType.HasFlag(CheckType.QueryString)) DoSomethingElse(...);
if(theCheckType.HasFlag(CheckType.TempData)) DoWhatever(...);
4 голосов
/ 25 июня 2009

Как насчет 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]();
}

(код не проверен)

3 голосов
/ 06 июня 2018

С помощью 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;
    }
}
1 голос
/ 25 июня 2009

Исходя из ваших правок и вашего реального кода, я бы, вероятно, обновил метод 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;
}
0 голосов
/ 12 сентября 2018

Должно быть возможно в C # 7 

switch (t1)
    {
        case var t when t.HasFlag(TST.M1):
            {
                break;
            }
        case var t when t.HasFlag(TST.M2):
            {
                break;
            }
0 голосов
/ 04 ноября 2017

Самый простой способ - просто выполнить перечисление 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;
}

Намного чище, и вам не нужно использовать оператор if с HasFlag. Вы можете создавать любые комбинации, какие хотите, а затем легко читать оператор switch.

Я бы порекомендовал разбить ваш enums, попробуйте посмотреть, не смешиваете ли вы разные вещи в одном enum. Вы можете установить несколько enums, чтобы уменьшить количество дел.

...