重构发现：指针操作问题

C++博客-梁兄 2009-11-02 21:05:07 浏览 3,481 次

标签指针重构

原来版本：

以下是代码片段：
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
}

重构发现：指针操作问题

建议继续学习