非议MFC(三)库代码的质量问题

原创 2003年02月18日 08:48:00

                          非议MFC(三)库代码的质量问题

关键字:C++,MFC,RECT,CRect,POINT,CPoint,质量

说明:程序片断仅包括理解所必需的代码,其余省略。

每个人的代码都不可能完全排除质量隐患,但MFC作为库代码,对其质量怎么苛求都不会过分。

1.只顾效率
file://in <WINDEF.H>
typedef struct tagRECT
{
      LONG left;
      LONG top;
      LONG right;
      LONG bottom;
} RECT, *PRECT, NEAR *NPRECT, FAR *LPRECT;
file://in <AFXWIN.H>
class CRect : public tagRECT
{
      CPoint& TopLeft();
      CPoint& BottomRight();
};
file://in <AFXWIN1.INL>
_AFXWIN_INLINE CPoint& CRect::TopLeft()
      { return *((CPoint*)this); }            file://[1]
_AFXWIN_INLINE CPoint& CRect::BottomRight()
      { return *((CPoint*)this+1); }            file://[2]
TopLeft()通过返回CPoint &同时提供Set和Get功能,并且,返回CPoint &比返回CPoint效率高。但是,函数的实现必须依赖指针的跨越性转换(即从CRect *转换成完全不相干的CPoint *),另外,还要假设编译器是顺序存放各数据成员。随意转换指针类型,不安全;依赖编译器实现,不可移植;以后扩展时可维护性降低(如增加数据成员),还有可能导致错误(如引入虚函数时,有的编译器将虚表放在对象存储地址的前部)。

2.不顾效率
file://in <AFXWIN.H>
class CRect : public tagRECT
{
      BOOL PtInRect(POINT point) const;
};
因为POINT结构体大于32位地址长度,形参使用值传递效率不高,应该改为引用。
软件的设计应该保持统一的取舍原则,如果说在上一点中,不惜采用那么极端的方式来提高效率,那么这里明显可以合理提高效率的地方为什么要放过呢?

3.算法不严谨
file://in <AFXWIN.H>
class CRect : public tagRECT
{
      BOOL IsRectEmpty() const;
};
IsRectEmpty()函数的功能是当矩形面积为空时返回1;当矩形面积为不空时返回0。
给出如下测试代码:
CRect rct(100,100,0,0);
BOOL b=rct.IsRectEmpty();
运行后b的值居然是1!?
有些CRect的成员函数如:IntersectRect()、UnionRect()等只有先调用NormalizeRect()才能确保获得正确结果。但IsRectEmpty()完全没必要依赖NormalizeRect(),例如可以这样实现:
BOOL CRect::IsRectEmpty() const
{
      return (left==right&&up==bottom ? 1 : 0);
}
推测起来,MFC中的实现可能是:若矩形的right<=left或bottom<=up则返回1。

4.无故破坏约定俗成的规则
file://in <AFXWIN.H>
class CRect : public tagRECT
{
      void operator=(const RECT& srcRect);
      void operator+=(LPCRECT lpRect);
};
自定义类型不要毫无价值的与内建类型不兼容(《Effective C++》语)。operator=()应该返回CRect &,这样做还可以支持链式赋值。同理operator+=()也应该返回CRect &。

5.没有尽力保证安全性
file://in <AFXWIN.H>
class CRect : public tagRECT
{
      CRect operator+(LPCRECT lpRect) const;
};
operator+()应该返回const CRect,这样做可以禁止形如(a+b)=c;的病态语句,同时也保持了与内建类型的行为一致。

6.没有尽力提高可用性和可靠性
file://in <AFXWIN.H>
class CDC : public CObject
{
      BOOL BitBlt(int x, int y, int nWidth, int nHeight, CDC* pSrcDC,
            int xSrc, int ySrc, DWORD dwRop);
};
做个简单的类比:
file://in <STRING.H>
size_t  __cdecl strlen(const char *);
形参为什么要声明为const char *?因为,其一,const char *既可以接受常量字符串又可以接受非常量字符串,而char *只能接受非常量字符串。其二,const可以保证函数体不更改原字符串这一契约。
所以BitBlt()的声明中,参数pSrcDC是原设备环境,不会改变,应该声明为const CDC *。


