《C程序设计伴侣》里的月薪10000的代码

mirnshi

Ager 发表于 2012-10-08 02:52
国内受影响养成这个习惯的，可能更大的源头是《C陷阱与缺陷》。

如果-Wall的话，一些正确的 if(t=s) ...

-Wall 依赖于编译器的版本。低版本的gcc可以完全通过，到了高版本就会出现警告错误了。不过-Wall可以最大限度地指出程序的隐患。

noword2k

明显的缓存区溢出漏洞啊。

koolcoy

0xC1988 发表于 2012-10-06 21:06
最后的大亮

一般情况下这么写不会影响性能{:3_189:}

1. 
   
2. hoolala@ubuntu:/tmp$ cat t.c
   
3. #include <stdio.h>
   
4. 
   
5. int main() {
   
6.         FILE* file = fopen("file.txt", "w");
   
7.         fclose(file);
   
8.         file = NULL;
   
9.         return 0;
   
10. }
    
11. hoolala@ubuntu:/tmp$ gcc t.c -S -O2
    
12. hoolala@ubuntu:/tmp$ cat t.s
    
13.         .file        "t.c"
    
14.         .section        .rodata.str1.1,"aMS",@progbits,1
    
15. .LC0:
    
16.         .string        "w"
    
17. .LC1:
    
18.         .string        "file.txt"
    
19.         .section        .text.startup,"ax",@progbits
    
20.         .p2align 4,,15
    
21.         .globl        main
    
22.         .type        main, @function
    
23. main:
    
24. .LFB22:
    
25.         .cfi_startproc
    
26.         subq        $8, %rsp
    
27.         .cfi_def_cfa_offset 16
    
28.         movl        $.LC0, %esi
    
29.         movl        $.LC1, %edi
    
30.         call        fopen
    
31.         movq        %rax, %rdi
    
32.         call        fclose
    
33.         xorl        %eax, %eax       <<======== 这里是设定返回值
    
34.         addq        $8, %rsp           <<======== 退栈操作
    
35.         .cfi_def_cfa_offset 8
    
36.         ret                       <<=========  函数返回，可见file = NULL这句完全被优化掉了
    
37.         .cfi_endproc
    

复制代码

starwing83

回复 43# koolcoy


    不是性能问题。这么写的人明显脑子不清醒。你敢交给他重要任务么

koolcoy

回复 44# starwing83

我只是说一下，书上的代码你跟它扣质量多没意思啊，闭上眼睛一抓都是一大把问题的
   

starwing83

回复 45# koolcoy


    想当年看APUE，那个代码是多么优美啊……别说抓bug，有些代码你就那么看都不一定能明白它的原理，还得配上文字说明。

dengxiayehu

starwing83 发表于 2012-10-08 11:43
回复 43# koolcoy

初衷可能是抱着“指针在不用时要将其栓住（置为NULL），避免野指针”。

不过在这里多此一举，比较教条。

starwing83

回复 47# dengxiayehu


    关键是，这条规则应该是，非局部的指针应该置为NULL，避免野指针。因为局部指针本身在生命期之后都会被释放掉的。另一个可能的问题是局部指针有可能在其后继续被使用。这种情况下适用的是另一个教条：尽量限制局部变量的作用域。

教条主义其实不是错，可是不分青红皂白地教条就只能叫愚蠢了。

mirnshi

starwing83 发表于 2012-10-08 11:43
回复 43# koolcoy

不是不清醒，而是没有跨过那个门槛。

yybmsrs

回复 30# Ager


    大侠，变量名不是不能带$符号吗？测试了下好像可以啊。。。