c++ - 简化复杂的方法

Question

我有一个方法，它基本上是一长串 if-else 的可变深度(有时很深)，因为它正在解释一个字节数组和根据每个字节的值，它会做一些不同的事情，例如从数组中“读取”更多内容并再次检查它们的某些值，或者它有一个直接的 Action 。像这样:

char current = array[current_index ++];
if(current == 1)
{
    char something = array[current_index ++];
    if(something == 2)
    {
         // open the fridge
    }
    else
    {
         char something_else = array[current_index ++];
         if(something_else == 3)
         {
             // make me a cake
         }
         else
         {
             // bring me a beer
         }
    }
}
else
if(current == check_this(another_variable))
{
     // so something else
}

我正在努力简化这整个困惑局面。所以，为了摆脱第一层的 if，我创建了一个函数指针数组，将 if 的主体提取到特定的函数中，基本上看看是否array 具有给定 current 的元素并执行函数(作为副作用，这会稍微降低执行速度并大大降低可读性)。

一个简短的例子

命令列表:

1 - 食物2 - 喝3 - 回家4 - 温暖5 - 寒冷6 - 肉7 - 番茄

所以:1 4 6 - 会给我烤肉(或者至少根据应用程序的实现)和 2 7 - 会给我番茄汁...和 3 - 退出。

编辑:switch - case 构造在这种情况下并不总是可用，因为有时会检查 current反对函数的返回值。

所以问题是:我很好奇您还会使用哪些其他技术来解决此类问题。

Answer 1

您可以使用类似于 Command 模式的东西。创建一个字典，其中键将是 array 中元素的可能值，值将是指向相应函数的指针，例如:

{
1 = (*open_the_fridge),
2 = (*make_me_a_cake),
...
}

然后就可以遍历数组，执行相应的函数了。

伪代码:

func_map{1=(*open_the_fridge), 2=(*make_me_a_cake),...}
for (i = 0; i < length(array); i++) {
    (*get_value_from_dict(func_map, array[i]))();
}