Question

在我的应用程序中，我必须在许多方法中基于变量值（ m_iIndex ）执行许多任务。为了实现它，我在大多数方法中使用switch case语句

例如：

MathMethod()
{
   switch(m_iIndex)
   {
    case 0 : CallAddition(); break; 
    case 1 : CallSubtraction(); break;
    case 2 : CallMultiplication(); break;
    case 3 : CallDivision(); break;
   }
}


StrangeMethod()
{
   switch(m_iIndex)
   {
    case 0 : CallStrange(10,"John"); break; 
    case 1 : CallStrange(20,"Paul"); break;
    case 2 : CallStrange(30,"Kurt"); break;
    case 3 : CallStrange(40,"Mark"); break;
   }
}

这还有大约10种方法。我想知道有没有办法通过减少我所有方法中的switch case语句来使这段代码更优雅和简短。

Answer 1

尝试使用Polymorphism并为每个操作创建一个类。这称为Command pattern。

我只能猜测你如何设置m_iIndex，但下面的例子说明了我的意思：

abstract class Operation
{
    abstract void Execute();
}

class Addition : Operation
{
    public override void Execute()
    {
      // ...
    }
}

// same for Subtraction etc.

void OnAdditionButtonClicked(...)
{
    // Instead of setting m_iIndex to 0, use this instead:
    _operation = new Addition();
}

如果您提供更多上下文，我可以更改我的示例以更好地满足您的要求。

Answer 2

让我们假设您的方法MathMethod()和StrangeMethod()以及成员m_iIndex是类YourClass的一部分。尽量消除m_iIndex;相反，使用YourClass的子类，其中MathMethod和StrangeMethod是虚拟的，并在子类中被覆盖。

Here 你会找到一个更精细的答案。

使用更优雅的代码重构多个switch语句

2 个答案: