通过一个小小的问题提起良好习惯的养成

类别:编程语言 点击:0 评论:0 推荐:
从学会c语言到现在已经很长时间了,并且一直习惯在作文件处理的时候使用

Stream I/O Routines,也自认为能够很熟练的使用了这些函数。但是今天在移植一段MFC代码的时候确碰到了一件让我始料不及的事。

我所移植的这段代码很简单。功能是从EXE的资源中导出一个资源(这个资源是一个zip文件,之后还有解压缩之类的操作),并且在硬盘上生成文件。MFC代码如下:

HRSRC hExeFile = ::FindResource(NULL, "STUB.EXE", "EXE_RESOURCE");
??? HGLOBAL hRes = ::LoadResource(NULL, hExeFile);
??? DWORD dwSize = ::SizeofResource(NULL, hExeFile);

?if (hRes != NULL)
??? {
??UINT FAR* lpnRes = (UINT FAR*)::LockResource(hRes);

??????? TRY
??????? {
??????????? CFile file(lpszFileName, CFile::modeCreate | CFile::modeWrite);

??????????? // Write the user-defined resource to the .exe file
??????????? file.WriteHuge(lpnRes, dwSize);
??????????? file.Flush();
??????? }
??????? CATCH(CFileException, e)
??????? {
??}
??????? END_CATCH
??????? ::FreeResource(hRes);

我移植后的程序:

FILE * exefile;
??? HRSRC hExeFile = ::FindResource(NULL, "STUB.EXE", "EXE_RESOURCE");
??? HGLOBAL hRes = ::LoadResource(NULL, hExeFile);
??? DWORD dwSize = ::SizeofResource(NULL, hExeFile);

?if (hRes != NULL)
??? {
??UINT FAR* lpnRes = (UINT FAR*)::LockResource(hRes);

? exefile = fopen(lpszFileName, "w");
??if (exefile != NULL)
??{
???// Write the user-defined resource to the .exe file
???fwrite(lpnRes, sizeof(char), dwSize, exefile);
???fflush(exefile);
???fclose(exefile);
??}
?????
???::FreeResource(hRes);

也许有的细心的朋友已经看出来我在移植过程中犯的错误了,可是我一开始并没有注意到。导致的结果是在中午的时候调试移植后的程序的时候发现运行结果与MFC时的程序运行结果不同,出现了错误,在调试过程种发现生成的文件与原有文件的大小不一致,无法运行。因此我又重新把注意力集中到这段在我看来再简单不过的程序上来了。可是是哪里出错呢。我把注意力集中在了fwrite上,我debug它的运行时候的返回值,发现返回值与dwSize并不一致,这很奇怪,而且实际写入的比dwSize要大,这说明在写入过程中肯定出现了问题,这时候我突然意识到我犯了一个很低级的错误。

? exefile = fopen(lpszFileName, "w");就是这一句,fopen在default的情况下是使用text (translated) mode “t”读写文件的,原来如此,一定有一些字符在fwrite过程中被转义了。于是我把它改成了exefile = fopen(lpszFileName, "wb");编译,运行,正常了。

其实本来这是再简单,再平常不过的工作了,可是我还是遇到了料想不到的麻烦,我想这也许就和我平时编程习惯,编程方法不好有关系,试想想如果我每一次使用fopen的时候都很清晰的表明是用哪种方式打开一个文件的话,就不会出现今天这个问题了。在此我也拿这个小小的问题提醒所有看过这篇文章的程序员,无论程序再小,结构再简单,都要养成一个良好的编程习惯。这样将使我们的工作事半功倍。

?

本文地址:http://com.8s8s.com/it/it28545.htm