C＃枚举标志比较

Question

鉴于以下标志，

[Flags] 
    public enum Operations 
    { 
     add = 1, 
     subtract = 2, 
     multiply = 4, 
     divide = 8, 
     eval = 16, 
    } 

我怎么能实现一个IF条件进行各项操作？在我的尝试中，第一个条件是加，eval，这是正确的。然而，第一个条件也是如此，减去，评估，这是不正确的。

 public double Evaluate(double input) 
    { 
     if ((operation & (Operations.add & Operations.eval)) == (Operations.add & Operations.eval)) 
      currentResult += input; 
     else if ((operation & (Operations.subtract & Operations.eval)) == (Operations.subtract & Operations.eval)) 
      currentResult -= input; 
     else 
      currentResult = input; 

     operation = null; 

     return currentResult; 
    } 

我看不出是什么问题。

Answer 1

改变内心&到|：

if ((operation & (Operations.add | Operations.eval)) == (Operations.add | Operations.eval)) 

这相当于：

if(((operation & Operations.add)==Operations.add) && 
    ((operation & Operations.eval)==Operations.eval)) 

这可能是更具可读性。您可能还需要考虑这样的一个扩展：

public static bool HasFlag(this Operations op, Operations checkflag) 
{ 
    return (op & checkflag)==checkflag; 
} 

，那么你可以这样做：

if(operation.HasFlag(Operations.add) && Operations.HasFlag(Operations.eval)) 

，这可能是更具有可读性。最后，你可以更有趣创建此扩展名：

public static bool HasAllFlags(this Operations op, params Operations[] checkflags) 
{ 
    foreach(Operations checkflag in checkflags) 
    { 
     if((op & checkflag)!=checkflag) 
      return false; 
    } 
    return true; 
} 

那么你的表达可能变成：

if(operation.HasAllFlags(Operations.add, Operations.eval)) 

Answer 2

你的操作失败的原因是因为你有错误的表达。 (Operations.add & Operations.eval)是总是零。你的第一个比较的左侧和右侧都是零。试试这个 - 我怀疑这就是你以后：

public double Evaluate(double input) 
{ 
    if ((operation & (Operations.add | Operations.eval)) == (Operations.add | Operations.eval)) 
     currentResult += input; 
    else if ((operation & (Operations.subtract | Operations.eval)) == (Operations.subtract | Operations.eval)) 
     currentResult -= input; 
    else 
     currentResult = input; 

    operation = null; 

    return currentResult; 
} 

Answer 3

试试这个：

public double Evaluate(double input) 
{ 
    if ((operation & (Operations.add | Operations.eval)) == (Operations.add | Operations.eval)) 
     currentResult += input; 
    else if ((operation & (Operations.subtract | Operations.eval)) == (Operations.subtract | Operations.eval)) 
     currentResult -= input; 
    else 
     currentResult = input; 

    operation = null; 

    return currentResult; 
} 

Answer 4

哇，我真不敢相信所有的错误答案..

很重要了解按位数学计算，如果你正在使用标志。你的情况，你有以下的（第一个条件）：

1 in binary is 00001 
16 in binary is 10000 

    00001 
& 10000 
-------- 
    00000 

所以，说我们有减（2）为operation

2 in binary is  00010 
previous result is 00000 

    00010 
& 00000 
-------- 
    00000 

自上次结果为00000什么，”与它一起将是零。所以你的病情总是会从0 == 0开始计算到true。

如果我们只是此开关OR，那么我们有以下几点：

1 in binary is 00001 
16 in binary is 10000 

    00001 
| 10000 
-------- 
    10001 (17) 

现在，假设我们有最终Add (1)为operation

1 in binary is  00001 
previous result is 10001 (17) 

    00001 
& 10001 
-------- 
    00001 

所以，1 & 17 => 1从而条件是(1 & (1 | 16)) == (1 | 16) =>1 & 17 == 17 =>1 == 17 =>false（仍然是假的！）

所以，你真正想要的是：

((operation | Operations.add | Operations.eval) & (Operations.add | Operations.eval)) == (Operations.add | Operations.eval) 

这成为((1 | 1 | 16) & (1 | 16)) == (1 | 16) =>(17 & 17) == 17 =>17 == 17 == true

这显然不是可读的，所以你应该选择提取到本一种方法（如建议）。但它仍然很重要明白为什么你的情况是不正确的。

Answer 5

重复：

参见How to Compare Flags in C#?

C＃4介绍了Enum.HasFlags()方法。

但是，如果性能是一个问题，请参阅What is it that makes Enum.HasFlag so slow?。