本文來自“The most stupid C bug ever”,很有意思,分享給大家。
我相信這樣的bug,就算你是高手你也會犯的,你來看看作者犯的這個Bug吧。。
首先,作者想用一段程序來創建一個文件,如果有文件名的話,就創建真正的文件,如果沒有的話,就調用tmpfile()創建臨時文件。
他這段程序就是HTTP下載的C程序。code==200就是HTTP的返回碼。
elseif(code==200){//Downloadingwholefile /*Writenewfile(plusallowreadingoncewefinish)*/ g=fname?fopen(fname,"w+"):tmpfile(); }
但是這個程序,只能在Unix/Linux下工作,因為 Microsoft 的tmpfile()的實現居然選擇了 C: 作為臨時文件的存放目錄,這對于那些沒有管理員權限的人來說就出大問題了。在Windows 7下,就算你有管理員權限也會有問題。
所以,上面的程序在Windows平臺下需要用不同的方式來處理,不能直接使用Windows的tmpfile()函數。
于是作者就先把這個問題記下來,在注釋中寫下了FIXME:
elseif(code==200){//Downloadingwholefile /*Writenewfile(plusallowreadingoncewefinish)*/ //FIXMEWin32nativeversionfailsherebecause //Microsoft'sversionoftmpfile()createsthefileinC: g=fname?fopen(fname,"w+"):tmpfile(); }
然后,作者覺得需要寫一個跨平臺的編譯:
FILE*tmpfile(void){ #ifndef_WIN32 returntmpfile(); #else //codeforWindows; #endif }
然后,作者覺得這樣實現很不好,會發現名字沖突,因為這樣一來這個函數太難看了。
于是他重構了一下他的代碼——寫一個自己實現的tmpfile() – w32_tmpfile,然后,在Windows 下用宏定義來重命名這個函數為tmpfile()。(注:這種用法是比較標準的跨平臺代碼的寫法)
#ifdef_WIN32 #definetmpfilew32_tmpfile #endif FILE*w32_tmpfile(void){ //codeforWindows; }
搞定!編譯程序,運行。
靠!居然沒有調用到我的w32_tmpfile(),什么問題?調試,單步跟蹤,果然沒有調用到!
難道是問號表達式有問題?改成if – else 語句,好了!
if(NULL!=fname){ g=fopen(fname,"w+"); }else{ g=tmpfile(); }
問號表達式不應該有問題吧,難道我們的宏對問號表達式不起作用,這難道是編譯器的預編譯的一個bug?作者懷疑到。
現在我們把所有的代碼連在一起看,并比較一下:
能正常工作的代碼
#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(); }
也許你在一開始就看到了這個bug,但是作者沒有。所有的問題都出在注釋上:
/*Writenewfile(plusallowreadingoncewefinish)*/ //FIXMEWin32nativeversionfailsherebecause //Microsoft'sversionoftmpfile()createsthefileinC:
你看到了最后那個C:嗎?在C中,“” 代表此行沒有結束,于是,后面的代碼也成了注釋。這就是這個bug的真正原因!
而之所以改成if-else能工作的原因是因為作者注釋了老的問號表達式的代碼,所以,那段能工作的代碼成了:
/*Writenewfile(plusallowreadingoncewefinish)*/ //FIXMEWin32nativeversionfailsherebecauseMicrosoft'sversionoftmpfile()createsthefileinC://g=fname?fopen(fname,"w+"):tmpfile(); if(NULL!=fname){ g=fopen(fname,"w+"); }else{ g=tmpfile(); }
我相信,當作者找到這個問題的原因后,一定會罵一句“媽的”!我也相信,這個bug花費了作者很多時間!
最后,我也share一個我以前犯的一個錯。
我有一個小函數,需要傳入一個int* pInt的類型,然后我需要在我的代碼里 把這個int* pInt作除數。于是我的代碼成了下面的這個樣子:
float result = num/*pInt; ….
/* some comments */
-x<10 ? f(result):f(-result);
因為我在我當時用vi編寫代碼,所以沒有語法高亮,而我的程序都編譯通過了,但是卻出現了很奇怪的事。
我也不知道,用gdb調式的時候,發現有些語句直接就過了。
這個問題讓我花了很多時間,最后發現問題原來是沒有空格導致的,TNND,下面我用代碼高亮的插件來顯示上面的代碼,
floatresult=num/*pInt; .... /*somecomments*/ -x<10???f(result):f(-result);?
Holly Shit! 我的代碼成了:
floatresult=num-x<10???f(result):f(-result);
媽的!我的這個錯誤在愚蠢程度上和上面那個作者出的錯誤有一拼。
-
嵌入式
+關注
關注
5136文章
19515瀏覽量
314501 -
Linux
+關注
關注
87文章
11449瀏覽量
212700 -
WINDOWS
+關注
關注
4文章
3603瀏覽量
90886 -
C語言
+關注
關注
180文章
7629瀏覽量
140156 -
BUG
+關注
關注
0文章
156瀏覽量
15947
原文標題:嵌入式 C 語言史上最愚蠢的一個BUG
文章出處:【微信號:玩轉嵌入式,微信公眾號:玩轉嵌入式】歡迎添加關注!文章轉載請注明出處。
發布評論請先 登錄
標準C語言與嵌入式C語言有哪些區別
嵌入式linux c語言,嵌入式LinuxC語言開發工具.pdf

評論