转自:http://blog.csdn.net/orbit/article/details/2120086
不知道从什么时候开始,switch-case语句成了代码坏味道的代名词,写代码的时候小心翼翼地避开它,看到别人代码中的switch-case就皱眉头,想想其实大可不必这样,switch-case语句并不是代码坏味道的根源,坏味道来自糟糕的代码(结构)设计,比如过多的switch-case分支,或者多重switch-case嵌套等等,这些都将导致代码可读性下降,如果再加上代码风格较差,代码不对齐,那么坏味道就相当地大了。
STATUS DriverIoControl(UINT function_no, PVOID para_in, PVOID para_out)
{
STATUS rc;
switch(function_no)
{
case PROCESSA:
rc = ProcessA(para_in,para_out);
break;
case PROCESSB:
rc = ProcessB(para_in,para_out);
break;
case PROCESSC:
rc = ProcessC(para_in,para_out);
break;
..........
default:
rc = UN_SUPPORT;
break
}
return rc;
}
STATUS ProcessA(PVOID para_in, PVOID para_out)
{
//一些代码....
}
STATUS ProcessB(PVOID para_in, PVOID para_out)
{
//一些代码....
}
STATUS ProcessC(PVOID para_in, PVOID para_out)
{
//一些代码....
}
#define DISPATCH_BEGIN(func) switch(func) /
{
#define DISPATCH_FUNCTION(func_c, function) case func_c: /
rc = function(para_in,para_out); /
break;
#define DISPATCH_END(code) default: /
rc = code; /
}
STATUS DriverIoControl(UINT function_no, PVOID para_in, PVOID para_out)
{
STATUS rc;
DISPATCH_BEGIN(function_no)
DISPATCH_FUNCTION(PROCESSA,ProcessA)
DISPATCH_FUNCTION(PROCESSB,ProcessB)
DISPATCH_FUNCTION(PROCESSC,ProcessC)
........................
DISPATCH_END(UN_SUPPORT)
return rc;
}
typedef STATUS (*ProcessFuncPtr)(PVOID para_in, PVOID para_out);
typedef struct tagDispatchItem
{
ProcessFuncPtr func_ptr;
}DISPATCH_ITEM;
DISPATCH_ITEM dispatch_table[MAX_DISPATCH_ITEM];
STATUS DriverIoControl(UINT function_no, PVOID para_in, PVOID para_out)
{
/*或者需要对function_no调整,使其区间落到[0 - MAX_DISPATCH_ITEM)*/
if((function_no >= 0) && (function_no < MAX_DISPATCH_ITEM))
return dispatch_table[function_no].func_ptr(para_in,para_out);
else
return UN_SUPPORT;
}
如果function_no不是线性的,那就需要查表:
typedef struct tagDispatchItem
{
UNIT func_no;
ProcessFuncPtr func_ptr;
}DISPATCH_ITEM;
DISPATCH_ITEM dispatch_table[MAX_DISPATCH_ITEM];
STATUS DriverIoControl(UINT function_no, PVOID para_in, PVOID para_out)
{
int i,find = -1;
for(i = 0; i < MAX_DISPATCH_ITEM; i++)
{
if(function_no == dispatch_table[i].func_no)
{
return dispatch_table[i].func_ptr(para_in,para_out);
}
}
return UN_SUPPORT;
}
使用表驱动的好处就是DriverIoControl的代码就这几行,添加新功能,只需要维护驱动表dispatch_table就行了,就这样摆脱了冗长乏味的switch-case。
前面例子中的switch-case语句中各个case分支参数比较简单整齐,也就是各个case分支都是用相同的参数para_in和para_out,如果各个分支使用的参数不整齐怎么办?那就需要封装,通常是用struct和union结合定义一个统一的数据结构做为接口参数,不同的分支dispatch函数内部根据需要从这个统一的数据结构中提取相应的数据。具体方法就是首先定义统一的dispatch函数接口,例如:
typedef STATUS (*ProcessFuncPtr)(DISPATCH_DATA *para);
然后用struct和union相结合定义DISPATCH_DATA,DISPATCH_DATA通常有这样的结构:
typedef struct tagDISPATCH_DATA
{
//此处声明所有的分支公用的数据
union
{
//此处声明processA分支使用的数据
}processA;
union
{
//此处声明processA分支使用的数据
}processB;
union
{
//此处声明processA分支使用的数据
}processC;
......
}DISPATCH_DATA;
做过Windows驱动程序的朋友肯定对IRP不陌生,IRP就是一个典型的例子。
就这样吧,打完,收工。
=============================================================================
以下是原文评论:说的是表驱动某些情况下会带来效率的损耗和可能在驱动里导致潜在的问题。
因为使用函数指针数组在循环查询来代替直接调用,一方面可能降低运行效率,每次IOCTL调用都要走一个循环,而且function越多越慢.(而如果你仔细查看switch生成得汇编代码,其实编译器在这种情况下也会优化成查表法的,只不过更有效率).另一方面就是将一些Bug由编译时报错延迟到了运行时才发生,导致篮屏.而效率和安全是驱动程序最需要的.