大家看看这个程序哪个地方错了

wpf8036328

1. strcmp1(char *p,char *q)
   
2. {
   
3.         while((*p != '\0')&&(*p = *q))
   
4.         {
   
5.                 p++;
   
6.                 q++;
   
7.         }
   
8.         if(*p >; *q)
   
9.                 return(1);
   
10.         else
    
11.                 return(-1);
    
12.         if(*p = *q)
    
13.                 return(0);
    
14. }

复制代码

FH

q比p短怎么办?
*p==*q时也返回-1
*p=*q这里少了一个=

wpf8036328

啊，对少个=，那就再加上一条件*q ！= '\0',这样能行了吗

FH

*p==*q的时候你也返回-1了

wpf8036328

改对了吧这样，呵呵，我刚调试过

1. strcmp1(char *p,char *q)
   
2. {
   
3.         while((*p != '\0')&&(*p == *q))
   
4.         {
   
5.                 p++;
   
6.                 q++;
   
7.         }
   
8.         if(*p >;= *q)
   
9.                 return(1);
   
10.         if(*p < *q)
    
11.                 return(-1);
    
12. }

复制代码

lenovo

忘了返回值是int了吧。
你看一看我写的一个。

1. 
   
2. int strcmp1( char *s1,  char *s2)
   
3. {
   
4.         while ( (*s1 != '\0') && (*s2 != '\0') && (*s1++ == *s2++) )
   
5.                 ;
   
6.         return (*s1 - *s2);
   
7. }
   

复制代码

whyglinux

原帖由 "lenovo" 发表：

你看一看我写的一个。

1. 
   
2. int strcmp1( char *s1,  char *s2)
   
3. {
   
4.    while ( (*s1 != '\0') && (*s2 != '\0') && (*s1++ == *s2++) )
   
5.       ;
   
6.    return (*s1 - *s2);
   
7. }
   

复制代码

确实很精炼，不过程序中有错误存在，注意 ++ 的使用。仔细检查一下循环结束时的状态就知道应该怎样修改这个程序了。

其实循环条件还有进一步精简的余地。如下所示，跟上面程序得到的结果完全相同：

1. 
   
2.   while ( (*s1++ == *s2++) && *s1 )
   
3.     ;
   

复制代码

wpf8036328

感觉斑竹这个算法和系统的strcmp有点象啊，应该没有什么错把？

FH

[quote]原帖由 "whyglinux"][/quote 发表：

版主的对，你的不对。
慢慢琢磨吧！

wpf8036328

我也试过了，斑竹的确实不错的，呵呵，也挺简洁的，