原来高焕堂也误用职责链模式

[wanglovec]

近日 见同事正在看 高焕堂新书 《UML+OOPC 嵌入式C语言开发精讲》偶然发现他对COR模式存在严重误解。

废话少说 ，具体如下：

在他的大作 32.5 章节里 用到该模式



代码如下：

#include "lw_oopc_kc.h"
#include "ex32-il.h"

CLASS(LED_P0)
{
IMPLEMENTS(IL);
void (*init)(LED_P0*);
IL* next;
};

CLASS(LED_P1)
{
IMPLEMENTS(IL);
void (*init)(LED_P1*);
IL* next;
};

CLASS(LED_P2)
{
IMPLEMENTS(IL);
void (*init)(LED_P2*);
IL* next;
};



/* EX32-led.c */
#include <REG51F.H>
#include "lw_oopc_kc.h"
#include "ex32-led.h"

static void init_1(LED_P1* t) {
t->next = NULL;
// xxx = 0xf3;
}

static int pass_1(char ty, char hx, void *t) {
LED_P1 *cthis = (LED_P1*)t;
IL* ps = cthis->next;
if(ty=='1')
{
P1 = hx;
return 1;
}
if(ty=='A') P1 = hx;
if(ps == NULL) return 0;
else return ps->pass(ty, hx, ps);
}


CTOR(LED_P1)
FUNCTION_SETTING(init, init_1)
FUNCTION_SETTING(IL.pass, pass_1)
END_CTOR
// -------------------------------
static void init_0(LED_P0* t) {
t->next = NULL;
}

static int pass_0(char ty, char hx, void *t) {
LED_P0 *cthis = (LED_P0*)t;
IL* ps = cthis->next;
if(ty=='0')
{
P0 = hx;
return 1;
}
if(ty=='A') P0 = hx;
if(ps == NULL) return 0;
else return ps->pass(ty, hx, ps);
}

CTOR(LED_P0)
FUNCTION_SETTING(init, init_0)
FUNCTION_SETTING(IL.pass, pass_0)
END_CTOR
//----------------------------------------
static void init_2(LED_P2* t) {
t->next = NULL;
}

static int pass_2(char ty, char hx, void *t) {
LED_P2 *cthis = (LED_P2*)t;
IL* ps = cthis->next;
if(ty=='2')
{
P2 = hx;
return 1;
}
if(ty=='A') P2 = hx;
if(ps == NULL) return 0;
else return ps->pass(ty, hx, ps);
}


CTOR(LED_P2)
FUNCTION_SETTING(init, init_2)
FUNCTION_SETTING(IL.pass, pass_2)
END_CTOR







这三个类 就是职责链中的负责处理请求与转发请求的类，但 严重问题是 他们与 请求基本存在一一对应的关系

if(ty=='0')
{
P0 = hx;
return 1;
}

这是LED_P0的责任, 如果 把'0'换为'1' 则就是 LED_P1的责任 这是我们从代码里发现的, 其实他自己在书里也已经写了(如果ty值为'0',表示这是LED_P0的责任,于是将另一个参数hx值送到P0。如果ty值为'A',表示这是所有对象的都有的责任，。。)



结论：高焕堂完全误解了COR模式 ，其实这些代码散发着 重复的臭味 完全可以重构到表模式 具体如何重构 不用我说了吧？？



摘自这本书的高焕堂简介：

从事IT行业近30年，被称为“台湾OO技术教父级代表人物”；近年来，专注与产业 企业及嵌入式产品的架构设计，被称为

“台湾软件架构设计大师”。。。。。

http://www.cnblogs.com/wangok/archive/2010/01/05/1639388.html

[夭夭]

代码不咋的,分析也不咋的

[猫已经找不回了]

看看。