程序找错和改进

思一克

程序找错和改进. 看以下程序有什么错误, 或有可能有什么错误, 如何改进, 如果有的话.

1. 
   
2. #include <.....>
   
3. #include <.....>
   
4. int func1(char *cp)
   
5. {
   
6.     ...................
   
7. 
   
8. }
   
9. 
   
10. main()
    
11. {
    
12. char *cp;
    
13. char buf[] = "abc.com\naaa.com\nbbb.com";
    
14. 
    
15.           cp = strtok(buf, "\n ");
    
16.           while(cp) {
    
17.             if(func1(cp))
    
18.                  printf("domain %s is local.\n", cp);
    
19.             cp = strtok(0, "\n ");
    
20.           }
    
21. }
    

复制代码

快乐的土豆

[ 本帖最后由 快乐的土豆 于 2008-1-29 22:21 编辑 ]

cugb_cat

func1是不是该写成
int func(const char *cp);
?

思一克

buf is NOT constant string

原帖由 快乐的土豆 于 2008-1-29 21:52 发表

namtso

func1的参数为char *，在func1里，可能对char *指向的字符串进行修改，根据func1对cp所进行的不同操作，后续的printf及strtok可能正常运行，也可能使程序进行死循环，或者崩溃

fl3w

给个标准答案哈，
我猜想应该不是strtok的不可重入问题，也不会是fun(const)的问题。

[ 本帖最后由 fl3w 于 2008-1-30 09:32 编辑 ]

思一克

是strtok不可重入问题.

func1中调用了strtok, 如果func1不是你自己编的, 你又不知道它调用了strtok, BUG就有了.
不是const的问题. func1不改变串内容.

这时候,应该用strtok_r

原帖由 namtso 于 2008-1-29 22:16 发表
func1的参数为char *，在func1里，可能对char *指向的字符串进行修改，根据func1对cp所进行的不同操作，后续的printf及strtok可能正常运行，也可能使程序进行死循环，或者崩溃

cugb_cat

原帖由 思一克 于 2008-1-30 09:39 发表
是strtok不可重入问题.

func1中调用了strtok, 如果func1不是你自己编的, 你又不知道它调用了strtok, BUG就有了.
不是const的问题. func1不改变串内容.

这时候,应该用strtok_r

晕，还真是这个问题，不过版主没说清楚func1是不是自己写的，我觉得如果func1按照const参数规则来写，这个应该就是没问题的吧？

fl3w

其实我也想到一个问题：在fun1里面将cp的结尾的null替换掉，然后让printf可能出问题。

思一克

func1有const, 不改变参数也不行. 因为strtok利用了自己内部的静台存储.

原帖由 cugb_cat 于 2008-1-30 09:48 发表

晕，还真是这个问题，不过版主没说清楚func1是不是自己写的，我觉得如果func1按照const参数规则来写，这个应该就是没问题的吧？