Question

我曾经想过一段时间，因为它似乎在我经验丰富的代码中出现了很多。

我有一些使用switch语句的代码很多，但它真正做的每次都是访问不同的队列。

void store(int toSwitchOn, float posx, float posy){ 
    myDataStruct newValue;
    newValue.psX = posx;
    newValue.psY = posy;

    switch(toSwitchOn){
        case 1:
            queue1.push(newValue);          
            break;
        case 2:
            queue2.push(newValue);          
            break;
        case 3:
            queue3.push(newValue);
            break;
        case 4:
            queue4.push(newValue);
            break;
        case 5:
            queue5.push(newValue);
            break;
    }


}

每个语句中唯一改变的是队列变量。是否有一些巧妙的方法来浓缩这种重复的代码？

Answer 1

将您的队列存储在矢量中。

std::vector<std::queue<someType> > queues (5);
//fill vector with your 5 queues

//this replaces the switch:
if (toSwitchOn >= 1 && toSwitchOn <= 5)
    queue [toSwitchOn - 1].push (newValue);
else
    //default switch case

Answer 2

std::vector<std::queue<someType> > queues (5); 
//toSwitchOn is of type size_t and zero indexed.
... 
if (toSwitchOn < queues.size())
   queue [toSwitchOn].push (newValue);  //0 - 1 = undefined land...     
else     //default switch case

Answer 3

显而易见的答案是将switch替换为vector或map查找要启用的内容。

但是，我看到整数和矢量索引之间的耦合是一个漏洞接口。

我想知道这个函数的调用者如何知道要使用哪个整数值。谁告诉他们要用什么？他们刚刚被赋予对Storage对象的引用吗？

替换：

int function_telling_which_index_to_use_for_storage();

使用：

Storage* storage_to_use();

然后你可以说：

Storage* storage = storage_to_use();
// ...
storage->store(posx, posy);

请记住：封装，封装，封装。

冷凝开关声明？

3 个答案: