[C] 有人用goto 处理错误么

fender0107401

我说要是说我从来没有用过goto，你们肯定会笑话我，

我觉着乱，哈哈。

vivian2086

还有，如果你的函数写成这样，我觉得有可能设计不合理了。 起码bar2依赖某个会在bar1改变的变量， ...
donotblock 发表于 2011-03-08 12:55

    你看得太仔细。。。那些是为了搭个结构，我承认这结构搭得不够简单没有岐义，但的确只是个摆设，我想说的只是有个loop 在那

vivian2086

回复 15# amarant


    好像就是在内核模块学到的，不记得了，也不知道里面用得多不多

liwangli1983

林锐放的屁你也当真， 他是在空想 goto的用法~~
三月廿七 发表于 2011-03-08 13:01

    林锐啥的我没看过，我自己就是这么用的．．．

mirnshi

搞那么复杂干嘛

1. 
   
2. ......
   
3. if( bar1() >= 0 &&  bar2() >= 0 )
   
4.          return 0;
   
5. ......
   

复制代码

mirnshi

一般情况下，不建议由子函数释放资源

zengc123

多层循环？一层一层出去不是更好吗？直接就跳出去了才不好吧。
我是尽量不用goto，exit（）之类的

csern

1. 
   
2. int foo(void)
   
3. {
   
4.         int err;
   
5.         int fd1, fd2;
   
6.         void *ptr;
   
7. 
   
8.         fd1 = open(...);
   
9.         if (fd1 == -1) {
   
10.                 err = -errno;
    
11.                 goto out;
    
12.         }      
    
13. 
    
14.         fd2 = open(...);
    
15.         if (fd2 == -1) {
    
16.                 err = -errno;
    
17.                 goto close_1;
    
18.         }      
    
19. 
    
20.         ptr = malloc(...);
    
21.         if (!ptr) {
    
22.                 err = -ENOMEN;
    
23.                 goto close_2;
    
24.         }
    
25.                
    
26.         /* ... */
    
27.         
    
28.         err = bar(...);
    
29.         if (err)
    
30.                 goto free_ptr;
    
31.                
    
32.         /* ... */
    
33.         
    
34.         err = 0;
    
35. free_ptr:
    
36.         free(ptr);
    
37. close_2:
    
38.         close(fd2);
    
39. close_1:
    
40.         close(fd1);
    
41. out:   
    
42.         return err;
    
43. }
    

复制代码

经常这么写

nizvoo

有资源需要释放的函数里面，用goto 很爽啊，比如

＝＝＝＝＝＝＝＝＝＝＝＝
写法一:
int foo()
{

...
vivian2086 发表于 2011-03-08 11:02

   
要不你分成两个函数

1. int foo2(int fd, char* p)
   
2. {
   
3.      if( bar1() < 0 ){      
   
4.           return -1;
   
5.      }
   
6.      
   
7. 
   
8.      if( bar2() < 0 ){         
   
9.           return -1
   
10.      }
    
11. 
    
12.      return 0;
    
13. 
    
14. }
    
15. 
    
16. int foo()
    
17. {
    
18.      int ret = 0;
    
19. 
    
20.      char *p = malloc();
    
21. 
    
22.      if (!p) return -1;
    
23. 
    
24.      int fd = open();
    
25. 
    
26.      if (fd) {
    
27. 
    
28.          if (foo2(fd, p) == -1)
    
29.                 ret = -1;
    
30. 
    
31.        fclose(fd);
    
32.     } else
    
33.        ret = -1;
    
34. 
    
35.     free(p);
    
36.    
    
37. 
    
38.      return ret;
    
39.      
    
40. }
    
41. 
    
42. 
    

复制代码

liseyy

回复 1# vivian2086


    linux驱动程序的probe方法一般使用GOTO作为错误的处理，可以回滚已经分配的资源。汇编里有用GOTO也较多，执行流程很复杂的程序里用GOTO较难容易错。