《12个有趣的C语言问答》评析(2)
前文链接:http://www.cnblogs.com/pmer/p/3313913.html
(没存盘,遭遇过热保护。至少4个问答的评论白写了。默哀一下)
2,main() 方法的返回类型
Q:请问下面这段代码能否通过编译?如果能的话,那么这段代码中隐含什么问题吗?
#include<stdio.h>
void
main(
void
)
{
char
*ptr = (
char
*)
malloc
(10);
if
(NULL == ptr)
{
printf
(
"\n Malloc failed \n"
);
return
;
}
else
{
// Do some processing
free
(ptr);
}
return
;
}
A:答案是代码能通过编译,但是会留下针对main()方法的返回类型的警告。main()方法的真正返回类型应该为'int'而非'void'。这是因为'int'返回类型能够让程序返回状态值。尤其是当这段程序作为其他应用的附属程序时这个状态值将更加重要。
Answer: The code will compile error free but with a warning (by most compilers) regarding the return type of main()function. Returntype of main() should be ‘int’ rather than ‘void’. This is because the ‘int’ return type lets the program to return a status value. This becomes important especially when the program is being run as a part of a script which relies on the success of the program execution.
评:
很难说这段代码能否通过编译,因为void main(void)根本就不是标准中规定的C程序的基本形式。诚然,在很多编译器上这段代码都能“带病”通过编译,但
不能保证在所有编译器上都能通过。C语言没有给出可以通过编译的承诺,通过编译只能理解为编译器擅自放水而已。
写到这里发现,其实原文中有这层意思(“by most compilers”),但翻译者却偷工减料,歪曲了原文的意思。
下风格问题:
代码中的写了一个if语句,怪异的是if字句中有return语句而else字句中没有,两条return语句一脚门里一脚门外。下面两种写法要好得多:
int main( void ) { char *ptr = ( char *) malloc (10); if ( NULL == ptr) { printf ( "\n Malloc failed \n" ); return 1; } // Do some processing free (ptr); return 0; } |
int main( void ) { char *ptr = ( char *) malloc (10); if (NULL == ptr) { printf ( "\n Malloc failed \n" ); return 1; } else { // Do some processing free (ptr); return 0; } } |
3,内存泄露
Q:请问以下代码有内存泄露吗?
#include<stdio.h>
void
main(
void
)
{
char
*ptr = (
char
*)
malloc
(10);
if
(NULL == ptr)
{
printf
(
"\n Malloc failed \n"
);
return
;
}
else
{
// Do some processing
}
return
;
}
A:好,虽然上面的代码没有对指针 ptr 进行内存释放,但实际上即使是程序结束也不会造成内存泄露,因为当程序结束时所有一开始被占据的内存就全部清空了。但如果上面这段代码是在 while 循环里面那将会造成严重的问题。
Answer: Well, Though the above code is not freeing up the memory allocated to ‘ptr’ but still this would not cause a memory leak asafter the processing is done the program exits. Since the program terminates so all the memory allocated by the program isautomatically freed as part of cleanup. But if the above code was all inside a while loop then this would have caused serious memoryleaks.Note: 如果你需要了解更多关于内存泄露的问题以及如何使用工具检测内存泄露,你可以参考这篇文章 Valgrind 。
评:
这段问答的第一个毛病是,main()函数不规范。前一个问答中作者把void main( void )写法否定了,这个问答中却自己这样写,自掌嘴巴。
诚然,这段代码确实不存在内存泄漏,但代码中自己不free,却把这个本该由程序员完成的活交给系统,这是很不足取的。尽管没有内存泄漏,但free还是应该写。
代码中另一个问题就是return语句一脚门里一脚门外的问题,前面说过,不再赘述。
4,free() 方法
Q:以下代码当用户输入'freeze'时会奔溃,而如果输入'zebra'则运行正常,这是为什么?
#include<stdio.h>
int
main(
int
argc,
char
*argv[])
{
char
*ptr = (
char
*)
malloc
(10);
if
(NULL == ptr)
{
printf
(
"\n Malloc failed \n"
);
return
-1;
}
else
if
(argc == 1)
{
printf
(
"\n Usage \n"
);
}
else
{
memset
(ptr, 0, 10);
strncpy
(ptr, argv[1], 9);
while
(*ptr !=
'z'
)
{
if
(*ptr ==
''
)
break
;
else
ptr++;
}
if
(*ptr ==
'z'
)
{
printf
(
"\n String contains 'z'\n"
);
// Do some more processing
}
free
(ptr);
}
return
0;
}
A:问题的根源是因为代码在while循环中改变了 ptr 指针的地址。当输入为'zebra'时,while循环甚至在执行 第一遍前就结束了,所以free()释放的内存地址就是一开始malloc()分配的地址。但是当输入'freeze'时, ptr记录的地址在while循环中被更改,因为将会是错误的地址传递到free()方法中引起崩溃。
Answer: The problem here is that the code changes the address in ‘ptr’ (by incrementing the ‘ptr’) inside the while loop. Now when ‘zebra’ is supplied as input, the while loop terminates before executing even once and so the argument passed to free() is the same address as given by malloc(). But in case of ‘freeze’ the address held by ptr is updated inside the while loop and hence incorrect address is passed to free() which causes the seg-fault or crash.
评:
Answer中的解释没什么问题。但造成问题更主要的原因其实是把所有代码都写在了main()中。如果其中
memset (ptr, 0, 10); strncpy (ptr, argv[1], 9); while (*ptr != 'z' ) { if (*ptr == '' ) break ; else ptr++; } if (*ptr == 'z' ) { printf ( "\n String contains 'z'\n" ); // Do some more processing } |
部分的功能由一函数完成,那么根本不会产生这样的问题。
翻译的毛病很多:
“改变了ptr指针的地址”,其实原意是 改变了ptr中(存放)的地址。
“因为将会是错误的地址传递到free()方法中引起崩溃”,这简直就不是中国话,另外漏翻了“seg-fault”。
风格方面依然存在return语句一脚门里一脚门外的毛病。其实存在多个return语句时,把if-else写成多个if通常显得更清爽。
return -1; |
是非标准写法,应该是编译器的方言。
一句,啰嗦,头脑不清。但在这个例子里,由于是通过malloc()申请的内存,所以当strlen(argv[1])大于等于9时,memset(ptr, 0, 10);会起到提前设置ptr[9]为\0的效果。但要实现这点其实简单地
就可以了,没必要兴师动众地调用memset()函数。
如果一定要将所申请内存中的各个数据对象初始化为0,其实可以通过
简单地实现。
和malloc()不同calloc()申请的内存中的数据初始化为0。而且,如果是为数组申请空间,calloc()也显得比malloc()更正式一点。
(未完待续)