點(diǎn)擊左上方藍(lán)色“混說(shuō)Linux”,選擇“設(shè)為星標(biāo)”

來(lái)源:https://www.elpauer.org/2011/08/the-most-stupid-c-bug-ever/
這是來(lái)自“The most stupid C bug ever”的一篇文章,很有意思,咱今兒個(gè)就分享給大家。相信這樣的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的返回碼。
else?if?(code?==?200)?{?????//?Downloading?whole?file
????/*?Write?new?file?(plus?allow?reading?once?we?finish)?*/
????g?=?fname???fopen(fname,?"w+")?:?tmpfile();
}所以,上面的程序在Windows平臺(tái)下需要用不同的方式來(lái)處理,不能直接使用Windows的tmpfile()函數(shù)。
于是作者就先把這個(gè)問(wèn)題記下來(lái),在注釋中寫(xiě)下了FIXME:
else?if?(code?==?200)?{?????//?Downloading?whole?file
????/*?Write?new?file?(plus?allow?reading?once?we?finish)?*/
????//?FIXME?Win32?native?version?fails?here?because
????//???Microsoft's?version?of?tmpfile()?creates?the?file?in?C:\
????g?=?fname???fopen(fname,?"w+")?:?tmpfile();
}FILE?*?tmpfile?(?void?)?{
#ifndef?_WIN32
????return?tmpfile();
#else
????//code?for?Windows;
#endif
}于是他重構(gòu)了一下他的代碼——寫(xiě)一個(gè)自己實(shí)現(xiàn)的tmpfile() – w32_tmpfile,然后,在Windows 下用宏定義來(lái)重命名這個(gè)函數(shù)為tmpfile()。(注:這種用法是比較標(biāo)準(zhǔn)的跨平臺(tái)代碼的寫(xiě)法)
#ifdef?_WIN32
??#define?tmpfile?w32_tmpfile
#endif
FILE?*?w32_tmpfile?(?void?)?{
????//code?for?Windows;
}靠!居然沒(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();
}現(xiàn)在我們把所有的代碼連在一起看,并比較一下:
能正常工作的代碼
#ifdef?_WIN32
#??define?tmpfile?w32_tmpfile
#endif
FILE?*?w32_tmpfile?(?void?)?{
????code?for?Windows;
}
else?if?(code?==?200)?{?????//?Downloading?whole?file
????/*?Write?new?file?(plus?allow?reading?once?we?finish)?*/
????//?FIXME?Win32?native?version?fails?here?because
????//?????Microsoft's?version?of?tmpfile()?creates?the?file?in?C:\
????//g?=?fname???fopen(fname,?"w+")?:?tmpfile();
????if(NULL?!=?fname)?{
????????g?=?fopen(fname,?"w+");
????}?else?{
????????g?=?tmpfile();
????}
}不能正常工作的代碼
#ifdef?_WIN32
#??define?tmpfile?w32_tmpfile
#endif
FILE?*?w32_tmpfile?(?void?)?{
????code?for?Windows;
}
else?if?(code?==?200)?{?????//?Downloading?whole?file
????/*?Write?new?file?(plus?allow?reading?once?we?finish)?*/
????//?FIXME?Win32?native?version?fails?here?because
????//????Microsoft's?version?of?tmpfile()?creates?the?file?in?C:\
????g?=?fname???fopen(fname,?"w+")?:?tmpfile();
}也許你在一開(kāi)始就看到了這個(gè)bug,但是作者沒(méi)有。所有的問(wèn)題都出在注釋上:
/*?Write?new?file?(plus?allow?reading?once?we?finish)?*/
//?FIXME?Win32?native?version?fails?here?because
//?????Microsoft's?version?of?tmpfile()?creates?the?file?in?C:\你看到了最后那個(gè)C:\嗎?在C中,“\” 代表此行沒(méi)有結(jié)束,于是,后面的代碼也成了注釋。這就是這個(gè)bug的真正原因!
而之所以改成if-else能工作的原因是因?yàn)樽髡咦⑨屃死系膯?wèn)號(hào)表達(dá)式的代碼,所以,那段能工作的代碼成了:
/*?Write?new?file?(plus?allow?reading?once?we?finish)?*/
//?FIXME?Win32?native?version?fails?here?because?Microsoft's?version?of?tmpfile()?creates?the?file?in?C:????//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ò)。
-x<10 ? f(result):f(-result);
因?yàn)槲以谖耶?dāng)時(shí)用 vi 編寫(xiě)代碼,所以沒(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)顯示上面的代碼:
float?result?=?num/*pInt;?….
/*?some?comments?*/
-x<10???f(result):f(-result);Holly Shit! 我的代碼成了:
float?result?=?num-x<10???f(result):f(-result);媽的!我的這個(gè)錯(cuò)誤在愚蠢程度上和上面那個(gè)作者出的錯(cuò)誤有一拼,所以說(shuō)代碼規(guī)范也很重要。

往期推薦