请参考上一篇《非议MFC(二)逻辑上的不完备》

非议MFC

非议MFC(一)宏和类型定义的困惑                          非议MFC(一)宏和类型定义的困惑关键字:C++,MFC,宏,macro,define,typedef有感于MFC...
  • huntjap
  • huntjap
  • 2005年01月05日 11:31
  • 1259

关于中国产品的质量问题-一个暖水袋引发的思考

今天和老妈吵了一架,只是因为30块钱的暖水袋。 老妈买了一个充电的,我担心充电的危险,而老妈确认为人家既然卖就是安全的,争吵过后我想了想,出发点都是好的,一个担心家人的安全,一个担心家人的温暖。 ...
  • zx6268476
  • zx6268476
  • 2015年05月31日 19:05
  • 677

代码质量问题的思考

代码质量问题,说起来是一个管理问题,但究其实质,是一个team建设问题,也是team风格铸造,team经验传承的问题,试想你的团队对代码风格,重构点的理解,架构调优的支撑点是有相似的认同度吗?是共同进...
  • shalaladanning
  • shalaladanning
  • 2009年09月29日 15:16
  • 127

解决数据质量问题是大数据应用的关键

研究称,整个人类文明所获得的全部数据量,有90%是最近两年内产生的。随着移动互联大潮的席卷,预计通过网路产生的数据量还将呈几何级增长。庞大的数据资源蕴藏着无限的宝藏,过去的一年无论是企业、政府还是媒体...
  • bluejason
  • bluejason
  • 2014年10月19日 14:32
  • 1627

代码审查常见代码质量问题

配套的Bug解释模式 为了有针对性的使用这个工具,减少bug的误报,提高使用效率,我们选择了10个左右的bug模式,下面就是对这10个模式的解释。 这些bug可能会引起程序的性能或逻辑问题. 需...
  • xingchao416
  • xingchao416
  • 2016年09月30日 10:41
  • 2048

软件开发中的质量问题

在软件开发过程中,软件质量是软件工程中一个经常被忽略的要素。在现代的RAD领域和视频开发领域,软件质量几乎是被人忽略的。标准组织不厌其烦地对软件质量作出规范,有些甚至提供了用来度量软件质量的工具、评价...
  • tozheng
  • tozheng
  • 2014年11月12日 16:03
  • 1309

《MFC游戏开发》笔记三 游戏贴图与透明特效的实现

本系列文章由七十一雾央编写,转载请注明出处。 http://blog.csdn.net/u011371356/article/details/9313239 作者:七十一雾央 新浪微博:h...
  • lsybryant
  • lsybryant
  • 2014年09月08日 13:46
  • 1257

C++五子棋的实现(MFC)

MFC实现的五子棋小游戏
  • Bugggget
  • Bugggget
  • 2017年12月05日 23:42
  • 2984

MFC开发 常见控件库

Xtrme toolkit,BCGControlBar,SkinMagic,AppFace,Skin++,Uskin++,SYGUI,LibUIDK,GuiToolkit,GardenUI等等,除了后...
  • caimagic
  • caimagic
  • 2016年03月12日 10:40
  • 675

软件配置管理与CMM/CMMI-三库管理

随着企业业务发展,产品线也变得越来越丰富。新产品的研发,可能由于周期短,有时直接快速组建一个新的项目组,大家热火朝天,技术攻关很快就把产品研发出来了。很多的设计点子,产品架构,技术攻关等通过高效的线下...
  • superbence
  • superbence
  • 2012年08月30日 09:08
  • 8294
内容举报
返回顶部
收藏助手
不良信息举报
您举报文章:非议MFC(三)库代码的质量问题
举报原因:
原因补充:

(最多只允许输入30个字)