开发学院WEB开发ASP 原来高焕堂也误用职责链模式 阅读

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

 2010-01-06 10:43:07 来源:WEB开发网   
核心提示: 近日 见同事正在看 高焕堂新书 《UML+OOPC 嵌入式C语言开发精讲》偶然发现他对COR模式存在严重误解,废话少说 ,原来高焕堂也误用职责链模式,具体如下:在他的大作 32.5 章节里 用到该模式 代码如下:#include "lw_oopc_kc.h"#include "ex32-
  近日 见同事正在看 高焕堂新书 《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技术教父级代表人物”;近年来,专注与产业 企业及嵌入式产品的架构设计,被称为

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




Tags:原来 高焕堂 误用

编辑录入:爽爽 [复制链接] [打 印]
[]
  • 好
  • 好的评价 如果觉得好,就请您
      0%(0)
  • 差
  • 差的评价 如果觉得差,就请您
      0%(0)
赞助商链接