我和goto一起被嘲笑

antigloss

原帖由 飞灰橙 于 2007-8-16 21:03 发表


这也不行，还有教条的人说一个函数只能有一个return

恩~我也很讨厌这个，最后 return 一个变量代表成功失败的……靠，这个变量中间变来变去的，天晓得最后会变成什么样子，还不如直接想 return 就 return 算了。
而且这种方法如果还不准用 goto 的话，那就可以死人了。

flw2

比如一个函数，检查4个参数的合法性，如果不合法，把不合法的理由打印到一个字符串中，然后返回－1。
我不喜欢很多return -1; 就会用goto

飞灰橙

原帖由 flw 于 2007-8-16 21:07 发表
用 goto 的时候，标号的名字一定要起得非常有水准。

美女市长喜欢这样的吗？

1. 
   
2. void func( )
   
3. {
   
4.      ....
   
5. malloc_failed:
   
6. open_failed:
   
7.     free(p);
   
8. read_failed:
   
9.     close(fd);
   
10. 
    
11.     return -1;
    
12. }
    
13. 
    

复制代码

converse

我从来不用goto的.

flw

原帖由 飞灰橙 于 2007-8-16 21:14 发表


美女市长喜欢这样的吗？

void func( )
{
     ....
malloc_failed:
open_failed:
    free(p);
read_failed:
    close(fd);

    return -1;
}

没有宾语，不好。

M.I.X

原帖由 flw2 于 2007-8-16 21:08 发表
比如一个函数，检查4个参数的合法性，如果不合法，把不合法的理由打印到一个字符串中，然后返回－1。
我不喜欢很多return -1; 就会用goto

赞同

void func( )
{
     ....
malloc_failed:
open_failed:
    free(p);
read_failed:
    close(fd);

    return -1;
}

不需要这么多label吧？

很多时候判断的条件比较多，比如查询数据库，
有N多的检测条件，如果一个失败就需要返回，
不用goto ,则函数里到处是

1. 
   
2. 
   
3. if( ... < 0)
   
4. {
   
5.    free(...);
   
6.    close(...);
   
7.    db_free(mysql_result);
   
8.    free_data_connection(...);
   
9.    ....
   
10.    return -1;
    
11. }
    
12. 
    

复制代码

晕死了

M.I.X

原帖由 converse 于 2007-8-16 21:20 发表
我从来不用goto的.

那你怎么写？

ivhb

原帖由 飞灰橙 于 2007-8-16 21:03 发表


这也不行，还有教条的人说一个函数只能有一个return

：）还真没有考虑到这个。确实看过有这么一条。早就忘记了这条了。程序里面经常是好多处return的。
经过你提醒，我修改了。那样写不是我喜欢的，嵌套太深了，不是好事情。这么写，仅仅是make leaders happy！
好在，我没有遇到过这样的leader，要不我就疯了。

antigloss

原帖由 M.I.X 于 2007-8-16 21:31 发表


那你怎么写？

C++ 里有个叫析构函数的东西~

flw

原帖由 M.I.X 于 2007-8-16 21:31 发表

那你怎么写？

他用 try{throw()}catch{}