这是当年开发的一个煤炭安全监控软件里增加新测点的函数。如果在当年看这段代码，感觉这应该是一段挺不错的代码，比如：注释很多，几乎每行都有注释;匈牙利命名法，使用像lp、w这样的前缀;结构、数组、宏定义，看起来都是有板有眼。
- //-----------------------------------------------------------------------------
- // 函数：NewPoint
- // 功能：分配一个测点定义缓冲区。
- // 入口参数：
- // lpPNum 测点号指针。
- // 出口参数：
- // 如成功，返回在测点定义表中的偏移；如失败，返回0xffff。
- //-----------------------------------------------------------------------------
- WORD NewPoint(LPCSTR lpPNum)
- {
- WORD i;
- LPTPOINT lpTmpPt;
- // 在测点定义表中找空的缓冲区
- for (i=0;i<PtTab.wCount;i++) if (PtTab.lpPt[i].tpDel) break;
- if (i==PtTab.wCount) {
- // 如未找到，则判断测点数是否已到***值，如已到，则返回失败
- if (PtTab.wCount+1>MAXPOINT) return(0xffff);
- // 如未找到***值，则测点总数加1。即在测点定义表的***追加一个缓冲区
- PtTab.wCount++;
- }
- // 初始化缓冲区
- lpTmpPt=&PtTab.lpPt[i];
- // 将删除标志(即缓冲区空标志)置1，作为后面具体定义时区分增加与修改的标志，
- // 如是修改，则此标志必为0
- lpTmpPt->tpDel=1;
- // 拷贝测点号到缓冲区
- lstrcpy(lpTmpPt->tpNum,lpPNum);
- // 将前一此操作测点的定义复制到缓冲区
- lstrcpy(lpTmpPt->tpName,HistPt.tpName);
- lpTmpPt->tpType=HistPt.tpType;
- lpTmpPt->tpOp=HistPt.tpOp;
- // 赋缺省的状态与数值
- lpTmpPt->tpState=PS_SUSP;
- lpTmpPt->tpRValue=0;
- lpTmpPt->tpDValue=0;
- // 置缺省的状态变化时间为当前时间
- _fmemcpy(&lpTmpPt->tpTime,&Time,sizeof(SYSTIME));
- // 将偏移值返回
- return(i);
- }
时间已经过去十几年了，计算机系统、开发语言、设计思想都有了很大转变，现在要写一段同样功能的代码，可能完全不同了。
***的变化应该是面向对象技术的应用，测点和测点表都会从struct变为class，测点数组可能会用std::vector或std::list代替，并且用new来动态分配，宏定义也会换成常量或枚举。而上面这个创建新测点的函数，也会被封装到测点表这个类中。面向对象技术改变了我们思考和实践的方式，而且这个改变作用是巨大的。
再有就是一些设计思想的转变，比如对注释的看法。以前，注释多可能意味着可读性强。但《重构》一书中说：“你看到一段代码有着长长的注释，然后发现，这些注释之所以存在乃是因为代码很糟糕。”再看看上面这段代码，前半段，其实是查找一个空的测点位置，完全可以用Extract Method(提炼函数)重构方法将其移入另外一个函数，并给其一个准确的命名，而不需要注释。后半段，“代码已经清楚说明了一切，注释已经变得多余了。”
匈牙利命名法，已成为争议***的命名法，其类型冗余常常大于它所带来的收益。它来自微软的一名匈牙利程序员，但现在微软的.Net和它的编程语言中，微软更换了这一法则，在C#中以骆驼命名法和帕斯卡命名法居多。
计算机系统的提升也改变着程序。早年16位OS上编程需要区分远指针和近指针，但现在32位环境下，C++指针变量前缀一般都是p，很少见到lp。
技术在进步，编程思想在转变，人的思维也在不断更新。也许再过十几年，现在听都没有听说过的新技术会大行其道，你回头再看看现在的代码，可能也会有像我今天的感觉。但无论怎样，请不要轻视我们的过去，因为没有过去的积累，就没有今天的进步。
原文链接：http://www.cnblogs.com/wanghui9072229/archive/2011/04/25/2028793.html
【编辑推荐】
