重构发现：指针操作问题 -- 其他 -- IT技术博客大学习 -- 共学习共进步！

您现在的位置：首页 --> 其他 --> 重构发现：指针操作问题

重构发现：指针操作问题

浏览:2277次出处信息

原来版本：

以下是代码片段：
bool GetCookieUseName(const string& cookie_data, string& usename,
        const char * falg, const char endfalg)
{
    char * p = strstr(cookie_data.c_str(), falg);
    if (p != NULL)
    {
        p += strlen(falg);
        for( ; *p != endfalg; p++)
        {
            usename += *p;
        }
        return true;
    }
    else
        usename="";
    return false;
}
void Test( )
{
    string cookie( "name=aaaxx%" );
    string usename;
    GetCookieUseName( cookie, usename, "name=", ’;’ );
    cout << usename.c_str( ) << endl;
}

点评：

1. 严重的崩溃：没有检查p指向结束符，一旦cookie_data里有falg但没有endfalg，程序就一直for下去到崩溃。
解决：*p && *p != endfalg

2. 性能低下：usename += *p; 每个字符都调用一次basic_string& operator+=(E c)，作为一个工具类函数，要考虑效率。
解决：应该是找到最后一个后，得到其长度，然后一次性调用basic_string& append(const E *s, size_type n);

3. 逻辑错误：如果只做第一点修改，程序虽然不崩溃了，但会把falg之后所有字符当作usename, 没有endfalg应该是找不到
合法的usename。

4. 代码组织差：头重脚轻型

简单改进版本：

以下是代码片段：
bool GetCookieUseName(const string& cookie_data, string& usename,
                      const char * falg, const char endfalg)
{
    char * p = strstr(cookie_data.c_str(), falg);
    if (NULL == p)
    {
        usename="";
        return false; // not found begin tag
    }
    p += strlen(falg);
    char * q = p;
    for ( ; *q && *q != endfalg; q++)
    {}
    if (*q == endfalg)
    {
        if (q == p)
        {
            usename="";
        }
        else
        {
            usename.append( p, q - p );
        }
        return true;
    }
    usename="";
    return false; // not found end tag
}

建议继续学习：

QQ技术交流群：445447336，欢迎加入！
扫一扫订阅我的微信号：IT技术博客大学习

<< 前一篇：周末闲谈：C and C++, why use c++?

后一篇：工作在钱少事多人累的小公司里 >>

文章信息

作者：梁-兄来源： C++博客-梁兄
标签：指针重构
发布时间：2009-11-02 21:05:07

建议继续学习

近3天十大热文