您的位置:知识库 » 软件设计

代码之丑(六)——分家的声明和使用

作者: dreamhead  发布时间: 2011-01-03 21:35  阅读: 817 次  推荐: 0   原文链接   [收藏]  

  这是一段长长的C++代码,我的问题是:relaPri、relaSec和 scoutBySec这三个变量在哪里用到了?

void DealForServiceA(const char *oprCode, const char *subID, const char *oID, XList *callCicsList) {
XString relaPri(“NULL”);
XString relaSec(“NULL”);
XString scoutBySec(“
0”);
XList
*tempList = new XList ;
callCicsList
->Add(tempList);
tempList
->Add(new XString(oprCode));
tempList
->Add(new XString(oID));
XString
*psTelNum = new XString;
tempList
->Add(psTelNum);
GetServnumberBySubsID(subID,
*psTelNum);
tempList
->Add(new XString(relaPri.table { font-size: 10pt;}c_str()));
tempList
->Add(new XString(relaSec.c_str()));
tempList
->Add(new XString(scoutBySec.c_str()));
}

  经过认真仔细的查看,或是使用传说的中“查找”功能,我们发现上面提到的那三个变量只在最后用了一下。

  不知道你是否注意到,我在最初特意强调了一下这是C++代码。这意味着,变量可以随用随声明,而不必像传统的C程序那样,只能在函数的开头把函数内部用到的变量一口气声明。 那么 ,我们就让声明和使用团聚吧!

XString relaPri(“NULL”);
tempList
->Add(new XString(relaPri.c_str()));
XString relaSec(“NULL”);
tempList
->Add(new XString(relaSec.c_str()));
XString scoutBySec(“
0”);
tempList
->Add(new XString(scoutBySec.c_str()));

  当声明和使用走到一起,我们的观察就有了新的视角,其实,这几个变量完全是可以不声明的,于是,代码再进一步:

tempList->Add(new XString(“NULL”));
tempList
->Add(new XString(“NULL”));
tempList
->Add(new XString(“0”));

  看到这里,我们就可以看出原来的做法到底有多么浪费:浪费时间给变量起名字——我们都知道,起个好名字不容易,也 浪费了时间在执行上,修改前的代码创建了两个XString对象,而修改后,只创建了一个对象。

  或许,你会觉得,有个变量会让我们了解这里实际上填加的内容到底是什么。不过,也许一个好的函数命名才是更好的选择,比如addRelaPri。这个疑问会揭示出这段代码存在另外一个问题,直接使用基本的数据结构而没有进行封装。不过,这不是这里讨论的目标,就到此打住吧!

  根据这段代码的调整,我们得出一条规则:

  • 代码的声明和使用应尽量接近。

  有的C程序员会暗自念叨,这个要求对C程序来说,简直太不合情理了。好吧!我承认,从语言的角度来说,是这样的。但是,我们需要仔细想想,为什么对于C语言来说,变量的声明和使用会距离遥远。通常,遥远的背后意味着硕大的函数,这才是让声明和使用天各一方的重要原因。

  在干净代码的世界里,大函数永远是不受欢迎的。为了让声明和使用尽早团聚,请把函数写小。

  作者简介:

  郑晔,ThoughtWorks公司咨询师,拥有多年企业级软件开发经验,热衷于探索各种程序设计语言在真实软件开发中所能发 挥的威力,致力于探寻合理的软件开发方式,加入ThoughtWorks公司后,投入到敏捷开发方法的实践之中,为其他公司提供敏 捷开发方法方面的咨询服务。他的blog是梦想风暴

  查看原文:代码之丑(六)

0
0
标签:重构

软件设计热门文章

    软件设计最新文章

      最新新闻

        热门新闻