如何在开关中使用枚举

我不知道如何将开关与枚举结合使用。你能告诉我我做错了什么，以及如何弥补吗？我必须使用枚举来制作一个基本的计算器。

public enum Operator
{
PLUS, MINUS, MULTIPLY, DIVIDE
}


public double Calculate(int left, int right, Operator op)
{


int i = (int) op;


switch(i)
{
case 0:
{
return left + right;
}


case 1:
{
return left - right;
}


case 2:
{
return left * right;
}


case 3:
{
return left / right;
}


default:
{
return 0.0;
}
}
}

最终的结果应该是这样的:

Console.WriteLine("The sum of 5 and 5 is " + Calculate(5, 5, PLUS))
Output: The sum of 5 and 5 is 10

你们能告诉我我怎么搞砸了吗？

282120 次浏览
小开
最佳答案

You don't need to convert it

switch(op)
{
case Operator.PLUS:
{
// your code
// for plus operator
break;
}
case Operator.MULTIPLY:
{
// your code
// for MULTIPLY operator
break;
}
default: break;
}

By the way, use brackets

小开

You should not cast to integer. And for the division, you need to cast left to double first, if not you will be doing an integer divide.

public enum Operator
{
PLUS, MINUS, MULTIPLY, DIVIDE
}


public double Calculate(int left, int right, Operator op)
{
double sum = 0.0;


switch(op)
{
case Operator.PLUS:
sum = left + right;
return sum;


case Operator.MINUS:
sum = left - right;
return sum;


case Operator.MULTIPLY:
sum = left * right;
return sum;


case Operator.DIVIDE:
sum = (double)left / right;
return sum;


default:
return sum;
}


return sum;
}
小开

simply don't cast to int

 switch(operator)
{
case Operator.Plus:
//todo
小开

All the other answers are correct, but you also need to call your method correctly:

Calculate(5, 5, Operator.PLUS))

And since you use int for left and right, the result will be int as well (3/2 will result in 1). you could cast to double before calculating the result or modify your parameters to accept double

小开 
 public enum Operator
{
PLUS, MINUS, MULTIPLY, DIVIDE
}


public class Calc
{
public void Calculate(int left, int right, Operator op)
{


switch (op)
{
case Operator.DIVIDE:
//Divide
break;
case Operator.MINUS:
//Minus
break;
case Operator.MULTIPLY:
//...
break;
case Operator.PLUS:
//;;
break;
default:
throw new InvalidOperationException("Couldn't process operation: " + op);
}
}
}
小开

Two things. First, you need to qualify the enum reference in your test - rather than "PLUS", it should be "Operator.PLUS". Second, this code would be a lot more readable if you used the enum member names rather than their integral values in the switch statement. I've updated your code:

public enum Operator
{
PLUS, MINUS, MULTIPLY, DIVIDE
}


public static double Calculate(int left, int right, Operator op)
{
switch (op)
{
default:
case Operator.PLUS:
return left + right;


case Operator.MINUS:
return left - right;


case Operator.MULTIPLY:
return left * right;


case Operator.DIVIDE:
return left / right;
}
}

Call this with:

Console.WriteLine("The sum of 5 and 5 is " + Calculate(5, 5, Operator.PLUS));
小开

The correct answer is already given, nevertheless here is the better way (than switch):

private Dictionary<Operator, Func<int, int, double>> operators =
new Dictionary<Operator, Func<int, int, double>>
{
{ Operator.PLUS, ( a, b ) => a + b },
{ Operator.MINUS, ( a, b ) => a - b },
{ Operator.MULTIPLY, ( a, b ) => a * b },
{ Operator.DIVIDE ( a, b ) => (double)a / b },
};


public double Calculate( int left, int right, Operator op )
{
return operators.ContainsKey( op ) ? operators[ op ]( left, right ) : 0.0;
}
小开

Your code is fine. In case you're not sure how to use Calculate function, try 

Calculate(5,5,(Operator)0); //this will add 5,5
Calculate(5,5,Operator.PLUS);// alternate

Default enum values start from 0 and increase by one for following elements, until you assign different values. Also you can do :

public enum Operator{PLUS=21,MINUS=345,MULTIPLY=98,DIVIDE=100};
小开

In case you don't want to use return statement for each case, try this:

Calculate(int left, int right, Operator op)
{
int result = 0;
switch(op)
{
case Operator.PLUS:
{
result = left + right;;
}
break;
....
}


return result;
}
小开

No need to convert. You can apply conditions on Enums inside a switch. Like so,

public enum Operator
{
PLUS,
MINUS,
MULTIPLY,
DIVIDE
}


public double Calculate(int left, int right, Operator op)
{
switch (op)
{
case Operator.PLUS: return left + right;
case Operator.MINUS: return left - right;
case Operator.MULTIPLY: return left * right;
case Operator.DIVIDE: return left / right;
default: return 0.0;
}
}

Then, call it like this:

Console.WriteLine("The sum of 5 and 5 is " + Calculate(5, 5, Operator.PLUS));
小开

Since C# 8.0 introduced a new switch expression for enums you can do it even more elegant:

public double Calculate(int left, int right, Operator op) =>
op switch
{
Operator.PLUS => left + right,
Operator.MINUS => left - right,
Operator.MULTIPLY => left * right,
Operator.DIVIDE => left / right,
_    =>  0
}

Ref. https://learn.microsoft.com/en-us/dotnet/csharp/whats-new/csharp-8


提交答案