C # як використовувати enum з перемикачем

Я не можу зрозуміти, як використовувати перемикачі в поєднанні з переліком. Не могли б ви сказати мені, що я роблю не так, і як це виправити? Я повинен використовувати перерахування, щоб скласти основний калькулятор.

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

Не могли б ви, хлопці, сказати мені, як я зіпсуюсь?

Не потрібно його перетворювати

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

До речі, використовуйте дужки

Оскільки C # 8.0 представив новий вираз перемикача для перелічень, ви можете зробити це ще елегантніше:

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
        }

Посилання https://docs.microsoft.com/en-us/dotnet/csharp/whats-new/csharp-8

Правильна відповідь уже дана, проте ось кращий спосіб (ніж перемикач):

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;
}

просто не кидайте на int

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

Ви не повинні перетворювати на ціле число. І для ділення, вам потрібно кинути ліворуч, щоб спочатку подвоїти, якщо ні, то ви будете робити ціле ділення.

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;
}

 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);
            }
        }
    }

Якщо ви не хочете використовувати оператор return для кожного випадку, спробуйте наступне:

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

   return result;
}

Усі інші відповіді правильні, але вам також потрібно правильно назвати свій метод:

Calculate(5, 5, Operator.PLUS))

А оскільки ви використовуєте intдля leftі right, результат буде intтакож ( 3/2 will result in 1). ви можете виконати привід до doubleобчислення результату або змінити параметри, щоб прийнятиdouble

Не потрібно конвертувати. Ви можете застосувати умови до Enums всередині комутатора. Як так,

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; 
    }
}

Тоді назвіть це так:

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

Дві речі. По-перше, вам потрібно визначити посилання на перелік у вашому тесті - а не "PLUS", це має бути "Operator.PLUS". По-друге, цей код був би набагато зручнішим для читання, якби ви використовували імена членів перерахування, а не їхні інтегральні значення в операторі switch. Я оновив ваш код:

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;
    }
}

Зателефонуйте за цим номером:

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

Ваш код чудовий. Якщо ви не впевнені, як використовувати функцію «Обчислити», спробуйте

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

Значення перерахування за замовчуванням починаються з 0 і збільшуються на одиницю для наступних елементів, поки ви не призначите різні значення. Також ви можете зробити:

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