Windows Embedded CE 6 msmqadm工具的一个bug
2010-08-24 05:26:00 来源:WEB开发网最近继续使用C++做MSMQ的开发,最简单的方法就是看微软的源代码,在做MSMQ管理部分,参考reg.cxx,发现了一个bug,其实这个bug没什么大不了,为了证明自己“眼光敏捷”,吹毛求疵的把这个bug报告一下。
这 bug在reg.cxx文件的273行,如果执行msmqadm register install 的时候,写注册表失败就会进入这段代码。我估计这个bug的出现是因为拷贝复制导致的,因为前后的代码都非常类似,如果把这段代码封装起来,那么就不会出现这个bug了。
dwValue = 1;if (ERROR_SUCCESS != (hr = RegSetValueEx (hKey, L"Index", 0, REG_DWORD, (BYTE *)&dwValue, sizeof(DWORD)))) { RegCloseKey (hKey); rsc_wprintf (IDS_CANT_WRITE_KEY, L"Keep", hr); return 1;}
应该改为
dwValue = 1;if (ERROR_SUCCESS != (hr = RegSetValueEx (hKey, L"Index", 0, REG_DWORD, (BYTE *)&dwValue, sizeof(DWORD)))) { RegCloseKey (hKey); rsc_wprintf (IDS_CANT_WRITE_KEY, L"Index", hr); return 1;}
我想这段代码没有走Unit Test,至少没有做写注册表失败的Unit Test。
其实Unit Test全面覆盖真的很难,最近自己也写很多Test cases,写的代码中,有一半为测试代码。其实还远远不够,因为要覆盖全部异常情况,测试代码应该是功能代码的好几倍。任重而道远呀。
作者:Jake Lin(Jake's Blog on 博客园)
出处:http://procoder.cnblogs.com
本作品由Jake Lin创作,采用知识共享署名-非商业性使用-禁止演绎 2.5 中国大陆许可协议进行许可。 任何转载必须保留完整文章,在显要地方显示署名以及原文链接。
更多精彩
赞助商链接