关于“永远的UNIX”主页上的一篇文章

羽人

http://fanqiang.chinaunix.net/a4/b2/20010416/134440.htmlurl

上面是“永远的UNIX”主页上的一篇文章，题目是“C语言中操作字符串的一些函数源代码”。我在里面看到一篇关于“streql”函数的源码如下：

4. streql(), 判断两个字符串是否相等.
int streql(char *str1,char *str2)
{
while((*str1==*str2)&&(*str1))
{
str1++;
str2++;
}
return((*str1==NULL)&&(*str2==NULL));
}

我仔细想了想，觉得最后的return语句写成“return(*str1==*str2)”就可以了，效率好像还高 呵呵，也许我的问题有些太简单了，不过真的是没想明白，或者它的做法有特殊的原因吧。如果哪位高手愿意的话，还望不吝赐教，指点一二。先谢过了！

yuxh

同意
这函数写得也一般般啊

assiss

呵呵。yuxh，这一般般可不是一般般就能说的，你得找一个比它好的（又有机会学习了，哈哈）

guile

我也同意楼主的观点

FH

这个函数有漏洞啊！没有判断str2越界嘛！

思一克

(*str1==NULL)是不好的写法，虽然也工作。
（*str1 == '\0' ) 或 （！*str1 ) 即可。

FH

[quote]原帖由 "思一克"]（*str1 == '\0' ) 或 （！*str1 ) 即可。[/quote 发表：

不行。

yuxh

[quote]原帖由 "FH"]这个函数有漏洞啊！没有判断str2越界嘛！[/quote 发表：

str2不会越界，*str2 == 0的时候要么*str1 == 0，要么*str1 != *str2，循环都结束

yuxh

原帖由 "思一克" 发表：
(*str1==NULL)是不好的写法，虽然也工作。
（*str1 == '\0' ) 或 （！*str1 ) 即可。

同意
一般NULL是一个指针，*str1是个char，（*str1 == '\0' ) 比较规范，但还是不如*str1 == *str2直接

FH

原帖由 "yuxh" 发表：

str2不会越界，*str2 == 0的时候要么*str1 == 0，要么*str1 != *str2，循环都结束

str2++的时候就越界了，还*str2？*str2可能根本就不存在了。