【原创】超强版 Trim 横空出世！

deathbravo

挖坟的能力很强啊

liubinbj

这种函数设计上就是错的：void trim(char* s)
应该是这样 char* trim(char* s)
似乎坛子里没人指出这一点，悲哀，大多数的中国程序员就是这么写程序的，还以为很高明。

FH

原帖由 liubinbj 于 2006-6-21 17:45 发表
这种函数设计上就是错的：void trim(char* s)
应该是这样 char* trim(char* s)
似乎坛子里没人指出这一点，悲哀，大多数的中国程序员就是这么写程序的，还以为很高明。

后一种定义用起来方便，但前一种也没有错啊？

liubinbj

原帖由 FH 于 2006-6-21 21:36 发表


后一种定义用起来方便，但前一种也没有错啊？

语法是对的，设计是错的，那就是错的，你看看微软的大牛会这样定义trim吗？你觉得glibc会这么定义trim吗？严重误人子弟。

FH

原帖由 liubinbj 于 2006-6-22 15:57 发表


语法是对的，设计是错的，那就是错的，你看看微软的大牛会这样定义trim吗？你觉得glibc会这么定义trim吗？严重误人子弟。

char *的返回值用途广泛些，但是多了一个return也是开销啊！
侧重点不同罢了。

流川

这段代码不用太紧张吧

ccjjhua

这是我写的一个剔出前后空格的函数，优点就是没有进行内存的搬移，缺点是要引用函数的返回值，而且对传入参数的内存值有一处修改。
char * my_trim(const char *str)
{
        char *p_str;
        char * head;

        if(str == NULL)
                return NULL;
        head = NULL;

        for(p_str=str;*p_str;p_str++)
        {
                if(*p_str == ' ' || *p_str == '\t')
                        continue;
                else
                {
                        head = p_str;
                        break;
                }
        }

        for(p_str=head+strlen(head)-1;p_str >= head;p_str--)
        {
                if(*p_str == ' ' || *p_str == '\t')
                        continue;
                else
                {
                        *(p_str+1) = 0;
                        break;
                }
        }

        return head;

}

[ 本帖最后由 ccjjhua 于 2006-7-3 16:52 编辑 ]

ccjjhua

本着学习的态度，要了解一下高手们都是怎么想的。

zhugcx

char *strim(char *str)
{
        char *p = str;
        if (NULL == p)
        {
                return NULL;
        }
        char *s = NULL;
        char *e = NULL;
        while(*p != '\0')
        {
                if (' ' == *p)
                {
                        if (NULL != s && NULL == e)
                        {
                                e = p;
                        }
                }
                else
                {
                        if (NULL == s)
                        {
                                s = p;
                        }
                        e = NULL;
                }
                ++p;
        }

        if (NULL != e)
        {
                *e = '\0';
        }
        return s;
}

清晰版

swxlion

好吧。我是十全十美。第100楼