优化开关结构 - 如何避免添加 if 子句 [关闭]

Question

我正在尝试优化我必须重构的代码。没有任何优化的代码会有一些 switch 语句。如果 switch 语句内部发生错误，则向调用方法返回错误，例如：

switch(var)
{ 
    case VAL1: 
    //do something... 
    break; 
    case VAL2: 
    //do something else... 
    //... 
    case VAL3: 
    if (...) // there is any case that can cause error 
    { 
         return error1;
    }
    break; 
    case VAL4: 
    if (...) // there is any case that can cause error 
    { 
         return error2;
    }
    break;
    case VAL5: 
    if (...) // there is any case that can cause error 
    { 
         return error1;
    }
    break;
    //and so on... 
    default: 
         break; 
} 

我正在重构代码，所以我没有在 switch 语句中返回错误，而是标记变量中存在错误：

int error_type = -1; 

switch(var)
{ 
    case VAL1: 
    //do something... 
    break; 
    case VAL2: 
    //do something else... 
    //... 
    case VAL3: 
    if (...) // there is any case that can cause error 
    { 
         error_type = error1;
    }
    break; 
    case VAL4: 
    if (...) // there is any case that can cause error 
    { 
         error_type = error2;
    }
    break;
    case VAL5: 
    if (...) // there is any case that can cause error 
    { 
         error_type = error1;
    }

    break;
    //and so on... 
    default: 
         break; 
}

if (error_type != -1) 
       return error_type; 

当没有错误时会出现问题，因为我们正在添加另一个 if 语句，如果该方法每秒被调用多次，这可能会导致性能问题。我想避免每次都检查条件。有什么建议可以改进此代码吗？有什么开关重构技巧吗？

//编辑：我知道这个例子可能看起来很愚蠢（因为那里的重构看起来不是很有用）但我正在重构的实际代码确实需要它（相信我）所以我尽量不降低性能在最终代码中。

Answer 1

一般的经验法则是switch 语句可以替换为查找表或数组。这些表的一个优点是它们可以很容易地以非常低的性能成本和对查找函数（引擎）的少量修改进行更新。

这里有一些想法：

如果有条件检查的模式，将变量放入表中。让引擎执行检查。

将函数指针放入表中以执行检查。如果函数指针为NULL，则不进行检查。