软件系统坏死的症状

50 阅读 0 评论 33 点赞

我是靠谱客的博主大意牛排，最近开发中收集的这篇文章主要介绍软件系统坏死的症状，觉得挺不错的，现在分享给大家，希望可以做个参考。

概述

“Copy”程序：

一个从键盘读入字符并输出到打印机的程序。

void Copy()
{
  int c;
  while ((c = RdKbd()) != EOF)
  WrtPtr(c);
}

用户希望Copy程序能从纸带读入机中读入信息。

现实中的约束－－不能改变接口

Copy程序的第一次修改结果：

bool ptFlag = false;
//remember to reset this flag
void Copy()
{
  int c;
  while ((c = (ptFlag ? Rdpt() : Rdkbd())) != EOF)
  WrtPtr(c);
}

客户又希望Copy程序有时可以输出到纸带穿孔机上。

//Copy程序的第二次修改结果
bool ptFlag = false;
bool punchFlag = false;
//remember to reset these flag
void Copy()
{
  int c;
  while ((c = (ptFlag ? Rdpt() : Rdkbd())) != EOF)
  punchFlag ? WrtPunch(c) : WrtPtr(c);
}

从中我们可以嗅到程序的坏味了：

僵化：

–单一的改动会导致有依赖关系模块中的连锁改动－－增加第3种读入/输出方式？

脆弱：

–进行一个改动，程序与之无概念上联系的地方都可能出现问题。

牢固：

–提出代码进行复用很复杂。

不必要的复杂：

–包含没有用的代码。

不必要的重复：

–每个重复的代码体中大体相似，但有细微的差别。

晦涩：

–模块难以理解。

可以肯定，随着需求的进一步变化，坏味的趋势会更严重，程序会变得混乱不堪。

解决方法：运用OO的做法

void Reader
{
public:
	virtual int read() = 0;
};

class KeyboardReader : public Reader
{
public:
	virtual int read() {return RdKbd();}
};

KeyboardReader g_defaultReader;
void Copy(reader& reader = g_defaultReader)
{
	int c;
	while ((c = reader.read()) != EOF)
	WrtPtr(c);
}

回顾：