C语言中史上最愚蠢的Bug
本文来自“The most stupid C bug ever”,很有意思,分享给大家。我相信这样的bug,就算你是高手你也会犯的。你来看看作者犯的这个Bug吧。。
首先,作者想用一段程序来创建一个文件,如果有文件名的话,就创建真正的文件,如果没有的话,就调用?tmpfile()?创建临时文件。他这段程序就是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(); }
但是这个程序,只能在Unix/Linux下工作,因为 Microsoft 的?tmpfile()的实现?居然选择了 C:\ 作为临时文件的存放目录,这对于那些没有管理员权限的人来说就出大问题了,在Windows 7下,就算你有管理员权限也会有问题。所以,上面的程序在Windows平台下需要用不同的方式来处理,不能直接使用Windows的tmpfile()函数。
于是作者就先把这个问题记下来,在注释中写下了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 }
然后,作者觉得这样实现很不好,会发现名字冲突,因为这样一来这个函数太难看了。于是他重构了一下他的代码——写一个自己实现的tmpfile() – w32_tmpfile,然后,在Windows 下用宏定义来重命名这个函数为tmpfile()。(陈皓注:这种用法是比较标准的跨平台代码的写法)
#ifdef _WIN32 #define tmpfile w32_tmpfile #endif FILE * w32_tmpfile ( void ) { //code for Windows; }
搞定!编译程序,运行。靠!居然没有调用到我的w32_tmpfile(),什么问题?调试,单步跟踪,果然没有调用到!难道是问号表达式有问题?改成if – else 语句,好了!
if(NULL != fname) { g = fopen(fname, "w+"); } else { g = tmpfile(); }
问号表达式不应该有问题吧,难道我们的宏对问号表达式不起作用,这难道是编译器的预编译的一个bug?作者怀疑到。
现在我们把所有的代码连在一起看,并比较一下:
能正常工作的代码
#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(); }
也许你在一开始就看到了这个bug,但是作者没有。所有的问题都出在注释上:
/* 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:\
你看到了最后那个C:\吗?在C中,“\” 代表此行没有结束,于是,后面的代码也成了注释。这就是这个bug的真正原因!
而之所以改成if-else能工作的原因是因为作者注释了老的问号表达式的代码,所以,那段能工作的代码成了:
/* 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(); }
我相信,当作者找到这个问题的原因后,一定会骂一句“妈的”!我也相信,这个bug花费了作者很多时间!
最后,我也share一个我以前犯的一个错。
我有一个小函数,需要传入一个int* pInt的类型,然后我需要在我的代码里 把这个int* pInt作除数。于是我的代码成了下面的这个样子:
float result = num/*pInt;
…./* some comments */
-x<10 ? f(result):f(-result);
因为我在我当时用vi编写代码,所以没有语法高亮,而我的程序都编译通过了,但是却出现了很奇怪的事。我也不知道,用gdb调式的时候,发现有些语句直接就过了。这个问题让我花了很多时间,最后发现问题原来是没有空格导致的,TNND,下面我用代码高亮的插件来显示上面的代码,
float result = num/*pInt; .... /* some comments */ -x<10 ? f(result):f(-result);
Holly Shit! 我的代码成了:
float result = num-x<10 ? f(result):f(-result);
妈的!我的这个错误在愚蠢程度上和上面那个作者出的错误有一拼。
(全文完)
(转载本站文章请注明作者和出处 酷 壳 – CoolShell ,请勿用于任何商业用途)
《C语言中史上最愚蠢的Bug》的相关评论
c++其实就是完全沿用了c的构架,只是在c上有所发展而已、、、
个人是这么理解的、、
这样傻逼的问题我也会遇到。
我是汇编学习者,表示汇编也有这问题.
@bob
我用windows下的gvim有非常明显的高亮!!!!
有一次我也出现过这种情况,但是当时VAssistX显示下一行也变成绿色,也就是注释的颜色了,我就纳闷了一下,结果才发现上面那一行最后居然是个\符号。
这就是编程的乐趣所在啊。互相交流这样的经验,大家都少走弯路。
不过用 Eclipse 编写 Java 的话,我好像除了运算优先级弄错了之外,好像还没有总结过类似的问题。
总结果然是很有意义的事情啊。
是的
BUG不可能最蠢
最蠢的应该是
scanf(“I love %s”, name);
这样的语句
现在有几个人不用语法高亮的呢?
Notepad++都有不错的高亮啊~~~
最近给朋友调试程序时遇到同样问题,
不过那个更恶心:注释最后一个字是汉字,这个汉字的unicode下最后一个byte是ascii的反斜杠。。。
曾经改代码的时候把
if (a == 0 && b == 0)
改成
if (a == 0 || b == 0)
结果手滑改成了
if (a == 0 ||& b == 0)
后来发现的时候,竟然认为自己发现了一个新的运算符 ||& 这个运算符是什么意思呢…
@Keekun
我觉得,unicode一般应当出现在程序内存,存入文件或网络传输或者别的什么的时候应该先进行编码,能避免出现这种情况吧。
这真是防不胜防呀。。。
哇哦!这个是 lexer 的问题。在编写词法分析器的时候也会遇到的!很坑爹但是也蛮有意思的。
我猜如果还支持 Unicode 常量什么的话,可能还会有更多问题吧。
我是最新评论
大家快点页面的头右侧有一个high一下!!!!!!!!!~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
@薛非
是的 你很聪明