.新手上路 高手指教 chmop函数原型......分析

wishrr

char *chomp(char *buf)
{
   char *p;
   for(p=buf;*p!= '\0';p++);
   if(p!=buf){p--;}
   while(p>=buf&&(*p == '\n' ||*p == '\r'))
   {
     *p-- = '\0';
   }
  return (buf);
}

int main()
{
   char *str="hello everybody!";
   char *str_cpy=NULL;
   str_cpy=chomp(str);
   printf("the str is %s\n",str_cpy);
   return 0;
}
这个代码我在rad hat上跑了，但是运行起来有点慢 ，不知道是什么原因 ，而且这个函数是掉换行符号才加．不明白！所以还请各位大哥，大姐，指点一二！
晚辈非常感谢！。。。嘻嘻

hellioncu

这个函数慢你也能感觉出来？
char *str="hello everybody!";应改成
char str[]="hello everybody!";

zhangsuozhu

char *buf    =>  char buf[]

1 p = buf + strlen(buf)-1;

2 while(p>=buf&&(*p == '\n' ||*p == '\r')) p --;
   
3 * ++p = '\0';

wishrr

回复 2# hellioncu

我运行了的，谢谢你的回复，但是我还是不咋的明白！{:3_201:} 到底怎么做！

wishrr

回复 3# zhangsuozhu
你可不可以 ，具体详细的说明一下！呜呜！，还是不怎么的明白！{:3_201:}   {:2_169:} 谢谢了！


~-~

zhangsuozhu

char *p = "hello everybody!";中，p中保存的地址是rodata区的地址，这个区是不可以改的。只能读，如果你改了。就会出错。所以。在你代码中的*p-- = '\0'; 就会产生错误。

*p-- = '\0';在循环里面，会多次在结尾赋'\0',这是不必要的，只在最后一个'\r'或'\n'中赋'\0'就可以了。

for(p=buf;*p!= '\0';p++); 其实就求字符串的长度，然后把p指向字符串的结尾。 完全可以strlen来实现。

glibc里优化了strlen这个函数。所以用标准的实现更快更好理解。

pengjianbokobe

楼上解释的够详细的了   很少有这么耐心的人那

wishrr

回复 7# pengjianbokobe
不好意思！不该拿年龄当会事！本人90，谢谢各位前辈！

wishrr

回复 6# zhangsuozhu

太谢谢了！ 谢谢了！