Question

我正在为我的Pic32编写一个小的源文件函数，但我却陷入了困境。它基本上是一个实用程序，应该将inc char数据存储到缓冲区中，然后，如果收到'\r'，它会将缓冲区与命令列表（数组names）进行比较，如果匹配则是发现，该项目的索引被返回。

这部分来自标题：

#define NAMECNT 6    
static const char names[NAMESCNT][10] = {   // 6commands, max 10 char each
        "korr",         // 1
        "adc",          // 2
        "fft",          // 3
        "data",         // 4 
        "pr",           // 5
        "prsc"};        // 6

/* functions */
extern int comm(char cdata);

在主文件中，有一个大开关：

switch( comm(recieved_ch) ){
case 1: foo1(); break; 
case 2: foo2(); break;
...
}

现在，为了更清晰，我想使用而不是1,2，...原始名称（如case KORR: case ADC:）所以我为每一个写了解释

#define KORR 1
#define ADC 2

但我不喜欢这个解决方案，因为我想在更多的项目中使用这个源文件，并且每个都会有不同的命令列表。有什么方法可以做到这一点吗？最好的办法是在预处理器中创建数组名称，但我怀疑这甚至是可能的。我正在考虑使用枚举类型（它与命令列表names具有相同的项目），但我不确定它会怎样。

Answer 1

我认为预处理器可以使用连接运算符##使事情更清晰，但它不会产生性能优势。编译器可以优化switch语句，但这依赖于实现。

而不是＆＃34;一个大开关，＆＃34;使用函数指针数组。像

这样的东西

func_ptrs[comm(received_ch) - 1]();

将调用相应的函数，其中foo1位于索引0，foo2位于1，等等。要添加命令，只需将命令名称附加到命令列表和函数指针func_ptrs。

毕竟，你一石二鸟：你创造了一种简单的方法来添加命令和提高性能。

此外，通过字符串数组进行线性搜索效率非常低。哈希表会产生性能优势。

Answer 2

您可以使用X-macros构建enum并填充数组，然后您可以使用enum中的switch值：

#define VARS \
    X(korr) \
    X(adc)  \
    X(fft)  \
    X(data) \
    X(pr)   \
    X(prsc)

static const char names[][10] = {   // 6commands, max 10 char each
#define X(name) #name,
    VARS
#undef X
};

enum evars {
#define X(name) name, 
    VARS
#undef X
};

extern int comm(char cdata);

int main(void)
{
    char x = 1;

    switch (comm(x)) {
        case korr:
            printf("korr");
            break;
        case adc:
            printf("adc");
            break;
        /* ... and so on */
    }
    return 0;
}

X的扩展是：

static const char names[][10] = {

 "korr", "adc", "fft", "data", "pr", "prsc",

};

enum evars {

 korr, adc, fft, data, pr, prsc,

};

编辑：正如@ 5gon12eder所指出的，您不需要在数组的第一维中硬编码6（您可以不指定）。

我可以使用预处理器使这一点更清晰吗？

2 个答案: