就算你是高手你也會(huì)犯的Bug

我相信這樣的bug，就算你是高手你也會(huì)犯的。你來(lái)看看作者犯的這個(gè)Bug吧。。

首先，作者想用一段程序來(lái)創(chuàng)建一個(gè)文件，如果有文件名的話，就創(chuàng)建真正的文件，如果沒(méi)有的話，就調(diào)用tmpfile()創(chuàng)建臨時(shí)文件。

他這段程序就是HTTP下載的C程序。code==200就是HTTP的返回碼。

elseif(code==200){//Downloadingwholefile
/*Writenewfile(plusallowreadingoncewefinish)*/
g=fname?fopen(fname,"w+"):tmpfile();
}

但是這個(gè)程序，只能在Unix/Linux下工作，因?yàn)?Microsoft 的tmpfile()的實(shí)現(xiàn)居然選擇了 C: 作為臨時(shí)文件的存放目錄，這對(duì)于那些沒(méi)有管理員權(quán)限的人來(lái)說(shuō)就出大問(wèn)題了。在Windows 7下，就算你有管理員權(quán)限也會(huì)有問(wèn)題。

所以，上面的程序在Windows平臺(tái)下需要用不同的方式來(lái)處理，不能直接使用Windows的tmpfile()函數(shù)。

于是作者就先把這個(gè)問(wèn)題記下來(lái)，在注釋中寫下了FIXME：

elseif(code==200){//Downloadingwholefile
/*Writenewfile(plusallowreadingoncewefinish)*/

//FIXMEWin32nativeversionfailsherebecause
//Microsoft'sversionoftmpfile()createsthefileinC:
g=fname?fopen(fname,"w+"):tmpfile();
}

然后，作者覺(jué)得需要寫一個(gè)跨平臺(tái)的編譯：

FILE*tmpfile(void){
#ifndef_WIN32
returntmpfile();
#else
//codeforWindows;
#endif
}

然后，作者覺(jué)得這樣實(shí)現(xiàn)很不好，會(huì)發(fā)現(xiàn)名字沖突，因?yàn)檫@樣一來(lái)這個(gè)函數(shù)太難看了。

于是他重構(gòu)了一下他的代碼——寫一個(gè)自己實(shí)現(xiàn)的tmpfile() – w32_tmpfile，然后，在Windows 下用宏定義來(lái)重命名這個(gè)函數(shù)為tmpfile()。（注：這種用法是比較標(biāo)準(zhǔn)的跨平臺(tái)代碼的寫法）

#ifdef_WIN32
#definetmpfilew32_tmpfile
#endif

FILE*w32_tmpfile(void){
//codeforWindows;
}

搞定！編譯程序，運(yùn)行。

靠！居然沒(méi)有調(diào)用到我的w32_tmpfile()，什么問(wèn)題？調(diào)試，單步跟蹤，果然沒(méi)有調(diào)用到！

難道是問(wèn)號(hào)表達(dá)式有問(wèn)題？改成if – else 語(yǔ)句，好了！

if(NULL!=fname){
g=fopen(fname,"w+");
}else{
g=tmpfile();
}

問(wèn)號(hào)表達(dá)式不應(yīng)該有問(wèn)題吧，難道我們的宏對(duì)問(wèn)號(hào)表達(dá)式不起作用，這難道是編譯器的預(yù)編譯的一個(gè)bug？作者懷疑到。

現(xiàn)在我們把所有的代碼連在一起看，并比較一下：

能正常工作的代碼

#ifdef_WIN32
#definetmpfilew32_tmpfile
#endif

FILE*w32_tmpfile(void){
codeforWindows;
}

elseif(code==200){//Downloadingwholefile
/*Writenewfile(plusallowreadingoncewefinish)*/
//FIXMEWin32nativeversionfailsherebecause
//Microsoft'sversionoftmpfile()createsthefileinC:
//g=fname?fopen(fname,"w+"):tmpfile();
if(NULL!=fname){
g=fopen(fname,"w+");
}else{
g=tmpfile();
}
}

不能正常工作的代碼

#ifdef_WIN32
#definetmpfilew32_tmpfile
#endif

FILE*w32_tmpfile(void){
codeforWindows;
}

elseif(code==200){//Downloadingwholefile
/*Writenewfile(plusallowreadingoncewefinish)*/
//FIXMEWin32nativeversionfailsherebecause
//Microsoft'sversionoftmpfile()createsthefileinC:
g=fname?fopen(fname,"w+"):tmpfile();
}

也許你在一開(kāi)始就看到了這個(gè)bug，但是作者沒(méi)有。所有的問(wèn)題都出在注釋上：

/*Writenewfile(plusallowreadingoncewefinish)*/
//FIXMEWin32nativeversionfailsherebecause
//Microsoft'sversionoftmpfile()createsthefileinC:

你看到了最后那個(gè)C:嗎？在C中，“” 代表此行沒(méi)有結(jié)束，于是，后面的代碼也成了注釋。這就是這個(gè)bug的真正原因！

而之所以改成if-else能工作的原因是因?yàn)樽髡咦⑨屃死系膯?wèn)號(hào)表達(dá)式的代碼，所以，那段能工作的代碼成了：

/*Writenewfile(plusallowreadingoncewefinish)*/
//FIXMEWin32nativeversionfailsherebecauseMicrosoft'sversionoftmpfile()createsthefileinC://g=fname?fopen(fname,"w+"):tmpfile();
if(NULL!=fname){
g=fopen(fname,"w+");
}else{
g=tmpfile();
}

我相信，當(dāng)作者找到這個(gè)問(wèn)題的原因后，一定會(huì)罵一句“媽的”！我也相信，這個(gè)bug花費(fèi)了作者很多時(shí)間！

最后，我也share一個(gè)我以前犯的一個(gè)錯(cuò)。

我有一個(gè)小函數(shù)，需要傳入一個(gè)int* pInt的類型，然后我需要在我的代碼里 把這個(gè)int* pInt作除數(shù)。于是我的代碼成了下面的這個(gè)樣子：

float result = num/*pInt; ….

/* some comments */

-x<10 ? f(result):f(-result);

因?yàn)槲以谖耶?dāng)時(shí)用vi編寫代碼，所以沒(méi)有語(yǔ)法高亮，而我的程序都編譯通過(guò)了，但是卻出現(xiàn)了很奇怪的事。

我也不知道，用gdb調(diào)式的時(shí)候，發(fā)現(xiàn)有些語(yǔ)句直接就過(guò)了。

這個(gè)問(wèn)題讓我花了很多時(shí)間，最后發(fā)現(xiàn)問(wèn)題原來(lái)是沒(méi)有空格導(dǎo)致的，TNND，下面我用代碼高亮的插件來(lái)顯示上面的代碼，

floatresult=num/*pInt;
....

/*somecomments*/

-x<10?f(result):f(-result);

Holly Shit! 我的代碼成了：

floatresult=num-x<10?f(result):f(-result);

我的這個(gè)錯(cuò)誤在愚蠢程度上和上面那個(gè)作者出的錯(cuò)誤有一拼。

審核編輯 ：李倩